Fatal error on Legacy_ModuleUninstaller

• Rapporteur: mumincacao
• Propriétaire: minahito
• Type: XOOPSCube2.1 Bugs
• État: Atteints
• Composant: (Aucun)
• Jalon: (Aucun)
• Priorité: 3
• Sévérité: 5 - moyen
• Résolution: Fixed
• Fichier: Aucun

Détails Répondre

/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

Commentaire Répondre

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

Commentaire Répondre

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

• Propriétaire Update from (Aucun) to minahito
• Résolution Update from Aucun to Accepted

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

• Ticket Close date is changed to 2007-03-20 12:09
• Priorité Update from 5 - moyen to 3
• Résolution Update from Accepted to Fixed
• État Update from Ouvert to Atteints