Changeset View
Changeset View
Standalone View
Standalone View
core/units/mailing_lists/mailing_lists_config.php
Show First 20 Lines • Show All 97 Lines • ▼ Show 20 Line(s) | |||||
), | ), | ||||
'Fields' => Array ( | 'Fields' => Array ( | ||||
'MailingId' => Array ('type' => 'int', 'not_null' => 1, 'default' => 0), | 'MailingId' => Array ('type' => 'int', 'not_null' => 1, 'default' => 0), | ||||
'PortalUserId' => Array ( | 'PortalUserId' => Array ( | ||||
'type' => 'int', | 'type' => 'int', | ||||
'formatter' => 'kLEFTFormatter', | 'formatter' => 'kLEFTFormatter', | ||||
'options' => Array (USER_ROOT => 'root'), | 'options' => Array (USER_ROOT => 'root', USER_SYSTEM => 'system'), | ||||
Lint: CodingStandard.Arrays.Array.SpaceAfterKeyword: There must be no space between the Array keyword and the opening parenthesis | |||||
PHP keywords must be lowercase; expected "array" but found "Array" Lint: Generic.PHP.LowerCaseKeyword.Found: PHP keywords must be lowercase; expected "array" but found "Array" | |||||
Done ReplyFor every change like this please update the test plan to include visiting the corresponding section with a non-empty grid to allow checking (faking DB field value in PMA to have -3 is perfectly fine to see if the "system" word is actually displayed), that there is no SQL error. alex: For every change like this please update the test plan to include visiting the corresponding… | |||||
'left_sql' => 'SELECT %s FROM ' . TABLE_PREFIX . 'Users WHERE %s', | 'left_sql' => 'SELECT %s FROM ' . TABLE_PREFIX . 'Users WHERE %s', | ||||
'left_key_field' => 'PortalUserId', 'left_title_field' => USER_TITLE_FIELD, | 'left_key_field' => 'PortalUserId', 'left_title_field' => USER_TITLE_FIELD, | ||||
'required' => 1, 'not_null' => 1, 'default' => USER_ROOT, | 'required' => 1, 'not_null' => 1, 'default' => USER_ROOT, | ||||
), | ), | ||||
'To' => Array ('type' => 'string', 'required' => 1, 'default' => NULL), | 'To' => Array ('type' => 'string', 'required' => 1, 'default' => NULL), | ||||
'ToParsed' => Array ('type' => 'string', 'default' => NULL), | 'ToParsed' => Array ('type' => 'string', 'default' => NULL), | ||||
▲ Show 20 Lines • Show All 42 Lines • Show Last 20 Lines |
There must be no space between the Array keyword and the opening parenthesis