| Conditions | 1 |
| Paths | 1 |
| Total Lines | 51 |
| Code Lines | 38 |
| Lines | 0 |
| Ratio | 0 % |
| Changes | 1 | ||
| Bugs | 0 | Features | 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 |
||
| 38 | protected function getCoreUpdateList() { |
||
| 39 | return [ |
||
| 40 | // 1.23 |
||
| 41 | [ 'addField', 'mwuser', 'user_password_expires', 'patch-user_password_expires.sql' ], |
||
| 42 | |||
| 43 | // 1.24 |
||
| 44 | [ 'addField', 'page', 'page_lang', 'patch-page_page_lang.sql' ], |
||
| 45 | |||
| 46 | // 1.25 |
||
| 47 | [ 'dropTable', 'hitcounter' ], |
||
| 48 | [ 'dropField', 'site_stats', 'ss_total_views', 'patch-drop-ss_total_views.sql' ], |
||
| 49 | [ 'dropField', 'page', 'page_counter', 'patch-drop-page_counter.sql' ], |
||
| 50 | // scripts were updated in 1.27 due to SQL errors; retaining old updatekeys so that people |
||
| 51 | // updating from 1.23->1.25->1.27 do not execute these scripts twice even though the |
||
| 52 | // updatekeys no longer make sense as they are. |
||
| 53 | [ 'updateSchema', 'categorylinks', 'cl_type-category_types-ck', |
||
| 54 | 'patch-categorylinks-constraints.sql' ], |
||
| 55 | [ 'updateSchema', 'filearchive', 'fa_major_mime-major_mime-ck', |
||
| 56 | 'patch-filearchive-constraints.sql' ], |
||
| 57 | [ 'updateSchema', 'oldimage', 'oi_major_mime-major_mime-ck', |
||
| 58 | 'patch-oldimage-constraints.sql' ], |
||
| 59 | [ 'updateSchema', 'image', 'img_major_mime-major_mime-ck', 'patch-image-constraints.sql' ], |
||
| 60 | [ 'updateSchema', 'uploadstash', 'us_media_type-media_type-ck', |
||
| 61 | 'patch-uploadstash-constraints.sql' ], |
||
| 62 | |||
| 63 | [ 'modifyField', 'image', 'img_major_mime', |
||
| 64 | 'patch-img_major_mime-chemical.sql' ], |
||
| 65 | [ 'modifyField', 'oldimage', 'oi_major_mime', |
||
| 66 | 'patch-oi_major_mime-chemical.sql' ], |
||
| 67 | [ 'modifyField', 'filearchive', 'fa_major_mime', |
||
| 68 | 'patch-fa_major_mime-chemical.sql' ], |
||
| 69 | |||
| 70 | // 1.27 |
||
| 71 | [ 'dropTable', 'msg_resource_links' ], |
||
| 72 | [ 'dropTable', 'msg_resource' ], |
||
| 73 | [ 'addField', 'watchlist', 'wl_id', 'patch-watchlist-wl_id.sql' ], |
||
| 74 | [ 'dropField', 'mwuser', 'user_options', 'patch-drop-user_options.sql' ], |
||
| 75 | [ 'addTable', 'bot_passwords', 'patch-bot_passwords.sql' ], |
||
| 76 | [ 'addField', 'pagelinks', 'pl_from_namespace', 'patch-pl_from_namespace.sql' ], |
||
| 77 | [ 'addField', 'templatelinks', 'tl_from_namespace', 'patch-tl_from_namespace.sql' ], |
||
| 78 | [ 'addField', 'imagelinks', 'il_from_namespace', 'patch-il_from_namespace.sql' ], |
||
| 79 | [ 'dropIndex', 'categorylinks', 'cl_collation', 'patch-kill-cl_collation_index.sql' ], |
||
| 80 | [ 'addIndex', 'categorylinks', 'cl_collation_ext', |
||
| 81 | 'patch-add-cl_collation_ext_index.sql' ], |
||
| 82 | [ 'dropField', 'recentchanges', 'rc_cur_time', 'patch-drop-rc_cur_time.sql' ], |
||
| 83 | [ 'addField', 'page_props', 'pp_sortkey', 'patch-pp_sortkey.sql' ], |
||
| 84 | [ 'updateSchema', 'oldimage', 'oldimage varchar', 'patch-oldimage-schema.sql' ], |
||
| 85 | [ 'updateSchema', 'filearchive', 'filearchive varchar', 'patch-filearchive-schema.sql' ], |
||
| 86 | [ 'updateSchema', 'image', 'image varchar', 'patch-image-schema.sql' ] |
||
| 87 | ]; |
||
| 88 | } |
||
| 89 | |||
| 121 |