txmodxoops /
tdmcreate
| Conditions | 12 |
| Paths | 90 |
| Total Lines | 155 |
| Code Lines | 111 |
| Lines | 0 |
| Ratio | 0 % |
| Changes | 2 | ||
| 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 |
||
| 322 | public function getFormEdit($fieldMid = null, $fieldTid = null, $action = false) |
||
| 323 | { |
||
| 324 | $helper = Tdmcreate\Helper::getInstance(); |
||
| 325 | $tablesHandler = $helper->getHandler('Tables'); |
||
| 326 | $tables = $tablesHandler->get($fieldTid); |
||
| 327 | $prefix = $tables->getVar('table_fieldname'); |
||
| 328 | // Header function class |
||
| 329 | $fieldsForm = self::getInstance(); |
||
| 330 | $form = $fieldsForm->getHeaderForm($prefix, $action); |
||
| 331 | |||
| 332 | $class = 'even'; |
||
| 333 | $tableAutoincrement = $tables->getVar('table_autoincrement'); |
||
| 334 | $fieldNumb = $tables->getVar('table_nbfields'); |
||
| 335 | $fName = $tables->getVar('table_fieldname'); |
||
| 336 | |||
| 337 | // Get the list of fields |
||
| 338 | $cr = new \CriteriaCompo(); |
||
| 339 | $cr->add(new \Criteria('field_mid', $fieldMid)); |
||
| 340 | $cr->add(new \Criteria('field_tid', $fieldTid)); |
||
| 341 | $cr->setSort('field_order'); |
||
| 342 | $fields = $helper->getHandler('Fields')->getObjects($cr); |
||
| 343 | unset($cr); |
||
| 344 | $fieldElements = $helper->getHandler('Fieldelements')->getAll(); |
||
| 345 | foreach ($fieldElements as $fe) { |
||
| 346 | $form->addElement(new \XoopsFormHidden('fe_defaulttype[' . $fe->getVar('fieldelement_id') . ']', $fe->getVar('fieldelement_deftype'))); |
||
| 347 | $form->addElement(new \XoopsFormHidden('fe_defaultvalue[' . $fe->getVar('fieldelement_id') . ']', $fe->getVar('fieldelement_defvalue'))); |
||
| 348 | } |
||
| 349 | $id = 1; |
||
| 350 | foreach ($fields as $field) { |
||
| 351 | $class = ('even' === $class) ? 'odd' : 'even'; |
||
| 352 | $fieldId = (int)$field->getVar('field_id'); |
||
| 353 | if ($id > $fieldNumb) { // delete additional fields, if number of fields is reduced - goffy |
||
| 354 | $fieldsObj = $helper->getHandler('Fields')->get($fieldId); |
||
| 355 | $helper->getHandler('Fields')->delete($fieldsObj, true); |
||
| 356 | } else { |
||
| 357 | // show field with settings |
||
| 358 | $form->addElement(new \XoopsFormHidden('field_id[' . $id . ']', $fieldId)); |
||
| 359 | |||
| 360 | $form->addElement(new Tdmcreate\Html\FormLabel('<tr class="' . $class . '">')); |
||
| 361 | // Index ID |
||
| 362 | $form->addElement(new Tdmcreate\Html\FormLabel('<td class="center">' . $id . '</td>')); |
||
| 363 | // Field Name |
||
| 364 | $fieldName = new \XoopsFormText(_AM_TDMCREATE_FIELD_NAME, 'field_name[' . $id . ']', 15, 255, $field->getVar('field_name')); |
||
| 365 | $form->addElement(new Tdmcreate\Html\FormLabel('<td class="center">' . $fieldName->render() . '</td>')); |
||
| 366 | // Field Element |
||
| 367 | if ((1 == $id) && (1 == $tableAutoincrement)) { |
||
| 368 | $form->addElement(new Tdmcreate\Html\FormLabel('<td> </td>')); |
||
| 369 | } else { |
||
| 370 | // Field Elements |
||
| 371 | $crElement = new \CriteriaCompo(); |
||
| 372 | $crElement->add(new \Criteria('fieldelement_tid', 0)); |
||
| 373 | $crTable = new \CriteriaCompo(); |
||
| 374 | $crTable->add(new \Criteria('fieldelement_mid', $fieldMid)); |
||
| 375 | $fieldElementsSelect = new \XoopsFormSelect(_AM_TDMCREATE_FIELD_ELEMENT_NAME, 'field_element[' . $id . ']', $field->getVar('field_element')); |
||
| 376 | $fieldElementsSelect->addOptionArray($helper->getHandler('Fieldelements')->getFieldElementsList($crElement)); |
||
| 377 | $fieldElementsSelect->addOptionArray($helper->getHandler('Fieldelements')->getList($crTable)); |
||
| 378 | $fieldElementsSelect->setExtra(" onchange='presetField(". $id . ")' "); |
||
| 379 | unset($crElement, $crTable); |
||
| 380 | $form->addElement(new Tdmcreate\Html\FormLabel('<td class="center">' . $fieldElementsSelect->render() . '</td>')); |
||
| 381 | unset($fieldElementsSelect); |
||
| 382 | } |
||
| 383 | // Field Type |
||
| 384 | $fieldTypeSelect = new \XoopsFormSelect(_AM_TDMCREATE_FIELD_TYPE, 'field_type[' . $id . ']', $field->getVar('field_type')); |
||
| 385 | $fieldTypeSelect->addOptionArray($helper->getHandler('Fieldtype')->getList()); |
||
| 386 | $form->addElement(new Tdmcreate\Html\FormLabel('<td class="center">' . $fieldTypeSelect->render() . '</td>')); |
||
| 387 | // Field Value |
||
| 388 | $fieldValue = new \XoopsFormText(_AM_TDMCREATE_FIELD_VALUE, 'field_value[' . $id . ']', 10, 200, $field->getVar('field_value')); |
||
| 389 | $form->addElement(new Tdmcreate\Html\FormLabel('<td class="center">' . $fieldValue->render() . '</td>')); |
||
| 390 | // Field Attributes |
||
| 391 | $fieldAttributesSelect = new \XoopsFormSelect(_AM_TDMCREATE_FIELD_TYPE, 'field_attribute[' . $id . ']', $field->getVar('field_attribute')); |
||
| 392 | $fieldAttributesSelect->addOptionArray($helper->getHandler('Fieldattributes')->getList()); |
||
| 393 | $form->addElement(new Tdmcreate\Html\FormLabel('<td class="center">' . $fieldAttributesSelect->render() . '</td>')); |
||
| 394 | // Field Null |
||
| 395 | $fieldNullSelect = new \XoopsFormSelect(_AM_TDMCREATE_FIELD_NULL, 'field_null[' . $id . ']', $field->getVar('field_null')); |
||
| 396 | $fieldNullSelect->addOptionArray($helper->getHandler('Fieldnull')->getList()); |
||
| 397 | $form->addElement(new Tdmcreate\Html\FormLabel('<td class="center">' . $fieldNullSelect->render() . '</td>')); |
||
| 398 | // Field Default |
||
| 399 | $fieldDefault = new \XoopsFormText(_AM_TDMCREATE_FIELD_DEFAULT, 'field_default[' . $id . ']', 15, 25, $field->getVar('field_default')); |
||
| 400 | $form->addElement(new Tdmcreate\Html\FormLabel('<td class="center">' . $fieldDefault->render() . '</td>')); |
||
| 401 | // Field Key |
||
| 402 | $fieldKeySelect = new \XoopsFormSelect(_AM_TDMCREATE_FIELD_KEY, 'field_key[' . $id . ']', $field->getVar('field_key')); |
||
| 403 | $fieldKeySelect->addOptionArray($helper->getHandler('Fieldkey')->getList()); |
||
| 404 | $form->addElement(new Tdmcreate\Html\FormLabel('<td class="center">' . $fieldKeySelect->render() . '</td>')); |
||
| 405 | // Field Void |
||
| 406 | if ((1 == $id) && (1 == $tableAutoincrement)) { |
||
| 407 | $form->addElement(new Tdmcreate\Html\FormLabel('<td> </td></tr>')); |
||
| 408 | } else { |
||
| 409 | // Box header row |
||
| 410 | $parametersTray = new \XoopsFormElementTray('', '<br>'); |
||
| 411 | |||
| 412 | $checkFieldParent = new \XoopsFormCheckBox(' ', 'field_parent[' . $id . ']', $field->getVar('field_parent')); |
||
| 413 | $checkFieldParent->addOption(1, _AM_TDMCREATE_FIELD_PARENT); |
||
| 414 | $parametersTray->addElement($checkFieldParent); |
||
| 415 | |||
| 416 | $checkFieldAdmin = new \XoopsFormCheckBox(' ', 'field_admin[' . $id . ']', $field->getVar('field_admin')); |
||
| 417 | $checkFieldAdmin->addOption(1, _AM_TDMCREATE_FIELD_ADMIN); |
||
| 418 | $parametersTray->addElement($checkFieldAdmin); |
||
| 419 | |||
| 420 | $checkFieldInList = new \XoopsFormCheckBox(' ', 'field_inlist[' . $id . ']', $field->getVar('field_inlist')); |
||
| 421 | $checkFieldInList->addOption(1, _AM_TDMCREATE_FIELD_INLIST); |
||
| 422 | $parametersTray->addElement($checkFieldInList); |
||
| 423 | |||
| 424 | $checkFieldInForm = new \XoopsFormCheckBox(' ', 'field_inform[' . $id . ']', $field->getVar('field_inform')); |
||
| 425 | $checkFieldInForm->addOption(1, _AM_TDMCREATE_FIELD_INFORM); |
||
| 426 | $parametersTray->addElement($checkFieldInForm); |
||
| 427 | |||
| 428 | $checkFieldUser = new \XoopsFormCheckBox(' ', 'field_user[' . $id . ']', $field->getVar('field_user')); |
||
| 429 | $checkFieldUser->addOption(1, _AM_TDMCREATE_FIELD_USER); |
||
| 430 | $parametersTray->addElement($checkFieldUser); |
||
| 431 | |||
| 432 | $checkFieldThead = new \XoopsFormCheckBox(' ', 'field_thead[' . $id . ']', $field->getVar('field_thead')); |
||
| 433 | $checkFieldThead->addOption(1, _AM_TDMCREATE_FIELD_THEAD); |
||
| 434 | $parametersTray->addElement($checkFieldThead); |
||
| 435 | |||
| 436 | $checkFieldTbody = new \XoopsFormCheckBox(' ', 'field_tbody[' . $id . ']', $field->getVar('field_tbody')); |
||
| 437 | $checkFieldTbody->addOption(1, _AM_TDMCREATE_FIELD_TBODY); |
||
| 438 | $parametersTray->addElement($checkFieldTbody); |
||
| 439 | |||
| 440 | $checkFieldTfoot = new \XoopsFormCheckBox(' ', 'field_tfoot[' . $id . ']', $field->getVar('field_tfoot')); |
||
| 441 | $checkFieldTfoot->addOption(1, _AM_TDMCREATE_FIELD_TFOOT); |
||
| 442 | $parametersTray->addElement($checkFieldTfoot); |
||
| 443 | |||
| 444 | $checkFieldBlock = new \XoopsFormCheckBox('', 'field_block[' . $id . ']', $field->getVar('field_block')); |
||
| 445 | $checkFieldBlock->addOption(1, _AM_TDMCREATE_FIELD_BLOCK); |
||
| 446 | $parametersTray->addElement($checkFieldBlock); |
||
| 447 | |||
| 448 | $check_field_search = new \XoopsFormCheckBox(' ', 'field_search[' . $id . ']', $field->getVar('field_search')); |
||
| 449 | $check_field_search->addOption(1, _AM_TDMCREATE_FIELD_SEARCH); |
||
| 450 | $parametersTray->addElement($check_field_search); |
||
| 451 | |||
| 452 | $checkFieldRequired = new \XoopsFormCheckBox(' ', 'field_required[' . $id . ']', $field->getVar('field_required')); |
||
| 453 | $checkFieldRequired->addOption(1, _AM_TDMCREATE_FIELD_REQUIRED); |
||
| 454 | $parametersTray->addElement($checkFieldRequired); |
||
| 455 | |||
| 456 | $fieldMain = (1 == $field->getVar('field_main')) ? $id : 1; |
||
| 457 | $checkFieldMain = new Tdmcreate\Form\FormRadio('', 'field_main', $fieldMain); |
||
| 458 | $checkFieldMain->addOption($id, _AM_TDMCREATE_FIELD_MAIN); |
||
| 459 | $parametersTray->addElement($checkFieldMain); |
||
| 460 | |||
| 461 | $form->addElement(new Tdmcreate\Html\FormLabel('<td><div class="portlet"><div class="portlet-header">' . _AM_TDMCREATE_FIELD_PARAMETERS_LIST . '</div><div class="portlet-content">' . $parametersTray->render() . '</div></div></td></tr>')); |
||
| 462 | } |
||
| 463 | } |
||
| 464 | ++$id; |
||
| 465 | } |
||
| 466 | // If you change number fields in tables, |
||
| 467 | // adding missing fields or delete unnecessary fields |
||
| 468 | // By goffy |
||
| 469 | for ($i = $id; $i <= $fieldNumb; ++$i) { |
||
| 470 | $class = ('even' === $class) ? 'odd' : 'even'; |
||
| 471 | $this->getFormNewLine($form, $class, $i, $fieldMid, $fieldTid, $fName, $tableAutoincrement); |
||
| 472 | } |
||
| 473 | unset($id); |
||
| 474 | |||
| 475 | // Footer form |
||
| 476 | return $fieldsForm->getFooterForm($form); |
||
| 477 | } |
||
| 528 |
Loading history...
