PEAK XOOPS - Re: イベント通知における2つの問題 in englishin japanese

Top  >  Comments  >  News comment ()

Re: イベント通知における2つの問題

  • As this forum is only for commentation, you cannot open a new topic
  • Guests cannot post into this forum
Target News
Subject イベント通知における2つの問題
Summary ayumiさんの指摘でイベント通知を見直しているのですが、ここに、いくつもの整合性チェック漏れがあって、かなり困った状況になっています。(1) カーネル側のチェック漏れadmin_only なイベントも、管理者以外が登録できる(単にフォームに表示されていないだけ)(2) 閲覧...
Previous post - Next post | Parent - Children.1 .2 | Posted on 2006/6/16 12:45
tohokuaiki  上等兵   Posts: 25
XOOPSの権限チェックは、しっかりしてそうに見えて結構いい加減ですよね。

理由は、
1・ページコントローラのために実装はモジュール側で100%必要
2・しかし、コアが与えてくれるgpermはいまいち使いにくい

1は、各モジュールの作者の力量と理解度によるんですが、XOOPS側で簡単に使えるクラスを作っておいてくれないかなぁと。
xoops_grouppermissionはいまいちなんですよね・・・。
どこがいまいちか忘れましたが、いまいちがゆえに、exFrameではexPermクラス、xanhteでも権限用のXanhte_GpermManagerクラスでxoops_grouppermissionをラップするクラスを作成しています。PEAKシリーズで言うと、read_config.phpですよね。確か。
Votes:1 Average:10.00
Previous post - Next post | Parent - No child | Posted on 2006/6/16 15:37
kurak_bu  上等兵   Posts: 34
report issue to xoops dev team
Votes:1 Average:0.00
Previous post - Next post | Parent - No child | Posted on 2006/6/17 5:06
GIJOE  先任軍曹   Posts: 4110
Quote:
XOOPSの権限チェックは、しっかりしてそうに見えて結構いい加減ですよね。

理由は、
1・ページコントローラのために実装はモジュール側で100%必要
2・しかし、コアが与えてくれるgpermはいまいち使いにくい

1は、各モジュールの作者の力量と理解度によるんですが、XOOPS側で簡単に使えるクラスを作っておいてくれないかなぁと。
xoops_grouppermissionはいまいちなんですよね・・・。
groupperm.php は数少ない「onokazu製であることが保証されたファイル」ですから。
つまり、使わないのが吉です。
Catzwolfさんも本家で、「なんでこんなコードなんだ!」とか怒ってましたが、onokazuさんは例によって知らんぷりでした。

でも、groupperm.phpも単なるユーティリティクラスです。
使う使わないは、モジュール作者が決めることで、このあたりの自由度がXOOPSの魅力とも言えます。
自分のモジュールだけはちゃんと作ればいいんですよ。

「スキルの信用できる作者のモジュールしか使わない」

これ、XOOPSの鉄則ですよね。
(そうだ。newsもさっさとbulletinに移行しなきゃ)

nitificationもある意味ユーティリティクラスです。おざなりに使うから権限指定が反映されないだけで、正しい使い方(必ずuser_listを指定する)さえ知っていれば、とても便利ですよね。コードの質もかなりまともです。
Votes:1 Average:0.00

  Advanced search


Login
Username or e-mail:

Password:

Remember Me

Lost Password?

Register now!