txmodxoops /
tdmcreate
Duplicate code is one of the most pungent code smells. A rule that is often used is to re-structure code once it is duplicated in three or more places.
Common duplication problems, and corresponding solutions are:
Complex classes like TDMCreateXoopsCode often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes. You can also have a look at the cohesion graph to spot any un-connected, or weakly-connected components.
Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.
While breaking up the class, it is a good idea to analyze how other classes use TDMCreateXoopsCode, and based on these observations, apply Extract Interface, too.
| 1 | <?php |
||
| 28 | class TDMCreateXoopsCode |
||
| 29 | { |
||
| 30 | /* |
||
| 31 | * @var mixed |
||
| 32 | */ |
||
| 33 | private $phpcode = null; |
||
| 34 | |||
| 35 | /* |
||
| 36 | * @public function constructor |
||
| 37 | * @param null |
||
| 38 | */ |
||
| 39 | /** |
||
| 40 | * |
||
| 41 | */ |
||
| 42 | public function __construct() |
||
| 46 | |||
| 47 | /* |
||
| 48 | * @static function &getInstance |
||
| 49 | * @param null |
||
| 50 | */ |
||
| 51 | /** |
||
| 52 | * @return TDMCreateXoopsCode |
||
| 53 | */ |
||
| 54 | public static function &getInstance() |
||
| 63 | |||
| 64 | /* |
||
| 65 | * @public function getXoopsCodeEqualsOperator |
||
| 66 | * @param string $left |
||
| 67 | * @param string $right |
||
| 68 | * @param boolean $ref |
||
| 69 | * @return string |
||
| 70 | */ |
||
| 71 | public function getXoopsCodeEqualsOperator($left, $right, $ref = false) |
||
| 81 | |||
| 82 | /* |
||
| 83 | * @private function getXoopsCodeSwitch |
||
| 84 | * @param $op |
||
| 85 | * @param $listCases |
||
| 86 | * @param $defaultList |
||
| 87 | * |
||
| 88 | * @return string |
||
| 89 | */ |
||
| 90 | private function getXoopsCodeSwitch($op = 'op', $listCases = array(), $defaultList = false) |
||
| 96 | |||
| 97 | /* |
||
| 98 | * @public function getXoopsCodeCPHeader |
||
| 99 | * @param null |
||
| 100 | * @return string |
||
| 101 | */ |
||
| 102 | public function getXoopsCodeCPHeader() |
||
| 106 | |||
| 107 | /* |
||
| 108 | * @public function getXoopsCodeCPFooter |
||
| 109 | * @param null |
||
| 110 | * @return string |
||
| 111 | */ |
||
| 112 | public function getXoopsCodeCPFooter() |
||
| 116 | |||
| 117 | /** |
||
| 118 | * @public function getXoopsCodeLoad |
||
| 119 | * |
||
| 120 | * @param $var |
||
| 121 | * |
||
| 122 | * @return string |
||
| 123 | */ |
||
| 124 | public function getXoopsCodeLoad($var = '') |
||
| 128 | |||
| 129 | /** |
||
| 130 | * @public function getXoopsCodeLoadLanguage |
||
| 131 | * |
||
| 132 | * @param $lang |
||
| 133 | * |
||
| 134 | * @return string |
||
| 135 | */ |
||
| 136 | public function getXoopsCodeLoadLanguage($lang) |
||
| 140 | |||
| 141 | /* |
||
| 142 | * @public function getXoopsCodeSetVar |
||
| 143 | * @param string $tableName |
||
| 144 | * @param string $fieldName |
||
| 145 | * @param string $var |
||
| 146 | * @return string |
||
| 147 | */ |
||
| 148 | public function getXoopsCodeSetVar($tableName, $fieldName, $var) |
||
| 152 | |||
| 153 | /* |
||
| 154 | * @public function getXoopsCodeGetVar |
||
| 155 | * @param string $varLeft |
||
| 156 | * @param string $handle |
||
| 157 | * @param string $var |
||
| 158 | * @param string $isParam |
||
| 159 | * |
||
| 160 | * @return string |
||
| 161 | */ |
||
| 162 | public function getXoopsCodeGetVar($varLeft = '', $handle = '', $var = '', $isParam = false) |
||
| 172 | |||
| 173 | /* |
||
| 174 | * @public function getXoopsCodeGroupPermForm |
||
| 175 | * @param string $varLeft |
||
| 176 | * @param string $formTitle |
||
| 177 | * @param string $moduleId |
||
| 178 | * @param string $permName |
||
| 179 | * @param string $permDesc |
||
| 180 | * @param string $filename |
||
| 181 | * |
||
| 182 | * @return string |
||
| 183 | */ |
||
| 184 | public function getXoopsCodeGroupPermForm($varLeft = '', $formTitle = '', $moduleId = '', $permName = '', $permDesc = '', $filename = '') |
||
| 188 | |||
| 189 | /* |
||
| 190 | * @public function getXoopsCodeAddItem |
||
| 191 | * @param string $varLeft |
||
| 192 | * @param string $paramLeft |
||
| 193 | * @param string $paramRight |
||
| 194 | * |
||
| 195 | * @return string |
||
| 196 | */ |
||
| 197 | public function getXoopsCodeAddItem($varLeft = '', $paramLeft = '', $paramRight = '') |
||
| 201 | |||
| 202 | /* |
||
| 203 | * @public function getXoopsCodeTextDateSelectSetVar |
||
| 204 | * @param string $tableName |
||
| 205 | * @param string $fieldName |
||
| 206 | * @return string |
||
| 207 | */ |
||
| 208 | public function getXoopsCodeTextDateSelectSetVar($tableName, $fieldName) |
||
| 212 | |||
| 213 | /* |
||
| 214 | * @public function getXoopsCodeCheckBoxOrRadioYNSetVar |
||
| 215 | * @param $tableName |
||
| 216 | * @param $fieldName |
||
| 217 | * @return string |
||
| 218 | */ |
||
| 219 | public function getXoopsCodeCheckBoxOrRadioYNSetVar($tableName, $fieldName) |
||
| 223 | |||
| 224 | /* |
||
| 225 | * @public function getXoopsCodeXoopsMediaUploader |
||
| 226 | * @param $var |
||
| 227 | * @param $dirPath |
||
| 228 | * @param $tableName |
||
| 229 | * @param $moduleDirname |
||
| 230 | * @return string |
||
| 231 | */ |
||
| 232 | public function getXoopsCodeXoopsMediaUploader($var = '', $dirPath, $tableName, $moduleDirname) |
||
| 239 | |||
| 240 | /* |
||
| 241 | * @public function getXoopsCodeGetConfig |
||
| 242 | * @param $moduleDirname |
||
| 243 | * @param $name |
||
| 244 | * @return string |
||
| 245 | */ |
||
| 246 | public function getXoopsCodeGetConfig($moduleDirname, $name) |
||
| 250 | |||
| 251 | /* |
||
| 252 | * @public function getXoopsCodeIdGetVar |
||
| 253 | * @param string $lpFieldName |
||
| 254 | * @return string |
||
| 255 | */ |
||
| 256 | public function getXoopsCodeIdGetVar($lpFieldName) |
||
| 260 | |||
| 261 | /* |
||
| 262 | * @public function getXoopsCodeGetVarAll |
||
| 263 | * @param string $lpFieldName |
||
| 264 | * @param string $rpFieldName |
||
| 265 | * @param string $tableName |
||
| 266 | * @param string $fieldName |
||
| 267 | * @return string |
||
| 268 | */ |
||
| 269 | public function getXoopsCodeGetVarAll($lpFieldName, $rpFieldName, $tableName, $fieldName) |
||
| 273 | |||
| 274 | /* |
||
| 275 | * @public function getXoopsHandlerLine |
||
| 276 | * @param $moduleDirname |
||
| 277 | * @param $tableName |
||
| 278 | * @return string |
||
| 279 | */ |
||
| 280 | public function getXoopsHandlerLine($moduleDirname, $tableName) |
||
| 284 | |||
| 285 | /* |
||
| 286 | * @public function getXoopsSimpleForm |
||
| 287 | * @param $left |
||
| 288 | * @param $element |
||
| 289 | * @param $elementsContent |
||
| 290 | * @param $caption |
||
| 291 | * @param $var |
||
| 292 | * @param $filename |
||
| 293 | * @param $type |
||
| 294 | * |
||
| 295 | * @return string |
||
| 296 | */ |
||
| 297 | public function getXoopsSimpleForm($left = '', $element = '', $elementsContent = '', $caption = '', $var = '', $filename = '', $type = 'post') |
||
| 308 | |||
| 309 | /* |
||
| 310 | * @public function getXoopsFormSelect |
||
| 311 | * @param $varSelect |
||
| 312 | * @param $caption |
||
| 313 | * @param $var |
||
| 314 | * @param $options |
||
| 315 | * @param $setExtra |
||
| 316 | * |
||
| 317 | * @return string |
||
| 318 | */ |
||
| 319 | public function getXoopsFormSelect($varSelect = '', $caption = '', $var = '', $options = array(), $setExtra = true) |
||
| 331 | |||
| 332 | /* |
||
| 333 | * @public function getXoopsCodeTopicGetVar |
||
| 334 | * @param string $lpFieldName |
||
| 335 | * @param string $rpFieldName |
||
| 336 | * @param string $tableName |
||
| 337 | * @param string $tableNameTopic |
||
| 338 | * @param string $fieldNameParent |
||
| 339 | * @param string $fieldNameTopic |
||
| 340 | * @return string |
||
| 341 | */ |
||
| 342 | public function getXoopsCodeTopicGetVar($lpFieldName, $rpFieldName, $tableName, $tableNameTopic, $fieldNameParent, $fieldNameTopic) |
||
| 352 | |||
| 353 | /* |
||
| 354 | * @public function getXoopsCodeParentTopicGetVar |
||
| 355 | * @param string $moduleDirname |
||
| 356 | * @param string $lpFieldName |
||
| 357 | * @param string $rpFieldName |
||
| 358 | * @param string $tableName |
||
| 359 | * @param string $tableSoleNameTopic |
||
| 360 | * @param string $tableNameTopic |
||
| 361 | * @param string $fieldNameParent |
||
| 362 | * @return string |
||
| 363 | */ |
||
| 364 | View Code Duplication | public function getXoopsCodeParentTopicGetVar($moduleDirname, $lpFieldName, $rpFieldName, $tableName, $tableSoleNameTopic, $tableNameTopic, $fieldNameParent) |
|
| 377 | |||
| 378 | /* |
||
| 379 | * @public function getXoopsCodeUploadImageGetVar |
||
| 380 | * @param string $lpFieldName |
||
| 381 | * @param string $rpFieldName |
||
| 382 | * @param string $tableName |
||
| 383 | * @param string $fieldName |
||
| 384 | * @return string |
||
| 385 | */ |
||
| 386 | public function getXoopsCodeUploadImageGetVar($lpFieldName, $rpFieldName, $tableName, $fieldName) |
||
| 396 | /* |
||
| 397 | * @public function getXoopsCodeUrlFileGetVar |
||
| 398 | * @param string $lpFieldName |
||
| 399 | * @param string $rpFieldName |
||
| 400 | * @param string $tableName |
||
| 401 | * @param string $fieldName |
||
| 402 | * @return string |
||
| 403 | */ |
||
| 404 | public function getXoopsCodeUrlFileGetVar($lpFieldName, $rpFieldName, $tableName, $fieldName) |
||
| 408 | /* |
||
| 409 | * @public function getXoopsCodeTextAreaGetVar |
||
| 410 | * @param string $lpFieldName |
||
| 411 | * @param string $rpFieldName |
||
| 412 | * @param string $tableName |
||
| 413 | * @param string $fieldName |
||
| 414 | * @return string |
||
| 415 | */ |
||
| 416 | public function getXoopsCodeTextAreaGetVar($lpFieldName, $rpFieldName, $tableName, $fieldName) |
||
| 420 | |||
| 421 | /* |
||
| 422 | * @public function getXoopsCodeSelectUserGetVar |
||
| 423 | * @param string $lpFieldName |
||
| 424 | * @param string $rpFieldName |
||
| 425 | * @param string $tableName |
||
| 426 | * @param string $fieldName |
||
| 427 | * @return string |
||
| 428 | */ |
||
| 429 | public function getXoopsCodeSelectUserGetVar($lpFieldName, $rpFieldName, $tableName, $fieldName) |
||
| 433 | |||
| 434 | /* |
||
| 435 | * @public function getXoopsCodeTextDateSelectGetVar |
||
| 436 | * @param string $lpFieldName |
||
| 437 | * @param string $rpFieldName |
||
| 438 | * @param string $tableName |
||
| 439 | * @param string $fieldName |
||
| 440 | * @return string |
||
| 441 | */ |
||
| 442 | public function getXoopsCodeTextDateSelectGetVar($lpFieldName, $rpFieldName, $tableName, $fieldName) |
||
| 446 | |||
| 447 | /* |
||
| 448 | * @public function getXoopsCodeUserHeader |
||
| 449 | * @param string $moduleDirname |
||
| 450 | * @param string $tableName |
||
| 451 | * @return string |
||
| 452 | */ |
||
| 453 | public function getXoopsCodeXoopsOptionTemplateMain($moduleDirname, $tableName) |
||
| 457 | |||
| 458 | /* |
||
| 459 | * @public function getXoopsCodeUserHeader |
||
| 460 | * @param string $moduleDirname |
||
| 461 | * @param string $tableName |
||
| 462 | * @return string |
||
| 463 | */ |
||
| 464 | public function getXoopsCodeUserHeader($moduleDirname, $tableName) |
||
| 472 | |||
| 473 | /* |
||
| 474 | * @public function getXoopsCodePermissionsHeader |
||
| 475 | * @param null |
||
| 476 | */ |
||
| 477 | /** |
||
| 478 | * @return string |
||
| 479 | */ |
||
| 480 | public function getXoopsCodePermissionsHeader() |
||
| 491 | |||
| 492 | /** |
||
| 493 | * @public function getXoopsCodeGetFieldId |
||
| 494 | * |
||
| 495 | * @param $fields |
||
| 496 | * |
||
| 497 | * @return string |
||
| 498 | */ |
||
| 499 | View Code Duplication | public function getXoopsCodeGetFieldId($fields) |
|
| 511 | |||
| 512 | /** |
||
| 513 | * @public function getXoopsCodeGetFieldName |
||
| 514 | * |
||
| 515 | * @param $fields |
||
| 516 | * |
||
| 517 | * @return string |
||
| 518 | */ |
||
| 519 | public function getXoopsCodeGetFieldName($fields) |
||
| 527 | |||
| 528 | /** |
||
| 529 | * @public function getXoopsCodeGetFieldParentId |
||
| 530 | * |
||
| 531 | * @param $fields |
||
| 532 | * |
||
| 533 | * @return string |
||
| 534 | */ |
||
| 535 | View Code Duplication | public function getXoopsCodeGetFieldParentId($fields) |
|
| 547 | |||
| 548 | /** |
||
| 549 | * @public function getXoopsCodeUserSaveElements |
||
| 550 | * |
||
| 551 | * @param $moduleDirname |
||
| 552 | * @param $tableName |
||
| 553 | * @param $fields |
||
| 554 | * |
||
| 555 | * @return string |
||
| 556 | */ |
||
| 557 | View Code Duplication | public function getXoopsCodeUserSaveElements($moduleDirname, $tableName, $fields) |
|
| 581 | |||
| 582 | /* |
||
| 583 | * @public function getXoopsCodeXoopsRequest |
||
| 584 | * @param $left |
||
| 585 | * @param $var1 |
||
| 586 | * @param $var2 |
||
| 587 | * @param $type |
||
| 588 | * @param $metod |
||
| 589 | * @return string |
||
| 590 | */ |
||
| 591 | public function getXoopsCodeXoopsRequest($left = '', $var1 = '', $var2 = '', $type = 'String', $metod = false) |
||
| 605 | |||
| 606 | /** |
||
| 607 | * @public function getXoopsCodeTplAssign |
||
| 608 | * |
||
| 609 | * @param string $tplString |
||
| 610 | * @param string $phpRender |
||
| 611 | * |
||
| 612 | * @return string |
||
| 613 | */ |
||
| 614 | public function getXoopsCodeTplAssign($tplString, $phpRender) |
||
| 618 | |||
| 619 | /** |
||
| 620 | * @public function getXoopsCodeXoopsTplAppend |
||
| 621 | * |
||
| 622 | * @param string $tplString |
||
| 623 | * @param string $phpRender |
||
| 624 | * |
||
| 625 | * @return string |
||
| 626 | */ |
||
| 627 | public function getXoopsCodeXoopsTplAppend($tplString, $phpRender) |
||
| 631 | |||
| 632 | /** |
||
| 633 | * @public function getXoopsCodeXoopsTplAppendByRef |
||
| 634 | * |
||
| 635 | * @param string $tplString |
||
| 636 | * @param string $phpRender |
||
| 637 | * |
||
| 638 | * @return string |
||
| 639 | */ |
||
| 640 | public function getXoopsCodeXoopsTplAppendByRef($tplString, $phpRender) |
||
| 644 | |||
| 645 | /** |
||
| 646 | * @public function getXoopsCodePath |
||
| 647 | * |
||
| 648 | * @param $directory |
||
| 649 | * @param $filename |
||
| 650 | * @param $condition |
||
| 651 | * |
||
| 652 | * @return string |
||
| 653 | */ |
||
| 654 | public function getXoopsCodePath($directory, $filename, $condition = false) |
||
| 664 | |||
| 665 | /** |
||
| 666 | * @public function getXoopsCodeTplDisplay |
||
| 667 | * |
||
| 668 | * @param null |
||
| 669 | * |
||
| 670 | * @return string |
||
| 671 | */ |
||
| 672 | public function getXoopsCodeTplDisplay() |
||
| 676 | |||
| 677 | /** |
||
| 678 | * @public function getXoopsCodeGetInfo |
||
| 679 | * |
||
| 680 | * @param $string |
||
| 681 | * @param $isParam |
||
| 682 | * |
||
| 683 | * @return string |
||
| 684 | */ |
||
| 685 | public function getXoopsCodeGetInfo($string, $isParam = false) |
||
| 695 | |||
| 696 | /** |
||
| 697 | * @public function getXoopsCodeObjHandlerCreate |
||
| 698 | * |
||
| 699 | * @param string $tableName |
||
| 700 | * |
||
| 701 | * @return string |
||
| 702 | */ |
||
| 703 | public function getXoopsCodeObjHandlerCreate($tableName) |
||
| 709 | |||
| 710 | /** |
||
| 711 | * @public function getXoopsCodeObjHandlerCount |
||
| 712 | * |
||
| 713 | * @param string $tableName |
||
| 714 | * |
||
| 715 | * @return string |
||
| 716 | */ |
||
| 717 | public function getXoopsCodeObjHandlerCount($tableName) |
||
| 724 | |||
| 725 | /** |
||
| 726 | * @public function getXoopsCodeObjHandlerAll |
||
| 727 | * |
||
| 728 | * @param string $tableName |
||
| 729 | * @param string $fieldMain |
||
| 730 | * @param string $start |
||
| 731 | * @param string $limit |
||
| 732 | * |
||
| 733 | * @return string |
||
| 734 | */ |
||
| 735 | public function getXoopsCodeObjHandlerAll($tableName, $fieldMain, $start = '0', $limit = '0') |
||
| 743 | |||
| 744 | /** |
||
| 745 | * @public function getXoopsCodeSetVarsObjects |
||
| 746 | * |
||
| 747 | * @param $moduleDirname |
||
| 748 | * @param $tableName |
||
| 749 | * @param $fields |
||
| 750 | * |
||
| 751 | * @return string |
||
| 752 | */ |
||
| 753 | public function getXoopsCodeSetVarsObjects($moduleDirname, $tableName, $fields) |
||
| 792 | |||
| 793 | /** |
||
| 794 | * @public function getXoopsCodeSecurity |
||
| 795 | * |
||
| 796 | * @param $tableName |
||
| 797 | * |
||
| 798 | * @return string |
||
| 799 | */ |
||
| 800 | public function getXoopsCodeSecurity($tableName) |
||
| 809 | |||
| 810 | /* |
||
| 811 | * @public function getXoopsCodeInsertData |
||
| 812 | * @param $tableName |
||
| 813 | * @param $language |
||
| 814 | * @return string |
||
| 815 | */ |
||
| 816 | public function getXoopsCodeInsertData($tableName, $language) |
||
| 823 | |||
| 824 | /* |
||
| 825 | * @public function getXoopsCodeRedirectHeader |
||
| 826 | * @param $tableName |
||
| 827 | * @param $options |
||
| 828 | * @param $numb |
||
| 829 | * @param $var |
||
| 830 | * @return string |
||
| 831 | */ |
||
| 832 | public function getXoopsCodeRedirectHeader($tableName, $options = '', $numb = '2', $var) |
||
| 836 | |||
| 837 | /* |
||
| 838 | * @public function getXoopsCodeXoopsConfirm |
||
| 839 | * @param $tableName |
||
| 840 | * @param $language |
||
| 841 | * @param $fieldId |
||
| 842 | * @param $fieldMain |
||
| 843 | * @param $options |
||
| 844 | * |
||
| 845 | * @return string |
||
| 846 | */ |
||
| 847 | public function getXoopsCodeXoopsConfirm($tableName, $language, $fieldId, $fieldMain, $options = 'delete') |
||
| 854 | |||
| 855 | /* |
||
| 856 | * @public function getXoopsCodeSecurityCheck |
||
| 857 | * @param null |
||
| 858 | * @return boolean |
||
| 859 | */ |
||
| 860 | public function getXoopsCodeSecurityCheck() |
||
| 864 | |||
| 865 | /* |
||
| 866 | * @public function getXoopsCodeSecurityGetError |
||
| 867 | * @param null |
||
| 868 | * @return string |
||
| 869 | */ |
||
| 870 | public function getXoopsCodeSecurityGetError() |
||
| 874 | |||
| 875 | /** |
||
| 876 | * @public function getXoopsCodeGetHtmlErrors |
||
| 877 | * |
||
| 878 | * @param $tableName |
||
| 879 | * @param $isParam |
||
| 880 | * @param $obj |
||
| 881 | * |
||
| 882 | * @return string |
||
| 883 | */ |
||
| 884 | public function getXoopsCodeGetHtmlErrors($tableName, $isParam = false, $obj = 'Obj') |
||
| 895 | |||
| 896 | /** |
||
| 897 | * @public function getXoopsCodeObjHandlerCount |
||
| 898 | * |
||
| 899 | * @param $left |
||
| 900 | * @param $tableName |
||
| 901 | * @param $obj |
||
| 902 | * |
||
| 903 | * @return string |
||
| 904 | */ |
||
| 905 | public function getXoopsCodeGetForm($left, $tableName, $obj = '') |
||
| 909 | |||
| 910 | /** |
||
| 911 | * @public function getXoopsCodeGet |
||
| 912 | * |
||
| 913 | * @param string $tableName |
||
| 914 | * @param string $var |
||
| 915 | * @param string $obj |
||
| 916 | * @param string $isHandler |
||
| 917 | * @param string $isParam |
||
| 918 | * |
||
| 919 | * @return string |
||
| 920 | */ |
||
| 921 | public function getXoopsCodeGet($tableName, $var, $obj = '', $isHandler = false, $isParam = false) |
||
| 939 | |||
| 940 | /** |
||
| 941 | * @public function getXoopsCodeHandler |
||
| 942 | * |
||
| 943 | * @param string $tableName |
||
| 944 | * @param string $var |
||
| 945 | * @param string $obj |
||
| 946 | * |
||
| 947 | * @return string |
||
| 948 | */ |
||
| 949 | public function getXoopsCodeInsert($tableName, $var, $obj = '') |
||
| 959 | |||
| 960 | /** |
||
| 961 | * @public function getXoopsCodeDelete |
||
| 962 | * |
||
| 963 | * @param string $tableName |
||
| 964 | * @param string $var |
||
| 965 | * @param string $obj |
||
| 966 | * @param string $isHandler |
||
| 967 | * |
||
| 968 | * @return string |
||
| 969 | */ |
||
| 970 | public function getXoopsCodeDelete($tableName, $var, $obj = '', $isHandler = false) |
||
| 987 | |||
| 988 | /** |
||
| 989 | * @public function getXoopsCodeHandler |
||
| 990 | * |
||
| 991 | * @param string $tableName |
||
| 992 | * @param string $var |
||
| 993 | * |
||
| 994 | * @return string |
||
| 995 | */ |
||
| 996 | public function getXoopsCodeHandler($tableName, $var, $get = false, $insert = false, $delete = false, $obj = '') |
||
| 1008 | |||
| 1009 | /* |
||
| 1010 | * @public function getXoopsCodeCaseDelete |
||
| 1011 | * @param string $tableName |
||
| 1012 | * @param string $language |
||
| 1013 | * @param string $fieldId |
||
| 1014 | * @param string $fieldMain |
||
| 1015 | * @return string |
||
| 1016 | */ |
||
| 1017 | public function getXoopsCodeCaseDelete($language, $tableName, $fieldId, $fieldMain) |
||
| 1041 | |||
| 1042 | /* |
||
| 1043 | * @public function getTopicGetVar |
||
| 1044 | * @param string $lpFieldName |
||
| 1045 | * @param string $rpFieldName |
||
| 1046 | * @param string $tableName |
||
| 1047 | * @param string $tableNameTopic |
||
| 1048 | * @param string $fieldNameParent |
||
| 1049 | * @param string $fieldNameTopic |
||
| 1050 | * @return string |
||
| 1051 | */ |
||
| 1052 | public function getTopicGetVar($lpFieldName, $rpFieldName, $tableName, $tableNameTopic, $fieldNameParent, $fieldNameTopic) |
||
| 1062 | |||
| 1063 | /* |
||
| 1064 | * @public function getUploadImageGetVar |
||
| 1065 | * @param string $lpFieldName |
||
| 1066 | * @param string $rpFieldName |
||
| 1067 | * @param string $tableName |
||
| 1068 | * @param string $fieldName |
||
| 1069 | * @return string |
||
| 1070 | */ |
||
| 1071 | public function getUploadImageGetVar($lpFieldName, $rpFieldName, $tableName, $fieldName) |
||
| 1082 | |||
| 1083 | /* |
||
| 1084 | * @public function getXoopsCodeUpdate |
||
| 1085 | * @param $language |
||
| 1086 | * @param $tableName |
||
| 1087 | * @param $fieldId |
||
| 1088 | * @param $fieldName |
||
| 1089 | * @return string |
||
| 1090 | */ |
||
| 1091 | public function getXoopsCodeUpdate($language, $tableName, $fieldId, $fieldName) |
||
| 1105 | |||
| 1106 | /** |
||
| 1107 | * @public function getXoopsCodeSaveFieldId |
||
| 1108 | * |
||
| 1109 | * @param $fields |
||
| 1110 | * |
||
| 1111 | * @return string |
||
| 1112 | */ |
||
| 1113 | View Code Duplication | public function getXoopsCodeSaveFieldId($fields) |
|
| 1123 | |||
| 1124 | /** |
||
| 1125 | * @public function getXoopsCodeSaveFieldMain |
||
| 1126 | * |
||
| 1127 | * @param $fields |
||
| 1128 | * |
||
| 1129 | * @return string |
||
| 1130 | */ |
||
| 1131 | View Code Duplication | public function getXoopsCodeSaveFieldMain($fields) |
|
| 1141 | |||
| 1142 | /** |
||
| 1143 | * @public function getXoopsCodeSaveElements |
||
| 1144 | * |
||
| 1145 | * @param $moduleDirname |
||
| 1146 | * @param $tableName |
||
| 1147 | * @param $fields |
||
| 1148 | * |
||
| 1149 | * @return string |
||
| 1150 | */ |
||
| 1151 | View Code Duplication | public function getXoopsCodeSaveElements($moduleDirname, $tableName, $fields) |
|
| 1175 | } |
||
| 1176 |
Loading history...
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.