PEAK XOOPS - two problems in xoops_notification in englishin japanese

Archive | RSS |
XOOPS
XOOPS : two problems in xoops_notification
Poster : GIJOE on 2006-06-16 05:07:40 (7679 reads)

in englishin japanese
I've just found "notification" has two problems.

(1) 'admin_only' has non-sense (kernel's bug)
(2) view permissions in the module have non-sense

core developper should check (1).
module developper should check (2) again.

At least newbb1 has bug of (2) about "private forum".
Any normal user can read whole of private post using (1) and (2).

Printer friendly page Send this story to a friend

Comments list

GIJOE  Posted on 2006/6/17 5:06
Quote:
XOOPSの権限チェックは、しっかりしてそうに見えて結構いい加減ですよね。

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

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

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

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

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

nitificationもある意味ユーティリティクラスです。おざなりに使うから権限指定が反映されないだけで、正しい使い方(必ずuser_listを指定する)さえ知っていれば、とても便利ですよね。コードの質もかなりまともです。
kurak_bu  Posted on 2006/6/16 15:37
report issue to xoops dev team
tohokuaiki  Posted on 2006/6/16 12:45
XOOPSの権限チェックは、しっかりしてそうに見えて結構いい加減ですよね。

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

1は、各モジュールの作者の力量と理解度によるんですが、XOOPS側で簡単に使えるクラスを作っておいてくれないかなぁと。
xoops_grouppermissionはいまいちなんですよね・・・。
どこがいまいちか忘れましたが、いまいちがゆえに、exFrameではexPermクラス、xanhteでも権限用のXanhte_GpermManagerクラスでxoops_grouppermissionをラップするクラスを作成しています。PEAKシリーズで言うと、read_config.phpですよね。確か。
Login
Username or e-mail:

Password:

Remember Me

Lost Password?

Register now!