Fatal error on Legacy_ModuleUninstaller

• 回報者: mumincacao
• 負責人: minahito
• 類型: XOOPSCube2.1 Bugs
• 狀態: 關閉
• 元件: (無)
• 里程碑: (無)
• 優先權: 3
• 嚴重程度: 5 - 中
• 處理結果: 修正
• 檔案: 無

細節 回覆

/modules/legacy/admin/class/ModuleUninstaller.class.php
L168-173
#10089 と似たようなものだけど
Legacy_ModuleUninstaller::_processReport() にて

if (is_object($this->_mXoopsModule)) {
$this->mLog->addError(XCube_Utils::formatMessage
(_AD_LEGACY_ERROR_UNINSTALLATION_MODULE_FAILURE, $this-
>_mXoopsModule->get('name')));
}
else {
$this->mLog->addError(XCube_Utils::formatMessage
(_AD_LEGACY_ERROR_UNINSTALLATION_MODULE_FAILURE, $this-
>_mXoopsModule->get('dirname')));
}

の else 内は $this->_mXoopsModule がおぶじぇくとじゃな
いときに通過するので
$this->_mXoopsModule->get('dirname')
で落ちちゃいそうです
updater では is_object ちぇっくはしてなかったけどどれか
に合わせたほうがいいのかなぁ？

Ticket History (3/4 Histories)

2007-03-16 19:58 Updated by: minahito

評語 回覆

Logged In: YES
user_id=8353

[EN]
/modules/legacy/admin/class/ModuleUninstaller.class.php
L168-173

This is like #10089.

In Legacy_ModuleUninstaller::_processReport();

if (is_object($this->_mXoopsModule)) {
$this->mLog->addError(XCube_Utils::formatMessage
(_AD_LEGACY_ERROR_UNINSTALLATION_MODULE_FAILURE, $this-
>_mXoopsModule->get('name')));
}
else {
$this->mLog->addError(XCube_Utils::formatMessage
(_AD_LEGACY_ERROR_UNINSTALLATION_MODULE_FAILURE, $this-
>_mXoopsModule->get('dirname')));
}

Else-Code is passed when $this->_mXoopsModule is not an
object, $this->_mXoopsModule->get('dirname') may be fatal
error. Updater doesn't check is_object(), so these check
code s have to be uniformed.

2007-03-16 20:11 Updated by: minahito

評語 回覆

Logged In: YES
user_id=8353

[JA]
前回のにもちょこっと書きましたが、３クラスとも
$this->_mXoopsModule の状態はアクション保証の precondition
にして削りましょうか。実際全部アクションがチェックしているの
で通過しないコードパスですし、そっちのほうが手っ取り早そうです。

[EN]
Shall we handle "$this->_mXoopsModule is an object or not?"
as the precondition of these three classes. In fact, three
actions about these classes check such preconditions. If
deletes those code which is never passed, we can fix it rapidly.

2007-03-16 20:23 Updated by: minahito

• 負責人 Update from (無) to minahito
• 處理結果 Update from 無 to Accepted

2007-03-20 12:09 Updated by: minahito

• Ticket Close date is changed to 2007-03-20 12:09
• 優先權 Update from 5 - 中 to 3
• 處理結果 Update from Accepted to 修正
• 狀態 Update from 開啟 to 關閉