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.