is_int() 調整
trunk/include/controller/user_manager_class.php (diff) trunk/include/option/option_class.php (diff) trunk/include/option/option_form_class.php (diff)| @@ -35,13 +35,25 @@ | ||
| 35 | 35 | $str = UserManagerMessage::ERROR_INPUT_TEXT . $back_url; |
| 36 | 36 | $empty = UserManagerMessage::ERROR_INPUT_EMPTY . $back_url; |
| 37 | 37 | if ($user_no < 1) { |
| 38 | - if ($uname == '') self::OutputError($title, UserManagerMessage::UNAME . $str); | |
| 39 | - if ($password == '') self::OutputError($title, UserManagerMessage::PASSWORD . $str); | |
| 38 | + if ($uname == '') { | |
| 39 | + self::OutputError($title, UserManagerMessage::UNAME . $str); | |
| 40 | + } | |
| 41 | + if ($password == '') { | |
| 42 | + self::OutputError($title, UserManagerMessage::PASSWORD . $str); | |
| 43 | + } | |
| 40 | 44 | } |
| 41 | - if ($handle_name == '') self::OutputError($title, UserManagerMessage::HANDLE_NAME . $str); | |
| 42 | - if (empty($sex)) self::OutputError($title, UserManagerMessage::SEX . $empty); | |
| 43 | - if (empty($role)) self::OutputError($title, UserManagerMessage::WISH_ROLE . $empty); | |
| 44 | - if (! is_int($icon_no)) self::OutputError($title, UserManagerMessage::ICON_NUMBER . $empty); | |
| 45 | + if ($handle_name == '') { | |
| 46 | + self::OutputError($title, UserManagerMessage::HANDLE_NAME . $str); | |
| 47 | + } | |
| 48 | + if (empty($sex)) { | |
| 49 | + self::OutputError($title, UserManagerMessage::SEX . $empty); | |
| 50 | + } | |
| 51 | + if (empty($role)) { | |
| 52 | + self::OutputError($title, UserManagerMessage::WISH_ROLE . $empty); | |
| 53 | + } | |
| 54 | + if (false === is_int($icon_no)) { | |
| 55 | + self::OutputError($title, UserManagerMessage::ICON_NUMBER . $empty); | |
| 56 | + } | |
| 45 | 57 | |
| 46 | 58 | //文字数制限チェック |
| 47 | 59 | $format = UserManagerMessage::ERROR_TEXT_LIMIT . $back_url; |
| @@ -356,7 +356,9 @@ | ||
| 356 | 356 | //選択値セット |
| 357 | 357 | protected function SetFormValue($type) { |
| 358 | 358 | foreach ($this->form_list as $key => $value) { |
| 359 | - if ($type == 'int' && ! is_int($key)) continue; | |
| 359 | + if ($type == 'int' && false === is_int($key)) { | |
| 360 | + continue; | |
| 361 | + } | |
| 360 | 362 | |
| 361 | 363 | if (DB::$ROOM->IsOption($type == 'key' ? $key : $value)) { |
| 362 | 364 | $this->value = $value; |
| @@ -33,7 +33,7 @@ | ||
| 33 | 33 | public static function Output() { |
| 34 | 34 | $class = ''; |
| 35 | 35 | foreach (self::$order as $group => $name) { |
| 36 | - if (! is_int($group)) { | |
| 36 | + if (false === is_int($group)) { | |
| 37 | 37 | $class = sprintf(' class="%s"', $group); //class 切り替え |
| 38 | 38 | } |
| 39 | 39 | is_null($name) ? self::OutputSeparator($group) : self::OutputForm($name, $class); |