Conditions | 1 |
Paths | 1 |
Total Lines | 65 |
Code Lines | 26 |
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 |
||
54 | function xoops_module_install_extgallery(\XoopsModule $module) |
||
55 | { |
||
56 | require_once \dirname(__DIR__) . '/preloads/autoloader.php'; |
||
57 | |||
58 | $moduleDirName = \basename(\dirname(__DIR__)); |
||
|
|||
59 | |||
60 | $module_id = $module->getVar('mid'); |
||
61 | /** @var \XoopsGroupPermHandler $grouppermHandler */ |
||
62 | $grouppermHandler = xoops_getHandler('groupperm'); |
||
63 | /** @var \XoopsModuleHandler $moduleHandler */ |
||
64 | /** @var \XoopsConfigHandler $configHandler */ |
||
65 | $configHandler = xoops_getHandler('config'); |
||
66 | |||
67 | /** |
||
68 | * Default public category permission mask |
||
69 | */ |
||
70 | |||
71 | // Access right |
||
72 | $grouppermHandler->addRight('extgallery_public_mask', 1, XOOPS_GROUP_ADMIN, $module_id); |
||
73 | $grouppermHandler->addRight('extgallery_public_mask', 1, XOOPS_GROUP_USERS, $module_id); |
||
74 | $grouppermHandler->addRight('extgallery_public_mask', 1, XOOPS_GROUP_ANONYMOUS, $module_id); |
||
75 | |||
76 | // Public rate |
||
77 | $grouppermHandler->addRight('extgallery_public_mask', 2, XOOPS_GROUP_ADMIN, $module_id); |
||
78 | $grouppermHandler->addRight('extgallery_public_mask', 2, XOOPS_GROUP_USERS, $module_id); |
||
79 | |||
80 | // Public eCard |
||
81 | $grouppermHandler->addRight('extgallery_public_mask', 4, XOOPS_GROUP_ADMIN, $module_id); |
||
82 | $grouppermHandler->addRight('extgallery_public_mask', 4, XOOPS_GROUP_USERS, $module_id); |
||
83 | |||
84 | // Public download |
||
85 | $grouppermHandler->addRight('extgallery_public_mask', 8, XOOPS_GROUP_ADMIN, $module_id); |
||
86 | $grouppermHandler->addRight('extgallery_public_mask', 8, XOOPS_GROUP_USERS, $module_id); |
||
87 | |||
88 | // Public upload |
||
89 | $grouppermHandler->addRight('extgallery_public_mask', 16, XOOPS_GROUP_ADMIN, $module_id); |
||
90 | |||
91 | // Public autoapprove |
||
92 | $grouppermHandler->addRight('extgallery_public_mask', 32, XOOPS_GROUP_ADMIN, $module_id); |
||
93 | |||
94 | // Public display |
||
95 | $grouppermHandler->addRight('extgallery_public_mask', 128, XOOPS_GROUP_ADMIN, $module_id); |
||
96 | $grouppermHandler->addRight('extgallery_public_mask', 128, XOOPS_GROUP_USERS, $module_id); |
||
97 | $grouppermHandler->addRight('extgallery_public_mask', 128, XOOPS_GROUP_ANONYMOUS, $module_id); |
||
98 | |||
99 | /** |
||
100 | * Default User's category permission |
||
101 | */ |
||
102 | |||
103 | // Private gallery |
||
104 | |||
105 | // Private rate |
||
106 | $grouppermHandler->addRight('extgallery_private', 2, XOOPS_GROUP_ADMIN, $module_id); |
||
107 | $grouppermHandler->addRight('extgallery_private', 2, XOOPS_GROUP_USERS, $module_id); |
||
108 | |||
109 | // Private eCard |
||
110 | $grouppermHandler->addRight('extgallery_private', 4, XOOPS_GROUP_ADMIN, $module_id); |
||
111 | $grouppermHandler->addRight('extgallery_private', 4, XOOPS_GROUP_USERS, $module_id); |
||
112 | |||
113 | // Private download |
||
114 | $grouppermHandler->addRight('extgallery_private', 8, XOOPS_GROUP_ADMIN, $module_id); |
||
115 | $grouppermHandler->addRight('extgallery_private', 8, XOOPS_GROUP_USERS, $module_id); |
||
116 | |||
117 | // Private autoapprove |
||
118 | $grouppermHandler->addRight('extgallery_private', 16, XOOPS_GROUP_ADMIN, $module_id); |
||
119 | } |
||
120 |