| Conditions | 4 | 
| Paths | 4 | 
| Total Lines | 23 | 
| Code Lines | 13 | 
| Lines | 0 | 
| Ratio | 0 % | 
| Changes | 3 | ||
| Bugs | 0 | Features | 1 | 
| 1 | <?php | ||
| 10 | public function deactivateOneDayAdmins() | ||
| 11 |     { | ||
| 12 | $oneDayAdminRoles = $this->getSystemConfig()->getOneDayAdminRoles(); | ||
| 13 | |||
| 14 |         foreach ($oneDayAdminRoles as $oneDayAdminRole) { | ||
| 15 | $adminRole = $this->getAdminRolesModel() | ||
| 16 | ->load($oneDayAdminRole, 'role_name'); | ||
| 17 | |||
| 18 |             if (!$adminRole->getId()) { | ||
| 19 | continue; | ||
| 20 | } | ||
| 21 | |||
| 22 | $adminUserIdArray = $adminRole->getRoleUsers(); | ||
| 23 | |||
| 24 | $adminUserCollection = $this->getAdminUserModel()->getCollection() | ||
| 25 |                 ->addFieldToFilter('user_id', array('in' => $adminUserIdArray)) | ||
| 26 |                 ->addFieldToFilter('is_active', 1); | ||
| 27 | |||
| 28 |             foreach ($adminUserCollection as $adminUser) { | ||
| 29 |                 $adminUser->setData('is_active', 0)->save(); | ||
| 30 | } | ||
| 31 | } | ||
| 32 | } | ||
| 33 | |||
| 61 | 
You can fix this by adding a namespace to your class:
When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries.