Does XoopsModule class need clean up?

• 報告者: minahito
• 担当者: (del#8225)
• チケットの種類: XOOPSCube2.1 Bugs
• 状況: 完了
• コンポーネント: (未割り当て)
• マイルストーン: (未割り当て)
• 優先度: 4
• 重要度: 5 - 中
• 解決法: 修正済み
• ファイル: なし

詳細 返信

Reported by Marijuana.

This item have the relation with 9759.

http://sourceforge.jp/tracker/index.php?func=detail&aid=9759&group_id=64&atid=7892

Some junk lines are in kernel/module.php. 'version' may
be a required item in xoops_version.php. Therefore,
line 214 'if' line can be removed. Also, line 89
'floatval' can be removed.


Tom_G3X said,
The method we get the version should be
Legacy_Utils::convertVersionFromModinfoToInt().

チケットの履歴 (2 件中 2 件表示)

2007-01-14 01:25 更新者: (del#8225)

• 担当者 が (未割り当て) から nobunobu に更新されました
• 解決法 が なし から 修正済み に更新されました

コメント 返信

Logged In: YES
user_id=8225

I changed a kernel/module.php source and commited along the
following lines.

In order to keep compatibility, 'version' should not be a
required parameter in xoops_version.php.
To keep compatibility, I add a logic that sets 0 to
'version', if $modversion['version'] is not set in
xoops_version.php.

In order to unify a way of process, to get version as
integer value should call
Legacy_Utils::convertVersionFromModinfoToInt()

2007-01-14 01:25 更新者: (del#8225)

• チケット完了時刻 が 2007-01-14 01:25 に更新されました
• 状況 が オープン から 完了 に更新されました