|
1
|
|
|
<?php |
|
|
|
|
|
|
2
|
|
|
/** |
|
3
|
|
|
* Upgrader from to 2.4.x to 2.5.0 |
|
4
|
|
|
* |
|
5
|
|
|
* You may not change or alter any portion of this comment or credits |
|
6
|
|
|
* of supporting developers from this source code or any supporting source code |
|
7
|
|
|
* which is considered copyrighted (c) material of the original comment or credit authors. |
|
8
|
|
|
* This program is distributed in the hope that it will be useful, |
|
9
|
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of |
|
10
|
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. |
|
11
|
|
|
* |
|
12
|
|
|
* @copyright (c) 2000-2016 XOOPS Project (www.xoops.org) |
|
13
|
|
|
* @license GNU GPL 2 (http://www.gnu.org/licenses/gpl-2.0.html) |
|
14
|
|
|
* @package upgrader |
|
15
|
|
|
* @since 2.5.0 |
|
16
|
|
|
* @author Andricq Nicolas (AKA MusS) |
|
17
|
|
|
*/ |
|
18
|
|
|
|
|
19
|
|
|
require_once 'dbmanager.php'; |
|
20
|
|
|
|
|
21
|
|
|
/** |
|
22
|
|
|
* Class upgrade_250 |
|
23
|
|
|
*/ |
|
24
|
|
|
class Upgrade_250 extends XoopsUpgrade |
|
25
|
|
|
{ |
|
26
|
|
|
/** |
|
27
|
|
|
* Check if cpanel config already exists |
|
28
|
|
|
* |
|
29
|
|
|
*/ |
|
30
|
|
View Code Duplication |
public function check_config() |
|
31
|
|
|
{ |
|
32
|
|
|
$sql = 'SELECT COUNT(*) FROM `' . $GLOBALS['xoopsDB']->prefix('config') . "` WHERE `conf_name` IN ('break1', 'usetips')"; |
|
33
|
|
|
if (!$result = $GLOBALS['xoopsDB']->queryF($sql)) { |
|
34
|
|
|
return false; |
|
35
|
|
|
} |
|
36
|
|
|
list($count) = $GLOBALS['xoopsDB']->fetchRow($result); |
|
37
|
|
|
|
|
38
|
|
|
return ($count != 0); |
|
39
|
|
|
} |
|
40
|
|
|
|
|
41
|
|
|
/** |
|
42
|
|
|
* @return bool |
|
43
|
|
|
*/ |
|
44
|
|
View Code Duplication |
public function check_templates() |
|
45
|
|
|
{ |
|
46
|
|
|
$sql = 'SELECT COUNT(*) FROM `' . $GLOBALS['xoopsDB']->prefix('tplfile') . "` WHERE `tpl_file` IN ('system_header.html', 'system_header.tpl') AND `tpl_type` = 'admin'"; |
|
47
|
|
|
if (!$result = $GLOBALS['xoopsDB']->queryF($sql)) { |
|
48
|
|
|
return false; |
|
49
|
|
|
} |
|
50
|
|
|
list($count) = $GLOBALS['xoopsDB']->fetchRow($result); |
|
51
|
|
|
|
|
52
|
|
|
return ($count != 0); |
|
53
|
|
|
} |
|
54
|
|
|
|
|
55
|
|
|
/** |
|
56
|
|
|
* @return bool |
|
57
|
|
|
*/ |
|
58
|
|
|
public function apply_config() |
|
59
|
|
|
{ |
|
60
|
|
|
$dbm = new Db_manager(); |
|
61
|
|
|
|
|
62
|
|
|
$sql = 'SELECT conf_id FROM `' . $GLOBALS['xoopsDB']->prefix('config') . "` WHERE `conf_name` IN ('cpanel')"; |
|
63
|
|
|
if (!$result = $GLOBALS['xoopsDB']->queryF($sql)) { |
|
64
|
|
|
return false; |
|
65
|
|
|
} |
|
66
|
|
|
$count = $GLOBALS['xoopsDB']->fetchRow($result); |
|
67
|
|
|
|
|
68
|
|
|
$sql = 'UPDATE `' . $GLOBALS['xoopsDB']->prefix('config') . "` SET `conf_value` = 'default' WHERE `conf_id` = " . $count[0]; |
|
69
|
|
|
if (!$result = $GLOBALS['xoopsDB']->queryF($sql)) { |
|
70
|
|
|
return false; |
|
71
|
|
|
} |
|
72
|
|
|
|
|
73
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'break1', '_MI_SYSTEM_PREFERENCE_BREAK_GENERAL', 'head', '', 'line_break', 'textbox', 0)"); |
|
74
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'usetips', '_MI_SYSTEM_PREFERENCE_TIPS', '1', '_MI_SYSTEM_PREFERENCE_TIPS_DSC', 'yesno', 'int', 10)"); |
|
75
|
|
|
$icon_id = $dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'typeicons', '_MI_SYSTEM_PREFERENCE_ICONS', 'default', '', 'select', 'text', 20)"); |
|
76
|
|
|
$bc_id = $dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'typebreadcrumb', '_MI_SYSTEM_PREFERENCE_BREADCRUMB', 'default', '', 'select', 'text', 30)"); |
|
77
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'break2', '_MI_SYSTEM_PREFERENCE_BREAK_ACTIVE', 'head', '', 'line_break', 'textbox', 40)"); |
|
78
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'active_avatars', '_MI_SYSTEM_PREFERENCE_ACTIVE_AVATARS', '1', '', 'yesno', 'int', 50)"); |
|
79
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'active_banners', '_MI_SYSTEM_PREFERENCE_ACTIVE_BANNERS', '1', '', 'yesno', 'int', 60)"); |
|
80
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'active_blocksadmin', '_MI_SYSTEM_PREFERENCE_ACTIVE_BLOCKSADMIN', '1', '', 'hidden', 'int', 70)"); |
|
81
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'active_comments', '_MI_SYSTEM_PREFERENCE_ACTIVE_COMMENTS', '1', '', 'yesno', 'int', 80)"); |
|
82
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'active_filemanager', '_MI_SYSTEM_PREFERENCE_ACTIVE_FILEMANAGER', '1', '', 'yesno', 'int', 90)"); |
|
83
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'active_groups', '_MI_SYSTEM_PREFERENCE_ACTIVE_GROUPS', '1', '', 'hidden', 'int', 100)"); |
|
84
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'active_images', '_MI_SYSTEM_PREFERENCE_ACTIVE_IMAGES', '1', '', 'yesno', 'int', 110)"); |
|
85
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'active_mailusers', '_MI_SYSTEM_PREFERENCE_ACTIVE_MAILUSERS', '1', '', 'yesno', 'int', 120)"); |
|
86
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'active_modulesadmin', '_MI_SYSTEM_PREFERENCE_ACTIVE_MODULESADMIN', '1', '', 'hidden', 'int', 130)"); |
|
87
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'active_maintenance', '_MI_SYSTEM_PREFERENCE_ACTIVE_MAINTENANCE', '1', '', 'yesno', 'int', 140)"); |
|
88
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'active_preferences', '_MI_SYSTEM_PREFERENCE_ACTIVE_PREFERENCES', '1', '', 'hidden', 'int', 150)"); |
|
89
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'active_smilies', '_MI_SYSTEM_PREFERENCE_ACTIVE_SMILIES', '1', '', 'yesno', 'int', 160)"); |
|
90
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'active_tplsets', '_MI_SYSTEM_PREFERENCE_ACTIVE_TPLSETS', '1', '', 'hidden', 'int', 170)"); |
|
91
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'active_userrank', '_MI_SYSTEM_PREFERENCE_ACTIVE_USERRANK', '1', '', 'yesno', 'int', 180)"); |
|
92
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'active_users', '_MI_SYSTEM_PREFERENCE_ACTIVE_USERS', '1', '', 'yesno', 'int', 190)"); |
|
93
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'break3', '_MI_SYSTEM_PREFERENCE_BREAK_PAGER', 'head', '', 'line_break', 'textbox', 200)"); |
|
94
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'avatars_pager', '_MI_SYSTEM_PREFERENCE_AVATARS_PAGER', '10', '', 'textbox', 'int', 210)"); |
|
95
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'banners_pager', '_MI_SYSTEM_PREFERENCE_BANNERS_PAGER', '10', '', 'textbox', 'int', 220)"); |
|
96
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'comments_pager', '_MI_SYSTEM_PREFERENCE_COMMENTS_PAGER', '20', '', 'textbox', 'int', 230)"); |
|
97
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'groups_pager', '_MI_SYSTEM_PREFERENCE_GROUPS_PAGER', '15', '', 'textbox', 'int', 240)"); |
|
98
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'images_pager', '_MI_SYSTEM_PREFERENCE_IMAGES_PAGER', '15', '', 'textbox', 'int', 250)"); |
|
99
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'smilies_pager', '_MI_SYSTEM_PREFERENCE_SMILIES_PAGER', '20', '', 'textbox', 'int', 260)"); |
|
100
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'userranks_pager', '_MI_SYSTEM_PREFERENCE_USERRANKS_PAGER', '20', '', 'textbox', 'int', 270)"); |
|
101
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'users_pager', '_MI_SYSTEM_PREFERENCE_USERS_PAGER', '20', '', 'textbox', 'int', 280)"); |
|
102
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'break4', '_MI_SYSTEM_PREFERENCE_BREAK_EDITOR', 'head', '', 'line_break', 'textbox', 290)"); |
|
103
|
|
|
$block_id = $dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'blocks_editor', '_MI_SYSTEM_PREFERENCE_BLOCKS_EDITOR', 'dhtmltextarea', '_MI_SYSTEM_PREFERENCE_BLOCKS_EDITOR_DSC', 'select', 'text', 300)"); |
|
104
|
|
|
$com_id = $dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'comments_editor', '_MI_SYSTEM_PREFERENCE_COMMENTS_EDITOR', 'dhtmltextarea', '_MI_SYSTEM_PREFERENCE_COMMENTS_EDITOR_DSC', 'select', 'text', 310)"); |
|
105
|
|
|
$main_id = $dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'general_editor', '_MI_SYSTEM_PREFERENCE_GENERAL_EDITOR', 'dhtmltextarea', '_MI_SYSTEM_PREFERENCE_GENERAL_EDITOR_DSC', 'select', 'text', 320)"); |
|
106
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'redirect', '_MI_SYSTEM_PREFERENCE_REDIRECT', 'admin.php?fct=preferences', '', 'hidden', 'text', 330)"); |
|
107
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'com_anonpost', '_MI_SYSTEM_PREFERENCE_ANONPOST', '', '', 'hidden', 'text', 340)"); |
|
108
|
|
|
$jquery_id = $dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (1, 0, 'jquery_theme', '_MI_SYSTEM_PREFERENCE_JQUERY_THEME', 'base', '', 'select', 'text', 35)"); |
|
109
|
|
|
|
|
110
|
|
|
$dbm->insert('config', " (conf_modid,conf_catid,conf_name,conf_title,conf_value,conf_desc,conf_formtype,conf_valuetype,conf_order) VALUES (0, 1, 'redirect_message_ajax', '_MD_AM_CUSTOM_REDIRECT', '1', '_MD_AM_CUSTOM_REDIRECT_DESC', 'yesno', 'int', 12)"); |
|
111
|
|
|
|
|
112
|
|
|
require_once '../class/xoopslists.php'; |
|
113
|
|
|
$editors = XoopsLists::getDirListAsArray('../class/xoopseditor'); |
|
114
|
|
|
foreach ($editors as $dir) { |
|
115
|
|
|
$dbm->insert('configoption', " (confop_name,confop_value,conf_id) VALUES ('" . $dir . "', '" . $dir . "', $block_id)"); |
|
116
|
|
|
} |
|
117
|
|
|
foreach ($editors as $dir) { |
|
118
|
|
|
$dbm->insert('configoption', " (confop_name,confop_value,conf_id) VALUES ('" . $dir . "', '" . $dir . "', $com_id)"); |
|
119
|
|
|
} |
|
120
|
|
|
foreach ($editors as $dir) { |
|
121
|
|
|
$dbm->insert('configoption', " (confop_name,confop_value,conf_id) VALUES ('" . $dir . "', '" . $dir . "', $main_id)"); |
|
122
|
|
|
} |
|
123
|
|
|
$icons = XoopsLists::getDirListAsArray('../modules/system/images/icons'); |
|
124
|
|
View Code Duplication |
foreach ($icons as $dir) { |
|
125
|
|
|
$dbm->insert('configoption', " (confop_name,confop_value,conf_id) VALUES ('" . $dir . "', '" . $dir . "', $icon_id)"); |
|
126
|
|
|
} |
|
127
|
|
|
$breadcrumb = XoopsLists::getDirListAsArray('../modules/system/images/breadcrumb'); |
|
128
|
|
View Code Duplication |
foreach ($breadcrumb as $dir) { |
|
129
|
|
|
$dbm->insert('configoption', " (confop_name,confop_value,conf_id) VALUES ('" . $dir . "', '" . $dir . "', $bc_id)"); |
|
130
|
|
|
} |
|
131
|
|
|
$jqueryui = XoopsLists::getDirListAsArray('../modules/system/css/ui'); |
|
132
|
|
View Code Duplication |
foreach ($jqueryui as $dir) { |
|
133
|
|
|
$dbm->insert('configoption', " (confop_name,confop_value,conf_id) VALUES ('" . $dir . "', '" . $dir . "', $jquery_id)"); |
|
134
|
|
|
} |
|
135
|
|
|
|
|
136
|
|
|
return true; |
|
137
|
|
|
} |
|
138
|
|
|
|
|
139
|
|
|
/** |
|
140
|
|
|
* @return bool |
|
141
|
|
|
*/ |
|
142
|
|
|
public function apply_templates() |
|
143
|
|
|
{ |
|
144
|
|
|
include_once '../modules/system/xoops_version.php'; |
|
145
|
|
|
|
|
146
|
|
|
$dbm = new Db_manager(); |
|
147
|
|
|
$time = time(); |
|
148
|
|
|
foreach ($modversion['templates'] as $tplfile) { |
|
|
|
|
|
|
149
|
|
|
// Admin templates |
|
150
|
|
|
if (isset($tplfile['type']) && $tplfile['type'] === 'admin' && $fp = fopen('../modules/system/templates/admin/' . $tplfile['file'], 'r')) { |
|
151
|
|
|
$newtplid = $dbm->insert('tplfile', " VALUES (0, 1, 'system', 'default', '" . addslashes($tplfile['file']) . "', '" . addslashes($tplfile['description']) . "', " . $time . ', ' . $time . ", 'admin')"); |
|
152
|
|
|
$tplsource = fread($fp, filesize('../modules/system/templates/admin/' . $tplfile['file'])); |
|
153
|
|
|
fclose($fp); |
|
154
|
|
|
$dbm->insert('tplsource', ' (tpl_id, tpl_source) VALUES (' . $newtplid . ", '" . addslashes($tplsource) . "')"); |
|
155
|
|
|
} |
|
156
|
|
|
} |
|
157
|
|
|
|
|
158
|
|
|
return true; |
|
159
|
|
|
} |
|
160
|
|
|
|
|
161
|
|
|
/** |
|
162
|
|
|
* Identify a block mangled in the change from XOOPS 2.4.x to 2.5.0 |
|
163
|
|
|
* |
|
164
|
|
|
* The user menu block was element 1 in the old $modversion['blocks'] array, but |
|
165
|
|
|
* became element 0 in 2.5.0. This results in the old block not being updated with |
|
166
|
|
|
* new settings. We see it in 2.5.8+ where the theme overrides fail because of the |
|
167
|
|
|
* extension change from .html to .tpl. |
|
168
|
|
|
* |
|
169
|
|
|
* Installs are not a problem, just upgrades. This is the only block affected. |
|
170
|
|
|
* |
|
171
|
|
|
* @return CriteriaElement |
|
172
|
|
|
*/ |
|
173
|
|
|
private function strayblockCriteria() |
|
174
|
|
|
{ |
|
175
|
|
|
$criteria = new CriteriaCompo(new Criteria('mid','1', '=')); |
|
176
|
|
|
$criteria->add(new Criteria('block_type','S', '=')); |
|
177
|
|
|
$criteria->add(new Criteria('func_num','1', '=')); |
|
178
|
|
|
$criteria->add(new Criteria('template','system_block_user.html', '=')); |
|
179
|
|
|
return $criteria; |
|
180
|
|
|
} |
|
181
|
|
|
|
|
182
|
|
|
/** |
|
183
|
|
|
* @return bool |
|
184
|
|
|
*/ |
|
185
|
|
|
public function check_strayblock() |
|
186
|
|
|
{ |
|
187
|
|
|
$criteria = $this->strayblockCriteria(); |
|
188
|
|
|
$count = Xmf\Database\TableLoad::countRows('newblocks', $criteria); |
|
189
|
|
|
|
|
190
|
|
|
return ($count === 0); |
|
191
|
|
|
} |
|
192
|
|
|
|
|
193
|
|
|
/** |
|
194
|
|
|
* @return bool |
|
195
|
|
|
*/ |
|
196
|
|
|
public function apply_strayblock() |
|
197
|
|
|
{ |
|
198
|
|
|
$criteria = $this->strayblockCriteria(); |
|
199
|
|
|
$tables = new Xmf\Database\Tables(); |
|
200
|
|
|
$tables->useTable('newblocks'); |
|
201
|
|
|
$tables->update('newblocks', array('func_num' => '0'), $criteria); |
|
202
|
|
|
|
|
203
|
|
|
return $tables->executeQueue(true); |
|
204
|
|
|
} |
|
205
|
|
|
|
|
206
|
|
|
public function __construct() |
|
207
|
|
|
{ |
|
208
|
|
|
parent::__construct(basename(__DIR__)); |
|
209
|
|
|
$this->tasks = array('config', 'templates', 'strayblock'); |
|
210
|
|
|
} |
|
211
|
|
|
} |
|
212
|
|
|
|
|
213
|
|
|
$upg = new Upgrade_250(); |
|
214
|
|
|
return $upg; |
|
215
|
|
|
|
The PSR-1: Basic Coding Standard recommends that a file should either introduce new symbols, that is classes, functions, constants or similar, or have side effects. Side effects are anything that executes logic, like for example printing output, changing ini settings or writing to a file.
The idea behind this recommendation is that merely auto-loading a class should not change the state of an application. It also promotes a cleaner style of programming and makes your code less prone to errors, because the logic is not spread out all over the place.
To learn more about the PSR-1, please see the PHP-FIG site on the PSR-1.