txmodxoops /
tdmcreate
| Conditions | 39 |
| Paths | 0 |
| Total Lines | 224 |
| Code Lines | 172 |
| 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 |
||
| 132 | public function getFormModules($action = false) |
||
| 133 | { |
||
| 134 | $tdmcreate = TDMCreateHelper::getInstance(); |
||
| 135 | if (false === $action) { |
||
| 136 | $action = $_SERVER['REQUEST_URI']; |
||
| 137 | } |
||
| 138 | $set = []; |
||
| 139 | $settings = $tdmcreate->getHandler('settings')->getAllSettings(0, 0, 'set_type'); |
||
| 140 | foreach ($settings as $setting) { |
||
| 141 | $set['name'] = $setting->getVar('set_name'); |
||
| 142 | $set['dirname'] = $setting->getVar('set_dirname'); |
||
| 143 | $set['version'] = $setting->getVar('set_version'); |
||
| 144 | $set['since'] = $setting->getVar('set_since'); |
||
| 145 | $set['min_php'] = $setting->getVar('set_min_php'); |
||
| 146 | $set['min_xoops'] = $setting->getVar('set_min_xoops'); |
||
| 147 | $set['min_admin'] = $setting->getVar('set_min_admin'); |
||
| 148 | $set['min_mysql'] = $setting->getVar('set_min_mysql'); |
||
| 149 | $set['description'] = $setting->getVar('set_description'); |
||
| 150 | $set['author'] = $setting->getVar('set_author'); |
||
| 151 | $set['license'] = $setting->getVar('set_license'); |
||
| 152 | $set['admin'] = $setting->getVar('set_admin'); |
||
| 153 | $set['user'] = $setting->getVar('set_user'); |
||
| 154 | $set['blocks'] = $setting->getVar('set_blocks'); |
||
| 155 | $set['search'] = $setting->getVar('set_search'); |
||
| 156 | $set['comments'] = $setting->getVar('set_comments'); |
||
| 157 | $set['notifications'] = $setting->getVar('set_notifications'); |
||
| 158 | $set['permissions'] = $setting->getVar('set_permissions'); |
||
| 159 | $set['inroot'] = $setting->getVar('set_inroot_copy'); |
||
| 160 | $set['image'] = $setting->getVar('set_image'); |
||
| 161 | $set['author_mail'] = $setting->getVar('set_author_mail'); |
||
| 162 | $set['author_website_url'] = $setting->getVar('set_author_website_url'); |
||
| 163 | $set['author_website_name'] = $setting->getVar('set_author_website_name'); |
||
| 164 | $set['credits'] = $setting->getVar('set_credits'); |
||
| 165 | $set['release_info'] = $setting->getVar('set_release_info'); |
||
| 166 | $set['release_file'] = $setting->getVar('set_release_file'); |
||
| 167 | $set['manual'] = $setting->getVar('set_manual'); |
||
| 168 | $set['manual_file'] = $setting->getVar('set_manual_file'); |
||
| 169 | $set['demo_site_url'] = $setting->getVar('set_demo_site_url'); |
||
| 170 | $set['demo_site_name'] = $setting->getVar('set_demo_site_name'); |
||
| 171 | $set['support_url'] = $setting->getVar('set_support_url'); |
||
| 172 | $set['support_name'] = $setting->getVar('set_support_name'); |
||
| 173 | $set['website_url'] = $setting->getVar('set_website_url'); |
||
| 174 | $set['website_name'] = $setting->getVar('set_website_name'); |
||
| 175 | $set['release'] = $setting->getVar('set_release'); |
||
| 176 | $set['status'] = $setting->getVar('set_status'); |
||
| 177 | $set['donations'] = $setting->getVar('set_donations'); |
||
| 178 | $set['subversion'] = $setting->getVar('set_subversion'); |
||
| 179 | } |
||
| 180 | |||
| 181 | $isNew = $this->isNew(); |
||
| 182 | $title = $isNew ? sprintf(_AM_TDMCREATE_MODULE_NEW) : sprintf(_AM_TDMCREATE_MODULE_EDIT); |
||
| 183 | |||
| 184 | include_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php'; |
||
| 185 | |||
| 186 | $form = new XoopsThemeForm($title, 'moduleform', $action, 'post', true); |
||
| 187 | $form->setExtra('enctype="multipart/form-data"'); |
||
| 188 | |||
| 189 | $modName = $isNew ? $set['name'] : $this->getVar('mod_name'); |
||
| 190 | $modName = new XoopsFormText(_AM_TDMCREATE_MODULE_NAME, 'mod_name', 50, 255, $modName); |
||
| 191 | $modName->setDescription(_AM_TDMCREATE_MODULE_NAME_DESC); |
||
| 192 | $form->addElement($modName, true); |
||
| 193 | |||
| 194 | $modDirname = $isNew ? $set['dirname'] : $this->getVar('mod_dirname'); |
||
| 195 | $modDirname = new XoopsFormText(_AM_TDMCREATE_MODULE_DIRNAME, 'mod_dirname', 25, 255, $modDirname); |
||
| 196 | $modDirname->setDescription(_AM_TDMCREATE_MODULE_DIRNAME_DESC); |
||
| 197 | $form->addElement($modDirname, true); |
||
| 198 | |||
| 199 | $modVersion = $isNew ? $set['version'] : $this->getVar('mod_version'); |
||
| 200 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_VERSION, 'mod_version', 10, 25, $modVersion), true); |
||
| 201 | |||
| 202 | $modSince = $isNew ? $set['since'] : $this->getVar('mod_since'); |
||
| 203 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_SINCE, 'mod_since', 10, 25, $modSince), true); |
||
| 204 | |||
| 205 | $modMinPhp = $isNew ? $set['min_php'] : $this->getVar('mod_min_php'); |
||
| 206 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_MIN_PHP, 'mod_min_php', 10, 25, $modMinPhp), true); |
||
| 207 | |||
| 208 | $modMinXoops = $isNew ? $set['min_xoops'] : $this->getVar('mod_min_xoops'); |
||
| 209 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_MIN_XOOPS, 'mod_min_xoops', 10, 25, $modMinXoops), true); |
||
| 210 | |||
| 211 | $modMinAdmin = $isNew ? $set['min_admin'] : $this->getVar('mod_min_admin'); |
||
| 212 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_MIN_ADMIN, 'mod_min_admin', 10, 25, $modMinAdmin), true); |
||
| 213 | |||
| 214 | $modMinMysql = $isNew ? $set['min_mysql'] : $this->getVar('mod_min_mysql'); |
||
| 215 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_MIN_MYSQL, 'mod_min_mysql', 10, 25, $modMinMysql), true); |
||
| 216 | // Name description |
||
| 217 | $editorConfigs = []; |
||
| 218 | $editorConfigs['name'] = 'mod_description'; |
||
| 219 | $editorConfigs['value'] = $isNew ? $set['description'] : $this->getVar('mod_description', 'e'); |
||
| 220 | $editorConfigs['rows'] = 5; |
||
| 221 | $editorConfigs['cols'] = 100; |
||
| 222 | $editorConfigs['width'] = '50%'; |
||
| 223 | $editorConfigs['height'] = '100px'; |
||
| 224 | $editorConfigs['editor'] = $tdmcreate->getConfig('tdmcreate_editor'); |
||
| 225 | $form->addElement(new XoopsFormEditor(_AM_TDMCREATE_MODULE_DESCRIPTION, 'mod_description', $editorConfigs), true); |
||
| 226 | // Author |
||
| 227 | $modAuthor = $isNew ? $set['author'] : $this->getVar('mod_author'); |
||
| 228 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_AUTHOR, 'mod_author', 50, 255, $modAuthor), true); |
||
| 229 | $modLicense = $isNew ? $set['license'] : $this->getVar('mod_license'); |
||
| 230 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_LICENSE, 'mod_license', 50, 255, $modLicense), true); |
||
| 231 | |||
| 232 | $optionsTray = new XoopsFormElementTray(_OPTIONS, '<br>'); |
||
| 233 | $optionsTray->setDescription(_AM_TDMCREATE_OPTIONS_DESC); |
||
| 234 | // Check All Modules Options |
||
| 235 | $checkAllOptions = new XoopsFormCheckBox('', 'modulebox', 1); |
||
| 236 | $checkAllOptions->addOption('allbox', _AM_TDMCREATE_MODULE_ALL); |
||
| 237 | $checkAllOptions->setExtra(' onclick="xoopsCheckAll(\'moduleform\', \'modulebox\');" '); |
||
| 238 | $checkAllOptions->setClass('xo-checkall'); |
||
| 239 | $optionsTray->addElement($checkAllOptions); |
||
| 240 | // Options |
||
| 241 | $checkbox = new XoopsFormCheckbox(' ', 'module_option', $this->getOptionsModules(), '<br>'); |
||
| 242 | foreach ($this->options as $option) { |
||
| 243 | $checkbox->addOption($option, self::getDefinedLanguage('_AM_TDMCREATE_MODULE_'.strtoupper($option))); |
||
| 244 | } |
||
| 245 | $optionsTray->addElement($checkbox); |
||
| 246 | |||
| 247 | $form->addElement($optionsTray); |
||
| 248 | |||
| 249 | $modImage = $this->getVar('mod_image'); |
||
| 250 | $modImage = $modImage ?: $set['image']; |
||
| 251 | |||
| 252 | $uploadDirectory = 'uploads/'.$GLOBALS['xoopsModule']->dirname().'/images/modules'; |
||
| 253 | $imgtray = new XoopsFormElementTray(_AM_TDMCREATE_MODULE_IMAGE, '<br>'); |
||
| 254 | $imgpath = sprintf(_AM_TDMCREATE_FORMIMAGE_PATH, './'.strtolower($uploadDirectory).'/'); |
||
| 255 | $imageselect = new XoopsFormSelect($imgpath, 'mod_image', $modImage); |
||
| 256 | $modImageArray = XoopsLists::getImgListAsArray(TDMC_UPLOAD_IMGMOD_PATH); |
||
| 257 | foreach ($modImageArray as $image) { |
||
| 258 | $imageselect->addOption("{$image}", $image); |
||
| 259 | } |
||
| 260 | $imageselect->setExtra("onchange='showImgSelected(\"image3\", \"mod_image\", \"".$uploadDirectory.'", "", "'.XOOPS_URL."\")'"); |
||
| 261 | $imgtray->addElement($imageselect); |
||
| 262 | $imgtray->addElement(new XoopsFormLabel('', "<br><img src='".TDMC_UPLOAD_IMGMOD_URL.'/'.$modImage."' name='image3' id='image3' alt='' /><br>")); |
||
| 263 | |||
| 264 | $fileseltray = new XoopsFormElementTray('', '<br>'); |
||
| 265 | $fileseltray->addElement(new XoopsFormFile(_AM_TDMCREATE_FORMUPLOAD, 'attachedfile', $tdmcreate->getConfig('maxsize'))); |
||
| 266 | $fileseltray->addElement(new XoopsFormLabel('')); |
||
| 267 | $imgtray->addElement($fileseltray); |
||
| 268 | $form->addElement($imgtray); |
||
| 269 | //---------- START LOGO GENERATOR ----------------- |
||
| 270 | $tables_img = $this->getVar('table_image') ?: 'about.png'; |
||
| 271 | $iconsdir = '/Frameworks/moduleclasses/icons/32'; |
||
| 272 | if (is_dir(XOOPS_ROOT_PATH.$iconsdir)) { |
||
| 273 | $uploadDirectory = $iconsdir; |
||
| 274 | $imgpath = sprintf(_AM_TDMCREATE_FORMIMAGE_PATH, ".{$iconsdir}/"); |
||
| 275 | } else { |
||
| 276 | $uploadDirectory = '/uploads/'.$GLOBALS['xoopsModule']->dirname().'/images/tables'; |
||
| 277 | $imgpath = sprintf(_AM_TDMCREATE_FORMIMAGE_PATH, './uploads/'.$GLOBALS['xoopsModule']->dirname().'/images/tables'); |
||
| 278 | } |
||
| 279 | $createLogoTray = new XoopsFormElementTray(_AM_TDMCREATE_MODULE_CREATENEWLOGO, '<br>'); |
||
| 280 | $iconSelect = new XoopsFormSelect($imgpath, 'tables_img', $tables_img, 8); |
||
| 281 | $tablesImagesArray = XoopsLists::getImgListAsArray(XOOPS_ROOT_PATH.$uploadDirectory); |
||
| 282 | foreach ($tablesImagesArray as $image) { |
||
| 283 | $iconSelect->addOption("{$image}", $image); |
||
| 284 | } |
||
| 285 | $iconSelect->setExtra(" onchange='showImgSelected2(\"image4\", \"tables_img\", \"".$uploadDirectory.'", "", "'.XOOPS_URL."\")' "); |
||
| 286 | $createLogoTray->addElement($iconSelect); |
||
| 287 | $createLogoTray->addElement(new XoopsFormLabel('', "<br><img src='".XOOPS_URL.'/'.$uploadDirectory.'/'.$tables_img."' name='image4' id='image4' alt='' />")); |
||
| 288 | // Create preview and submit buttons |
||
| 289 | $buttonLogoGenerator4 = new XoopsFormButton('', 'button4', _AM_TDMCREATE_MODULE_CREATENEWLOGO, 'button'); |
||
| 290 | $buttonLogoGenerator4->setExtra(" onclick='createNewModuleLogo(\"".TDMC_URL."\")' "); |
||
| 291 | $createLogoTray->addElement($buttonLogoGenerator4); |
||
| 292 | |||
| 293 | $form->addElement($createLogoTray); |
||
| 294 | //------------ END LOGO GENERATOR -------------------- |
||
| 295 | |||
| 296 | $modAuthorMail = $isNew ? $set['author_mail'] : $this->getVar('mod_author_mail'); |
||
| 297 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_AUTHOR_MAIL, 'mod_author_mail', 50, 255, $modAuthorMail)); |
||
| 298 | |||
| 299 | $modAuthorWebsiteUrl = $isNew ? $set['author_website_url'] : $this->getVar('mod_author_website_url'); |
||
| 300 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_AUTHOR_WEBSITE_URL, 'mod_author_website_url', 50, 255, $modAuthorWebsiteUrl)); |
||
| 301 | |||
| 302 | $modAuthorWebsiteName = $isNew ? $set['author_website_name'] : $this->getVar('mod_author_website_name'); |
||
| 303 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_AUTHOR_WEBSITE_NAME, 'mod_author_website_name', 50, 255, $modAuthorWebsiteName)); |
||
| 304 | |||
| 305 | $modCredits = $isNew ? $set['credits'] : $this->getVar('mod_credits'); |
||
| 306 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_CREDITS, 'mod_credits', 50, 255, $modCredits)); |
||
| 307 | |||
| 308 | $modReleaseInfo = $isNew ? $set['release_info'] : $this->getVar('mod_release_info'); |
||
| 309 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_RELEASE_INFO, 'mod_release_info', 50, 255, $modReleaseInfo)); |
||
| 310 | |||
| 311 | $modReleaseFile = $isNew ? $set['release_file'] : $this->getVar('mod_release_file'); |
||
| 312 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_RELEASE_FILE, 'mod_release_file', 50, 255, $modReleaseFile)); |
||
| 313 | |||
| 314 | $modManual = $isNew ? $set['manual'] : $this->getVar('mod_manual'); |
||
| 315 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_MANUAL, 'mod_manual', 50, 255, $modManual)); |
||
| 316 | |||
| 317 | $modManualFile = $isNew ? $set['manual_file'] : $this->getVar('mod_manual_file'); |
||
| 318 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_MANUAL_FILE, 'mod_manual_file', 50, 255, $modManualFile)); |
||
| 319 | |||
| 320 | $modDemoSiteUrl = $isNew ? $set['demo_site_url'] : $this->getVar('mod_demo_site_url'); |
||
| 321 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_DEMO_SITE_URL, 'mod_demo_site_url', 50, 255, $modDemoSiteUrl)); |
||
| 322 | |||
| 323 | $modDemoSiteName = $isNew ? $set['demo_site_name'] : $this->getVar('mod_demo_site_name'); |
||
| 324 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_DEMO_SITE_NAME, 'mod_demo_site_name', 50, 255, $modDemoSiteName)); |
||
| 325 | |||
| 326 | $modSupportUrl = $isNew ? $set['support_url'] : $this->getVar('mod_support_url'); |
||
| 327 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_SUPPORT_URL, 'mod_support_url', 50, 255, $modSupportUrl)); |
||
| 328 | |||
| 329 | $modSupportName = $isNew ? $set['support_name'] : $this->getVar('mod_support_name'); |
||
| 330 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_SUPPORT_NAME, 'mod_support_name', 50, 255, $modSupportName)); |
||
| 331 | |||
| 332 | $modWebsiteUrl = $isNew ? $set['website_url'] : $this->getVar('mod_website_url'); |
||
| 333 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_WEBSITE_URL, 'mod_website_url', 50, 255, $modWebsiteUrl)); |
||
| 334 | |||
| 335 | $modWebsiteName = $isNew ? $set['website_name'] : $this->getVar('mod_website_name'); |
||
| 336 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_WEBSITE_NAME, 'mod_website_name', 50, 255, $modWebsiteName)); |
||
| 337 | |||
| 338 | $modRelease = $isNew ? $set['release'] : $this->getVar('mod_release'); |
||
| 339 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_RELEASE, 'mod_release', 50, 255, $modRelease)); |
||
| 340 | |||
| 341 | $modStatus = $isNew ? $set['status'] : $this->getVar('mod_status'); |
||
| 342 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_STATUS, 'mod_status', 50, 255, $modStatus)); |
||
| 343 | |||
| 344 | $modDonations = $isNew ? $set['donations'] : $this->getVar('mod_donations'); |
||
| 345 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_PAYPAL_BUTTON, 'mod_donations', 50, 255, $modDonations)); |
||
| 346 | |||
| 347 | $modSubversion = $isNew ? $set['subversion'] : $this->getVar('mod_subversion'); |
||
| 348 | $form->addElement(new XoopsFormText(_AM_TDMCREATE_MODULE_SUBVERSION, 'mod_subversion', 50, 255, $modSubversion)); |
||
| 349 | |||
| 350 | $buttonTray = new XoopsFormElementTray(_REQUIRED.' <sup class="red bold">*</sup>', ''); |
||
| 351 | $buttonTray->addElement(new XoopsFormHidden('op', 'save')); |
||
| 352 | $buttonTray->addElement(new XoopsFormButton('', 'submit', _SUBMIT, 'submit')); |
||
| 353 | $form->addElement($buttonTray); |
||
| 354 | |||
| 355 | return $form; |
||
| 356 | } |
||
| 571 |
Loading history...
The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g.
excluded_paths: ["lib/*"], you can move it to the dependency path list as follows:For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths