| Conditions | 1 |
| Paths | 1 |
| Total Lines | 406 |
| Code Lines | 347 |
| Lines | 0 |
| Ratio | 0 % |
| Changes | 0 | ||
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
| 1 | <?php |
||
| 42 | protected function getCoreUpdateList() { |
||
| 43 | return [ |
||
| 44 | # rename tables 1.7.3 |
||
| 45 | # r15791 Change reserved word table names "user" and "text" |
||
| 46 | [ 'renameTable', 'user', 'mwuser' ], |
||
| 47 | [ 'renameTable', 'text', 'pagecontent' ], |
||
| 48 | [ 'renameIndex', 'mwuser', 'user_pkey', 'mwuser_pkey' ], |
||
| 49 | [ 'renameIndex', 'mwuser', 'user_user_name_key', 'mwuser_user_name_key' ], |
||
| 50 | [ 'renameIndex', 'pagecontent', 'text_pkey', 'pagecontent_pkey' ], |
||
| 51 | |||
| 52 | # renamed sequences |
||
| 53 | [ 'renameSequence', 'ipblocks_ipb_id_val', 'ipblocks_ipb_id_seq' ], |
||
| 54 | [ 'renameSequence', 'rev_rev_id_val', 'revision_rev_id_seq' ], |
||
| 55 | [ 'renameSequence', 'text_old_id_val', 'text_old_id_seq' ], |
||
| 56 | [ 'renameSequence', 'rc_rc_id_seq', 'recentchanges_rc_id_seq' ], |
||
| 57 | [ 'renameSequence', 'log_log_id_seq', 'logging_log_id_seq' ], |
||
| 58 | [ 'renameSequence', 'pr_id_val', 'page_restrictions_pr_id_seq' ], |
||
| 59 | [ 'renameSequence', 'us_id_seq', 'uploadstash_us_id_seq' ], |
||
| 60 | |||
| 61 | # since r58263 |
||
| 62 | [ 'renameSequence', 'category_id_seq', 'category_cat_id_seq' ], |
||
| 63 | |||
| 64 | # new sequences if not renamed above |
||
| 65 | [ 'addSequence', 'logging', false, 'logging_log_id_seq' ], |
||
| 66 | [ 'addSequence', 'page_restrictions', false, 'page_restrictions_pr_id_seq' ], |
||
| 67 | [ 'addSequence', 'filearchive', 'fa_id', 'filearchive_fa_id_seq' ], |
||
| 68 | [ 'addSequence', 'archive', false, 'archive_ar_id_seq' ], |
||
| 69 | [ 'addSequence', 'externallinks', false, 'externallinks_el_id_seq' ], |
||
| 70 | [ 'addSequence', 'watchlist', false, 'watchlist_wl_id_seq' ], |
||
| 71 | [ 'addSequence', 'change_tag', false, 'change_tag_ct_id_seq' ], |
||
| 72 | [ 'addSequence', 'tag_summary', false, 'tag_summary_ts_id_seq' ], |
||
| 73 | |||
| 74 | # new tables |
||
| 75 | [ 'addTable', 'category', 'patch-category.sql' ], |
||
| 76 | [ 'addTable', 'page', 'patch-page.sql' ], |
||
| 77 | [ 'addTable', 'querycachetwo', 'patch-querycachetwo.sql' ], |
||
| 78 | [ 'addTable', 'page_props', 'patch-page_props.sql' ], |
||
| 79 | [ 'addTable', 'page_restrictions', 'patch-page_restrictions.sql' ], |
||
| 80 | [ 'addTable', 'profiling', 'patch-profiling.sql' ], |
||
| 81 | [ 'addTable', 'protected_titles', 'patch-protected_titles.sql' ], |
||
| 82 | [ 'addTable', 'redirect', 'patch-redirect.sql' ], |
||
| 83 | [ 'addTable', 'updatelog', 'patch-updatelog.sql' ], |
||
| 84 | [ 'addTable', 'change_tag', 'patch-change_tag.sql' ], |
||
| 85 | [ 'addTable', 'tag_summary', 'patch-tag_summary.sql' ], |
||
| 86 | [ 'addTable', 'valid_tag', 'patch-valid_tag.sql' ], |
||
| 87 | [ 'addTable', 'user_properties', 'patch-user_properties.sql' ], |
||
| 88 | [ 'addTable', 'log_search', 'patch-log_search.sql' ], |
||
| 89 | [ 'addTable', 'l10n_cache', 'patch-l10n_cache.sql' ], |
||
| 90 | [ 'addTable', 'iwlinks', 'patch-iwlinks.sql' ], |
||
| 91 | [ 'addTable', 'module_deps', 'patch-module_deps.sql' ], |
||
| 92 | [ 'addTable', 'uploadstash', 'patch-uploadstash.sql' ], |
||
| 93 | [ 'addTable', 'user_former_groups', 'patch-user_former_groups.sql' ], |
||
| 94 | [ 'addTable', 'sites', 'patch-sites.sql' ], |
||
| 95 | [ 'addTable', 'bot_passwords', 'patch-bot_passwords.sql' ], |
||
| 96 | |||
| 97 | # Needed before new field |
||
| 98 | [ 'convertArchive2' ], |
||
| 99 | |||
| 100 | # new fields |
||
| 101 | [ 'addPgField', 'updatelog', 'ul_value', 'TEXT' ], |
||
| 102 | [ 'addPgField', 'archive', 'ar_deleted', 'SMALLINT NOT NULL DEFAULT 0' ], |
||
| 103 | [ 'addPgField', 'archive', 'ar_len', 'INTEGER' ], |
||
| 104 | [ 'addPgField', 'archive', 'ar_page_id', 'INTEGER' ], |
||
| 105 | [ 'addPgField', 'archive', 'ar_parent_id', 'INTEGER' ], |
||
| 106 | [ 'addPgField', 'archive', 'ar_content_model', 'TEXT' ], |
||
| 107 | [ 'addPgField', 'archive', 'ar_content_format', 'TEXT' ], |
||
| 108 | [ 'addPgField', 'categorylinks', 'cl_sortkey_prefix', "TEXT NOT NULL DEFAULT ''" ], |
||
| 109 | [ 'addPgField', 'categorylinks', 'cl_collation', "TEXT NOT NULL DEFAULT 0" ], |
||
| 110 | [ 'addPgField', 'categorylinks', 'cl_type', "TEXT NOT NULL DEFAULT 'page'" ], |
||
| 111 | [ 'addPgField', 'image', 'img_sha1', "TEXT NOT NULL DEFAULT ''" ], |
||
| 112 | [ 'addPgField', 'ipblocks', 'ipb_allow_usertalk', 'SMALLINT NOT NULL DEFAULT 0' ], |
||
| 113 | [ 'addPgField', 'ipblocks', 'ipb_anon_only', 'SMALLINT NOT NULL DEFAULT 0' ], |
||
| 114 | [ 'addPgField', 'ipblocks', 'ipb_by_text', "TEXT NOT NULL DEFAULT ''" ], |
||
| 115 | [ 'addPgField', 'ipblocks', 'ipb_block_email', 'SMALLINT NOT NULL DEFAULT 0' ], |
||
| 116 | [ 'addPgField', 'ipblocks', 'ipb_create_account', 'SMALLINT NOT NULL DEFAULT 1' ], |
||
| 117 | [ 'addPgField', 'ipblocks', 'ipb_deleted', 'SMALLINT NOT NULL DEFAULT 0' ], |
||
| 118 | [ 'addPgField', 'ipblocks', 'ipb_enable_autoblock', 'SMALLINT NOT NULL DEFAULT 1' ], |
||
| 119 | [ 'addPgField', 'ipblocks', 'ipb_parent_block_id', |
||
| 120 | 'INTEGER DEFAULT NULL REFERENCES ipblocks(ipb_id) ' . |
||
| 121 | 'ON DELETE SET NULL DEFERRABLE INITIALLY DEFERRED' ], |
||
| 122 | [ 'addPgField', 'filearchive', 'fa_deleted', 'SMALLINT NOT NULL DEFAULT 0' ], |
||
| 123 | [ 'addPgField', 'filearchive', 'fa_sha1', "TEXT NOT NULL DEFAULT ''" ], |
||
| 124 | [ 'addPgField', 'logging', 'log_deleted', 'SMALLINT NOT NULL DEFAULT 0' ], |
||
| 125 | [ 'addPgField', 'logging', 'log_id', |
||
| 126 | "INTEGER NOT NULL PRIMARY KEY DEFAULT nextval('logging_log_id_seq')" ], |
||
| 127 | [ 'addPgField', 'logging', 'log_params', 'TEXT' ], |
||
| 128 | [ 'addPgField', 'mwuser', 'user_editcount', 'INTEGER' ], |
||
| 129 | [ 'addPgField', 'mwuser', 'user_newpass_time', 'TIMESTAMPTZ' ], |
||
| 130 | [ 'addPgField', 'oldimage', 'oi_deleted', 'SMALLINT NOT NULL DEFAULT 0' ], |
||
| 131 | [ 'addPgField', 'oldimage', 'oi_major_mime', "TEXT NOT NULL DEFAULT 'unknown'" ], |
||
| 132 | [ 'addPgField', 'oldimage', 'oi_media_type', 'TEXT' ], |
||
| 133 | [ 'addPgField', 'oldimage', 'oi_metadata', "BYTEA NOT NULL DEFAULT ''" ], |
||
| 134 | [ 'addPgField', 'oldimage', 'oi_minor_mime', "TEXT NOT NULL DEFAULT 'unknown'" ], |
||
| 135 | [ 'addPgField', 'oldimage', 'oi_sha1', "TEXT NOT NULL DEFAULT ''" ], |
||
| 136 | [ 'addPgField', 'page', 'page_content_model', 'TEXT' ], |
||
| 137 | [ 'addPgField', 'page_restrictions', 'pr_id', |
||
| 138 | "INTEGER NOT NULL UNIQUE DEFAULT nextval('page_restrictions_pr_id_seq')" ], |
||
| 139 | [ 'addPgField', 'profiling', 'pf_memory', 'NUMERIC(18,10) NOT NULL DEFAULT 0' ], |
||
| 140 | [ 'addPgField', 'recentchanges', 'rc_deleted', 'SMALLINT NOT NULL DEFAULT 0' ], |
||
| 141 | [ 'addPgField', 'recentchanges', 'rc_log_action', 'TEXT' ], |
||
| 142 | [ 'addPgField', 'recentchanges', 'rc_log_type', 'TEXT' ], |
||
| 143 | [ 'addPgField', 'recentchanges', 'rc_logid', 'INTEGER NOT NULL DEFAULT 0' ], |
||
| 144 | [ 'addPgField', 'recentchanges', 'rc_new_len', 'INTEGER' ], |
||
| 145 | [ 'addPgField', 'recentchanges', 'rc_old_len', 'INTEGER' ], |
||
| 146 | [ 'addPgField', 'recentchanges', 'rc_params', 'TEXT' ], |
||
| 147 | [ 'addPgField', 'redirect', 'rd_interwiki', 'TEXT NULL' ], |
||
| 148 | [ 'addPgField', 'redirect', 'rd_fragment', 'TEXT NULL' ], |
||
| 149 | [ 'addPgField', 'revision', 'rev_deleted', 'SMALLINT NOT NULL DEFAULT 0' ], |
||
| 150 | [ 'addPgField', 'revision', 'rev_len', 'INTEGER' ], |
||
| 151 | [ 'addPgField', 'revision', 'rev_parent_id', 'INTEGER DEFAULT NULL' ], |
||
| 152 | [ 'addPgField', 'revision', 'rev_content_model', 'TEXT' ], |
||
| 153 | [ 'addPgField', 'revision', 'rev_content_format', 'TEXT' ], |
||
| 154 | [ 'addPgField', 'site_stats', 'ss_active_users', "INTEGER DEFAULT '-1'" ], |
||
| 155 | [ 'addPgField', 'user_newtalk', 'user_last_timestamp', 'TIMESTAMPTZ' ], |
||
| 156 | [ 'addPgField', 'logging', 'log_user_text', "TEXT NOT NULL DEFAULT ''" ], |
||
| 157 | [ 'addPgField', 'logging', 'log_page', 'INTEGER' ], |
||
| 158 | [ 'addPgField', 'interwiki', 'iw_api', "TEXT NOT NULL DEFAULT ''" ], |
||
| 159 | [ 'addPgField', 'interwiki', 'iw_wikiid', "TEXT NOT NULL DEFAULT ''" ], |
||
| 160 | [ 'addPgField', 'revision', 'rev_sha1', "TEXT NOT NULL DEFAULT ''" ], |
||
| 161 | [ 'addPgField', 'archive', 'ar_sha1', "TEXT NOT NULL DEFAULT ''" ], |
||
| 162 | [ 'addPgField', 'uploadstash', 'us_chunk_inx', "INTEGER NULL" ], |
||
| 163 | [ 'addPgField', 'job', 'job_timestamp', "TIMESTAMPTZ" ], |
||
| 164 | [ 'addPgField', 'job', 'job_random', "INTEGER NOT NULL DEFAULT 0" ], |
||
| 165 | [ 'addPgField', 'job', 'job_attempts', "INTEGER NOT NULL DEFAULT 0" ], |
||
| 166 | [ 'addPgField', 'job', 'job_token', "TEXT NOT NULL DEFAULT ''" ], |
||
| 167 | [ 'addPgField', 'job', 'job_token_timestamp', "TIMESTAMPTZ" ], |
||
| 168 | [ 'addPgField', 'job', 'job_sha1', "TEXT NOT NULL DEFAULT ''" ], |
||
| 169 | [ 'addPgField', 'archive', 'ar_id', |
||
| 170 | "INTEGER NOT NULL PRIMARY KEY DEFAULT nextval('archive_ar_id_seq')" ], |
||
| 171 | [ 'addPgField', 'externallinks', 'el_id', |
||
| 172 | "INTEGER NOT NULL PRIMARY KEY DEFAULT nextval('externallinks_el_id_seq')" ], |
||
| 173 | [ 'addPgField', 'uploadstash', 'us_props', "BYTEA" ], |
||
| 174 | |||
| 175 | # type changes |
||
| 176 | [ 'changeField', 'archive', 'ar_deleted', 'smallint', '' ], |
||
| 177 | [ 'changeField', 'archive', 'ar_minor_edit', 'smallint', |
||
| 178 | 'ar_minor_edit::smallint DEFAULT 0' ], |
||
| 179 | [ 'changeField', 'filearchive', 'fa_deleted', 'smallint', '' ], |
||
| 180 | [ 'changeField', 'filearchive', 'fa_height', 'integer', '' ], |
||
| 181 | [ 'changeField', 'filearchive', 'fa_metadata', 'bytea', "decode(fa_metadata,'escape')" ], |
||
| 182 | [ 'changeField', 'filearchive', 'fa_size', 'integer', '' ], |
||
| 183 | [ 'changeField', 'filearchive', 'fa_width', 'integer', '' ], |
||
| 184 | [ 'changeField', 'filearchive', 'fa_storage_group', 'text', '' ], |
||
| 185 | [ 'changeField', 'filearchive', 'fa_storage_key', 'text', '' ], |
||
| 186 | [ 'changeField', 'image', 'img_metadata', 'bytea', "decode(img_metadata,'escape')" ], |
||
| 187 | [ 'changeField', 'image', 'img_size', 'integer', '' ], |
||
| 188 | [ 'changeField', 'image', 'img_width', 'integer', '' ], |
||
| 189 | [ 'changeField', 'image', 'img_height', 'integer', '' ], |
||
| 190 | [ 'changeField', 'interwiki', 'iw_local', 'smallint', 'iw_local::smallint' ], |
||
| 191 | [ 'changeField', 'interwiki', 'iw_trans', 'smallint', 'iw_trans::smallint DEFAULT 0' ], |
||
| 192 | [ 'changeField', 'ipblocks', 'ipb_auto', 'smallint', 'ipb_auto::smallint DEFAULT 0' ], |
||
| 193 | [ 'changeField', 'ipblocks', 'ipb_anon_only', 'smallint', |
||
| 194 | "CASE WHEN ipb_anon_only=' ' THEN 0 ELSE ipb_anon_only::smallint END DEFAULT 0" ], |
||
| 195 | [ 'changeField', 'ipblocks', 'ipb_create_account', 'smallint', |
||
| 196 | "CASE WHEN ipb_create_account=' ' THEN 0 ELSE ipb_create_account::smallint END DEFAULT 1" ], |
||
| 197 | [ 'changeField', 'ipblocks', 'ipb_enable_autoblock', 'smallint', |
||
| 198 | "CASE WHEN ipb_enable_autoblock=' ' THEN 0 ELSE ipb_enable_autoblock::smallint END DEFAULT 1" ], |
||
| 199 | [ 'changeField', 'ipblocks', 'ipb_block_email', 'smallint', |
||
| 200 | "CASE WHEN ipb_block_email=' ' THEN 0 ELSE ipb_block_email::smallint END DEFAULT 0" ], |
||
| 201 | [ 'changeField', 'ipblocks', 'ipb_address', 'text', 'ipb_address::text' ], |
||
| 202 | [ 'changeField', 'ipblocks', 'ipb_deleted', 'smallint', 'ipb_deleted::smallint DEFAULT 0' ], |
||
| 203 | [ 'changeField', 'mwuser', 'user_token', 'text', '' ], |
||
| 204 | [ 'changeField', 'mwuser', 'user_email_token', 'text', '' ], |
||
| 205 | [ 'changeField', 'objectcache', 'keyname', 'text', '' ], |
||
| 206 | [ 'changeField', 'oldimage', 'oi_height', 'integer', '' ], |
||
| 207 | [ 'changeField', 'oldimage', 'oi_metadata', 'bytea', "decode(img_metadata,'escape')" ], |
||
| 208 | [ 'changeField', 'oldimage', 'oi_size', 'integer', '' ], |
||
| 209 | [ 'changeField', 'oldimage', 'oi_width', 'integer', '' ], |
||
| 210 | [ 'changeField', 'page', 'page_is_redirect', 'smallint', |
||
| 211 | 'page_is_redirect::smallint DEFAULT 0' ], |
||
| 212 | [ 'changeField', 'page', 'page_is_new', 'smallint', 'page_is_new::smallint DEFAULT 0' ], |
||
| 213 | [ 'changeField', 'querycache', 'qc_value', 'integer', '' ], |
||
| 214 | [ 'changeField', 'querycachetwo', 'qcc_value', 'integer', '' ], |
||
| 215 | [ 'changeField', 'recentchanges', 'rc_bot', 'smallint', 'rc_bot::smallint DEFAULT 0' ], |
||
| 216 | [ 'changeField', 'recentchanges', 'rc_deleted', 'smallint', '' ], |
||
| 217 | [ 'changeField', 'recentchanges', 'rc_minor', 'smallint', 'rc_minor::smallint DEFAULT 0' ], |
||
| 218 | [ 'changeField', 'recentchanges', 'rc_new', 'smallint', 'rc_new::smallint DEFAULT 0' ], |
||
| 219 | [ 'changeField', 'recentchanges', 'rc_type', 'smallint', 'rc_type::smallint DEFAULT 0' ], |
||
| 220 | [ 'changeField', 'recentchanges', 'rc_patrolled', 'smallint', |
||
| 221 | 'rc_patrolled::smallint DEFAULT 0' ], |
||
| 222 | [ 'changeField', 'revision', 'rev_deleted', 'smallint', 'rev_deleted::smallint DEFAULT 0' ], |
||
| 223 | [ 'changeField', 'revision', 'rev_minor_edit', 'smallint', |
||
| 224 | 'rev_minor_edit::smallint DEFAULT 0' ], |
||
| 225 | [ 'changeField', 'templatelinks', 'tl_namespace', 'smallint', 'tl_namespace::smallint' ], |
||
| 226 | [ 'changeField', 'user_newtalk', 'user_ip', 'text', 'host(user_ip)' ], |
||
| 227 | [ 'changeField', 'uploadstash', 'us_image_bits', 'smallint', '' ], |
||
| 228 | [ 'changeField', 'profiling', 'pf_time', 'float', '' ], |
||
| 229 | [ 'changeField', 'profiling', 'pf_memory', 'float', '' ], |
||
| 230 | |||
| 231 | # null changes |
||
| 232 | [ 'changeNullableField', 'oldimage', 'oi_bits', 'NULL' ], |
||
| 233 | [ 'changeNullableField', 'oldimage', 'oi_timestamp', 'NULL' ], |
||
| 234 | [ 'changeNullableField', 'oldimage', 'oi_major_mime', 'NULL' ], |
||
| 235 | [ 'changeNullableField', 'oldimage', 'oi_minor_mime', 'NULL' ], |
||
| 236 | [ 'changeNullableField', 'image', 'img_metadata', 'NOT NULL' ], |
||
| 237 | [ 'changeNullableField', 'filearchive', 'fa_metadata', 'NOT NULL' ], |
||
| 238 | [ 'changeNullableField', 'recentchanges', 'rc_cur_id', 'NULL' ], |
||
| 239 | [ 'changeNullableField', 'recentchanges', 'rc_cur_time', 'NULL' ], |
||
| 240 | |||
| 241 | [ 'checkOiDeleted' ], |
||
| 242 | |||
| 243 | # New indexes |
||
| 244 | [ 'addPgIndex', 'archive', 'archive_user_text', '(ar_user_text)' ], |
||
| 245 | [ 'addPgIndex', 'image', 'img_sha1', '(img_sha1)' ], |
||
| 246 | [ 'addPgIndex', 'ipblocks', 'ipb_parent_block_id', '(ipb_parent_block_id)' ], |
||
| 247 | [ 'addPgIndex', 'oldimage', 'oi_sha1', '(oi_sha1)' ], |
||
| 248 | [ 'addPgIndex', 'page', 'page_mediawiki_title', '(page_title) WHERE page_namespace = 8' ], |
||
| 249 | [ 'addPgIndex', 'pagelinks', 'pagelinks_title', '(pl_title)' ], |
||
| 250 | [ 'addPgIndex', 'page_props', 'pp_propname_page', '(pp_propname, pp_page)' ], |
||
| 251 | [ 'addPgIndex', 'revision', 'rev_text_id_idx', '(rev_text_id)' ], |
||
| 252 | [ 'addPgIndex', 'recentchanges', 'rc_timestamp_bot', '(rc_timestamp) WHERE rc_bot = 0' ], |
||
| 253 | [ 'addPgIndex', 'templatelinks', 'templatelinks_from', '(tl_from)' ], |
||
| 254 | [ 'addPgIndex', 'watchlist', 'wl_user', '(wl_user)' ], |
||
| 255 | [ 'addPgIndex', 'watchlist', 'wl_user_notificationtimestamp', |
||
| 256 | '(wl_user, wl_notificationtimestamp)' ], |
||
| 257 | [ 'addPgIndex', 'logging', 'logging_user_type_time', |
||
| 258 | '(log_user, log_type, log_timestamp)' ], |
||
| 259 | [ 'addPgIndex', 'logging', 'logging_page_id_time', '(log_page,log_timestamp)' ], |
||
| 260 | [ 'addPgIndex', 'iwlinks', 'iwl_prefix_from_title', '(iwl_prefix, iwl_from, iwl_title)' ], |
||
| 261 | [ 'addPgIndex', 'iwlinks', 'iwl_prefix_title_from', '(iwl_prefix, iwl_title, iwl_from)' ], |
||
| 262 | [ 'addPgIndex', 'job', 'job_timestamp_idx', '(job_timestamp)' ], |
||
| 263 | [ 'addPgIndex', 'job', 'job_sha1', '(job_sha1)' ], |
||
| 264 | [ 'addPgIndex', 'job', 'job_cmd_token', '(job_cmd, job_token, job_random)' ], |
||
| 265 | [ 'addPgIndex', 'job', 'job_cmd_token_id', '(job_cmd, job_token, job_id)' ], |
||
| 266 | [ 'addPgIndex', 'filearchive', 'fa_sha1', '(fa_sha1)' ], |
||
| 267 | [ 'addPgIndex', 'logging', 'logging_user_text_type_time', |
||
| 268 | '(log_user_text, log_type, log_timestamp)' ], |
||
| 269 | [ 'addPgIndex', 'logging', 'logging_user_text_time', '(log_user_text, log_timestamp)' ], |
||
| 270 | |||
| 271 | [ 'checkIndex', 'pagelink_unique', [ |
||
| 272 | [ 'pl_from', 'int4_ops', 'btree', 0 ], |
||
| 273 | [ 'pl_namespace', 'int2_ops', 'btree', 0 ], |
||
| 274 | [ 'pl_title', 'text_ops', 'btree', 0 ], |
||
| 275 | ], |
||
| 276 | 'CREATE UNIQUE INDEX pagelink_unique ON pagelinks (pl_from,pl_namespace,pl_title)' ], |
||
| 277 | [ 'checkIndex', 'cl_sortkey', [ |
||
| 278 | [ 'cl_to', 'text_ops', 'btree', 0 ], |
||
| 279 | [ 'cl_sortkey', 'text_ops', 'btree', 0 ], |
||
| 280 | [ 'cl_from', 'int4_ops', 'btree', 0 ], |
||
| 281 | ], |
||
| 282 | 'CREATE INDEX cl_sortkey ON "categorylinks" ' . |
||
| 283 | 'USING "btree" ("cl_to", "cl_sortkey", "cl_from")' ], |
||
| 284 | [ 'checkIndex', 'iwl_prefix_title_from', [ |
||
| 285 | [ 'iwl_prefix', 'text_ops', 'btree', 0 ], |
||
| 286 | [ 'iwl_title', 'text_ops', 'btree', 0 ], |
||
| 287 | [ 'iwl_from', 'int4_ops', 'btree', 0 ], |
||
| 288 | ], |
||
| 289 | 'CREATE INDEX iwl_prefix_title_from ON "iwlinks" ' . |
||
| 290 | 'USING "btree" ("iwl_prefix", "iwl_title", "iwl_from")' ], |
||
| 291 | [ 'checkIndex', 'logging_times', [ |
||
| 292 | [ 'log_timestamp', 'timestamptz_ops', 'btree', 0 ], |
||
| 293 | ], |
||
| 294 | 'CREATE INDEX "logging_times" ON "logging" USING "btree" ("log_timestamp")' ], |
||
| 295 | [ 'dropIndex', 'oldimage', 'oi_name' ], |
||
| 296 | [ 'checkIndex', 'oi_name_archive_name', [ |
||
| 297 | [ 'oi_name', 'text_ops', 'btree', 0 ], |
||
| 298 | [ 'oi_archive_name', 'text_ops', 'btree', 0 ], |
||
| 299 | ], |
||
| 300 | 'CREATE INDEX "oi_name_archive_name" ON "oldimage" ' . |
||
| 301 | 'USING "btree" ("oi_name", "oi_archive_name")' ], |
||
| 302 | [ 'checkIndex', 'oi_name_timestamp', [ |
||
| 303 | [ 'oi_name', 'text_ops', 'btree', 0 ], |
||
| 304 | [ 'oi_timestamp', 'timestamptz_ops', 'btree', 0 ], |
||
| 305 | ], |
||
| 306 | 'CREATE INDEX "oi_name_timestamp" ON "oldimage" ' . |
||
| 307 | 'USING "btree" ("oi_name", "oi_timestamp")' ], |
||
| 308 | [ 'checkIndex', 'page_main_title', [ |
||
| 309 | [ 'page_title', 'text_pattern_ops', 'btree', 0 ], |
||
| 310 | ], |
||
| 311 | 'CREATE INDEX "page_main_title" ON "page" ' . |
||
| 312 | 'USING "btree" ("page_title" "text_pattern_ops") WHERE ("page_namespace" = 0)' ], |
||
| 313 | [ 'checkIndex', 'page_mediawiki_title', [ |
||
| 314 | [ 'page_title', 'text_pattern_ops', 'btree', 0 ], |
||
| 315 | ], |
||
| 316 | 'CREATE INDEX "page_mediawiki_title" ON "page" ' . |
||
| 317 | 'USING "btree" ("page_title" "text_pattern_ops") WHERE ("page_namespace" = 8)' ], |
||
| 318 | [ 'checkIndex', 'page_project_title', [ |
||
| 319 | [ 'page_title', 'text_pattern_ops', 'btree', 0 ], |
||
| 320 | ], |
||
| 321 | 'CREATE INDEX "page_project_title" ON "page" ' . |
||
| 322 | 'USING "btree" ("page_title" "text_pattern_ops") ' . |
||
| 323 | 'WHERE ("page_namespace" = 4)' ], |
||
| 324 | [ 'checkIndex', 'page_talk_title', [ |
||
| 325 | [ 'page_title', 'text_pattern_ops', 'btree', 0 ], |
||
| 326 | ], |
||
| 327 | 'CREATE INDEX "page_talk_title" ON "page" ' . |
||
| 328 | 'USING "btree" ("page_title" "text_pattern_ops") ' . |
||
| 329 | 'WHERE ("page_namespace" = 1)' ], |
||
| 330 | [ 'checkIndex', 'page_user_title', [ |
||
| 331 | [ 'page_title', 'text_pattern_ops', 'btree', 0 ], |
||
| 332 | ], |
||
| 333 | 'CREATE INDEX "page_user_title" ON "page" ' . |
||
| 334 | 'USING "btree" ("page_title" "text_pattern_ops") WHERE ' . |
||
| 335 | '("page_namespace" = 2)' ], |
||
| 336 | [ 'checkIndex', 'page_utalk_title', [ |
||
| 337 | [ 'page_title', 'text_pattern_ops', 'btree', 0 ], |
||
| 338 | ], |
||
| 339 | 'CREATE INDEX "page_utalk_title" ON "page" ' . |
||
| 340 | 'USING "btree" ("page_title" "text_pattern_ops") ' . |
||
| 341 | 'WHERE ("page_namespace" = 3)' ], |
||
| 342 | [ 'checkIndex', 'ts2_page_text', [ |
||
| 343 | [ 'textvector', 'tsvector_ops', 'gist', 0 ], |
||
| 344 | ], |
||
| 345 | 'CREATE INDEX "ts2_page_text" ON "pagecontent" USING "gist" ("textvector")' ], |
||
| 346 | [ 'checkIndex', 'ts2_page_title', [ |
||
| 347 | [ 'titlevector', 'tsvector_ops', 'gist', 0 ], |
||
| 348 | ], |
||
| 349 | 'CREATE INDEX "ts2_page_title" ON "page" USING "gist" ("titlevector")' ], |
||
| 350 | |||
| 351 | [ 'checkOiNameConstraint' ], |
||
| 352 | [ 'checkPageDeletedTrigger' ], |
||
| 353 | [ 'checkRevUserFkey' ], |
||
| 354 | [ 'dropIndex', 'ipblocks', 'ipb_address' ], |
||
| 355 | [ 'checkIndex', 'ipb_address_unique', [ |
||
| 356 | [ 'ipb_address', 'text_ops', 'btree', 0 ], |
||
| 357 | [ 'ipb_user', 'int4_ops', 'btree', 0 ], |
||
| 358 | [ 'ipb_auto', 'int2_ops', 'btree', 0 ], |
||
| 359 | [ 'ipb_anon_only', 'int2_ops', 'btree', 0 ], |
||
| 360 | ], |
||
| 361 | 'CREATE UNIQUE INDEX ipb_address_unique ' . |
||
| 362 | 'ON ipblocks (ipb_address,ipb_user,ipb_auto,ipb_anon_only)' ], |
||
| 363 | |||
| 364 | [ 'checkIwlPrefix' ], |
||
| 365 | |||
| 366 | # All FK columns should be deferred |
||
| 367 | [ 'changeFkeyDeferrable', 'archive', 'ar_user', 'mwuser(user_id) ON DELETE SET NULL' ], |
||
| 368 | [ 'changeFkeyDeferrable', 'categorylinks', 'cl_from', 'page(page_id) ON DELETE CASCADE' ], |
||
| 369 | [ 'changeFkeyDeferrable', 'externallinks', 'el_from', 'page(page_id) ON DELETE CASCADE' ], |
||
| 370 | [ 'changeFkeyDeferrable', 'filearchive', 'fa_deleted_user', |
||
| 371 | 'mwuser(user_id) ON DELETE SET NULL' ], |
||
| 372 | [ 'changeFkeyDeferrable', 'filearchive', 'fa_user', 'mwuser(user_id) ON DELETE SET NULL' ], |
||
| 373 | [ 'changeFkeyDeferrable', 'image', 'img_user', 'mwuser(user_id) ON DELETE SET NULL' ], |
||
| 374 | [ 'changeFkeyDeferrable', 'imagelinks', 'il_from', 'page(page_id) ON DELETE CASCADE' ], |
||
| 375 | [ 'changeFkeyDeferrable', 'ipblocks', 'ipb_by', 'mwuser(user_id) ON DELETE CASCADE' ], |
||
| 376 | [ 'changeFkeyDeferrable', 'ipblocks', 'ipb_user', 'mwuser(user_id) ON DELETE SET NULL' ], |
||
| 377 | [ 'changeFkeyDeferrable', 'ipblocks', 'ipb_parent_block_id', |
||
| 378 | 'ipblocks(ipb_id) ON DELETE SET NULL' ], |
||
| 379 | [ 'changeFkeyDeferrable', 'langlinks', 'll_from', 'page(page_id) ON DELETE CASCADE' ], |
||
| 380 | [ 'changeFkeyDeferrable', 'logging', 'log_user', 'mwuser(user_id) ON DELETE SET NULL' ], |
||
| 381 | [ 'changeFkeyDeferrable', 'oldimage', 'oi_name', |
||
| 382 | 'image(img_name) ON DELETE CASCADE ON UPDATE CASCADE' ], |
||
| 383 | [ 'changeFkeyDeferrable', 'oldimage', 'oi_user', 'mwuser(user_id) ON DELETE SET NULL' ], |
||
| 384 | [ 'changeFkeyDeferrable', 'pagelinks', 'pl_from', 'page(page_id) ON DELETE CASCADE' ], |
||
| 385 | [ 'changeFkeyDeferrable', 'page_props', 'pp_page', 'page (page_id) ON DELETE CASCADE' ], |
||
| 386 | [ 'changeFkeyDeferrable', 'page_restrictions', 'pr_page', |
||
| 387 | 'page(page_id) ON DELETE CASCADE' ], |
||
| 388 | [ 'changeFkeyDeferrable', 'protected_titles', 'pt_user', |
||
| 389 | 'mwuser(user_id) ON DELETE SET NULL' ], |
||
| 390 | [ 'changeFkeyDeferrable', 'recentchanges', 'rc_user', |
||
| 391 | 'mwuser(user_id) ON DELETE SET NULL' ], |
||
| 392 | [ 'changeFkeyDeferrable', 'redirect', 'rd_from', 'page(page_id) ON DELETE CASCADE' ], |
||
| 393 | [ 'changeFkeyDeferrable', 'revision', 'rev_page', 'page (page_id) ON DELETE CASCADE' ], |
||
| 394 | [ 'changeFkeyDeferrable', 'revision', 'rev_user', 'mwuser(user_id) ON DELETE RESTRICT' ], |
||
| 395 | [ 'changeFkeyDeferrable', 'templatelinks', 'tl_from', 'page(page_id) ON DELETE CASCADE' ], |
||
| 396 | [ 'changeFkeyDeferrable', 'user_groups', 'ug_user', 'mwuser(user_id) ON DELETE CASCADE' ], |
||
| 397 | [ 'changeFkeyDeferrable', 'user_newtalk', 'user_id', 'mwuser(user_id) ON DELETE CASCADE' ], |
||
| 398 | [ 'changeFkeyDeferrable', 'user_properties', 'up_user', |
||
| 399 | 'mwuser(user_id) ON DELETE CASCADE' ], |
||
| 400 | [ 'changeFkeyDeferrable', 'watchlist', 'wl_user', 'mwuser(user_id) ON DELETE CASCADE' ], |
||
| 401 | |||
| 402 | # r81574 |
||
| 403 | [ 'addInterwikiType' ], |
||
| 404 | # end |
||
| 405 | [ 'tsearchFixes' ], |
||
| 406 | |||
| 407 | // 1.23 |
||
| 408 | [ 'addPgField', 'recentchanges', 'rc_source', "TEXT NOT NULL DEFAULT ''" ], |
||
| 409 | [ 'addPgField', 'page', 'page_links_updated', "TIMESTAMPTZ NULL" ], |
||
| 410 | [ 'addPgField', 'mwuser', 'user_password_expires', 'TIMESTAMPTZ NULL' ], |
||
| 411 | [ 'changeFieldPurgeTable', 'l10n_cache', 'lc_value', 'bytea', |
||
| 412 | "replace(lc_value,'\','\\\\')::bytea" ], |
||
| 413 | // 1.23.9 |
||
| 414 | [ 'rebuildTextSearch' ], |
||
| 415 | |||
| 416 | // 1.24 |
||
| 417 | [ 'addPgField', 'page_props', 'pp_sortkey', 'float NULL' ], |
||
| 418 | [ 'addPgIndex', 'page_props', 'pp_propname_sortkey_page', |
||
| 419 | '( pp_propname, pp_sortkey, pp_page ) WHERE ( pp_sortkey IS NOT NULL )' ], |
||
| 420 | [ 'addPgField', 'page', 'page_lang', 'TEXT default NULL' ], |
||
| 421 | [ 'addPgField', 'pagelinks', 'pl_from_namespace', 'INTEGER NOT NULL DEFAULT 0' ], |
||
| 422 | [ 'addPgField', 'templatelinks', 'tl_from_namespace', 'INTEGER NOT NULL DEFAULT 0' ], |
||
| 423 | [ 'addPgField', 'imagelinks', 'il_from_namespace', 'INTEGER NOT NULL DEFAULT 0' ], |
||
| 424 | |||
| 425 | // 1.25 |
||
| 426 | [ 'dropTable', 'hitcounter' ], |
||
| 427 | [ 'dropField', 'site_stats', 'ss_total_views', 'patch-drop-ss_total_views.sql' ], |
||
| 428 | [ 'dropField', 'page', 'page_counter', 'patch-drop-page_counter.sql' ], |
||
| 429 | [ 'dropFkey', 'recentchanges', 'rc_cur_id' ], |
||
| 430 | |||
| 431 | // 1.27 |
||
| 432 | [ 'dropTable', 'msg_resource_links' ], |
||
| 433 | [ 'dropTable', 'msg_resource' ], |
||
| 434 | [ |
||
| 435 | 'addPgField', 'watchlist', 'wl_id', |
||
| 436 | "INTEGER NOT NULL PRIMARY KEY DEFAULT nextval('watchlist_wl_id_seq')" |
||
| 437 | ], |
||
| 438 | |||
| 439 | // 1.28 |
||
| 440 | [ 'addPgIndex', 'recentchanges', 'rc_name_type_patrolled_timestamp', |
||
| 441 | '( rc_namespace, rc_type, rc_patrolled, rc_timestamp )' ], |
||
| 442 | [ 'addPgField', 'change_tag', 'ct_id', |
||
| 443 | "INTEGER NOT NULL PRIMARY KEY DEFAULT nextval('change_tag_ct_id_seq')" ], |
||
| 444 | [ 'addPgField', 'tag_summary', 'ts_id', |
||
| 445 | "INTEGER NOT NULL PRIMARY KEY DEFAULT nextval('tag_summary_ts_id_seq')" ], |
||
| 446 | ]; |
||
| 447 | } |
||
| 448 | |||
| 984 |
There are different options of fixing this problem.
If you want to be on the safe side, you can add an additional type-check:
If you are sure that the expression is traversable, you might want to add a doc comment cast to improve IDE auto-completion and static analysis:
Mark the issue as a false-positive: Just hover the remove button, in the top-right corner of this issue for more options.