待辦事項 #10107

problem process on updatePreference
啟用日期: 2007-03-16 16:45 最後更新: 2007-03-20 17:07

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

細節

/modules/Legacy/admin/class/Legacy_ModuleInstallUtils.c
lass.php

Legacy_ModuleInstallUtils::updatePreferenceByInfo() の
L1002 - 1004 にある
if (count($configArr) == 0) {
$log->addError('Execption Error: Could not find
config.');
}

Legacy_ModuleInstallUtils::updatePreferenceOrderByInfo
() のL1063 - 1065 にある
if (!(count($configArr) > 0 && is_object($configArr
[0]))) {
$log->addError('Execption Error: Could not find
config.');
}

の2箇所はえらーを出した後に return しておいたほうがよ
くないですか?
このまま処理を続けると
$config->set(...);
のところで Fatal Error が出ちゃうと思います
sumartUpdate の判定をみすしなければたぶんきっと通らない
処理だけど・・・

ついでにおんなじ判定をしてるからえらーの条件も合わせた
ほうがいいかも?

Ticket History (2/2 Histories)

2007-03-20 12:11 Updated by: minahito
  • 負責人 Update from (無) to minahito
  • 優先權 Update from 5 - 中 to 3
  • 處理結果 Update from to Accepted
2007-03-20 17:07 Updated by: minahito
  • Ticket Close date is changed to 2007-03-20 17:07
  • 處理結果 Update from Accepted to 修正
  • 狀態 Update from 開啟 to 關閉

Attachment File List

No attachments

編輯

You are not logged in. I you are not logged in, your comment will be treated as an anonymous post. » 登入