@@ -49,11 +49,11 @@ |
||
| 49 | 49 | public function migrateAction() |
| 50 | 50 | { |
| 51 | 51 | $handler = $this->handler; |
| 52 | - foreach($this->migrators as $migrator){ |
|
| 52 | + foreach ($this->migrators as $migrator) { |
|
| 53 | 53 | $status = $handler->findOrCreate($migrator, true); |
| 54 | - if(!$status->isMigrated()){ |
|
| 54 | + if (!$status->isMigrated()) { |
|
| 55 | 55 | $success = $migrator->migrate(); |
| 56 | - if($success){ |
|
| 56 | + if ($success) { |
|
| 57 | 57 | $handler->migrated($migrator); |
| 58 | 58 | } |
| 59 | 59 | } |
@@ -20,8 +20,8 @@ |
||
| 20 | 20 | |
| 21 | 21 | $value = null; |
| 22 | 22 | $cSubject = $subject; |
| 23 | - foreach($exp as $name){ |
|
| 24 | - if(array_key_exists($name, $cSubject)){ |
|
| 23 | + foreach ($exp as $name) { |
|
| 24 | + if (array_key_exists($name, $cSubject)) { |
|
| 25 | 25 | $value = $cSubject[$name]; |
| 26 | 26 | $cSubject = $cSubject[$name]; |
| 27 | 27 | } |
@@ -67,7 +67,7 @@ |
||
| 67 | 67 | foreach ($this->repository->getUserJobs($user->getId(), $limit) as $job) |
| 68 | 68 | { |
| 69 | 69 | $title = $job->getTitle() ?: $view->translate('untitled'); |
| 70 | - $title .= ' ('. $view->dateFormat($job->getDateCreated(), 'short', 'none') . ')'; |
|
| 70 | + $title .= ' ('.$view->dateFormat($job->getDateCreated(), 'short', 'none').')'; |
|
| 71 | 71 | $url = $view->url('lang/jobs/manage', ['action' => 'edit'], [ |
| 72 | 72 | 'query' => [ |
| 73 | 73 | 'id' => $job->getId() |
@@ -203,7 +203,7 @@ |
||
| 203 | 203 | * |
| 204 | 204 | * @return UserInterface|null $user |
| 205 | 205 | */ |
| 206 | - public function getUser() ; |
|
| 206 | + public function getUser(); |
|
| 207 | 207 | |
| 208 | 208 | /** |
| 209 | 209 | * Gets the link to the application form |
@@ -69,7 +69,7 @@ discard block |
||
| 69 | 69 | public function findByAssignedPermissionsResourceId($resourceId) |
| 70 | 70 | { |
| 71 | 71 | $criteria = $this->getIsDeletedCriteria( |
| 72 | - ['permissions.assigned.' . $resourceId => [ '$exists' => true]] |
|
| 72 | + ['permissions.assigned.'.$resourceId => ['$exists' => true]] |
|
| 73 | 73 | ); |
| 74 | 74 | |
| 75 | 75 | return $this->findBy($criteria); |
@@ -140,7 +140,7 @@ discard block |
||
| 140 | 140 | $qb = $this->createQueryBuilder(); |
| 141 | 141 | $qb->distinct('organization') |
| 142 | 142 | ->hydrate(true) |
| 143 | - ->field('status.name')->notIn([ StatusInterface::EXPIRED, StatusInterface::INACTIVE ]); |
|
| 143 | + ->field('status.name')->notIn([StatusInterface::EXPIRED, StatusInterface::INACTIVE]); |
|
| 144 | 144 | |
| 145 | 145 | $q = $qb->getQuery(); |
| 146 | 146 | $r = $q->execute(); |
@@ -150,10 +150,10 @@ discard block |
||
| 150 | 150 | $qb->field('_id')->in($r); |
| 151 | 151 | if ($term) { |
| 152 | 152 | $qb->field('_organizationName') |
| 153 | - ->equals(new Regex('/' . addslashes($term) . '/i')); |
|
| 153 | + ->equals(new Regex('/'.addslashes($term).'/i')); |
|
| 154 | 154 | } |
| 155 | 155 | |
| 156 | - if($execute){ |
|
| 156 | + if ($execute) { |
|
| 157 | 157 | return $qb->getQuery()->execute(); |
| 158 | 158 | } |
| 159 | 159 | |
@@ -228,7 +228,7 @@ discard block |
||
| 228 | 228 | */ |
| 229 | 229 | public function createQueryBuilder($isDeleted = false): QueryBuilder |
| 230 | 230 | { |
| 231 | - $qb = parent::createQueryBuilder(); |
|
| 231 | + $qb = parent::createQueryBuilder(); |
|
| 232 | 232 | |
| 233 | 233 | if (null !== $isDeleted) { |
| 234 | 234 | $qb->addAnd( |
@@ -53,7 +53,7 @@ |
||
| 53 | 53 | $repo = $this->getDoctrine()->getRepository(Organization::class); |
| 54 | 54 | $org = $repo->findOneBy(['_organizationName' => $name]); |
| 55 | 55 | |
| 56 | - if(is_null($org)){ |
|
| 56 | + if (is_null($org)) { |
|
| 57 | 57 | $org = new Organization(); |
| 58 | 58 | $org->setOrganizationName(new OrganizationName($name)); |
| 59 | 59 | $this->getDoctrine()->persist($org); |
@@ -62,71 +62,71 @@ |
||
| 62 | 62 | $parent = new Organization(); |
| 63 | 63 | $permissions = $this->createMock(Permissions::class); |
| 64 | 64 | return [ |
| 65 | - [ 'externalId', '1234' ], |
|
| 66 | - [ 'name', [ 'value'=> '', 'ignore_setter' => true] ], |
|
| 67 | - [ 'name', [ 'pre' => 'setupGetName', 'value' => 'Test', 'ignore_setter' => true] ], |
|
| 68 | - [ 'organizationName' , [ |
|
| 69 | - '@value' => [ '\Organizations\Entity\OrganizationName', [ 'Test' ] ], |
|
| 65 | + ['externalId', '1234'], |
|
| 66 | + ['name', ['value'=> '', 'ignore_setter' => true]], |
|
| 67 | + ['name', ['pre' => 'setupGetName', 'value' => 'Test', 'ignore_setter' => true]], |
|
| 68 | + ['organizationName', [ |
|
| 69 | + '@value' => ['\Organizations\Entity\OrganizationName', ['Test']], |
|
| 70 | 70 | 'post' => 'organizationNamePostAsserts' |
| 71 | 71 | ]], |
| 72 | - [ 'organizationName', [ |
|
| 72 | + ['organizationName', [ |
|
| 73 | 73 | 'pre' => 'setupOrganizationName', |
| 74 | - '@value' => [ '\Organizations\Entity\OrganizationName', [ 'TestNew' ] ], |
|
| 74 | + '@value' => ['\Organizations\Entity\OrganizationName', ['TestNew']], |
|
| 75 | 75 | 'post' => 'assertOrganizationNameCounter' |
| 76 | 76 | ]], |
| 77 | - [ 'permissions',[ |
|
| 77 | + ['permissions', [ |
|
| 78 | 78 | 'value' => $permissions |
| 79 | 79 | ]], |
| 80 | - [ 'images',[ |
|
| 80 | + ['images', [ |
|
| 81 | 81 | 'value' => new ImageSet(), |
| 82 | 82 | ]], |
| 83 | - [ 'isDraft',[ |
|
| 83 | + ['isDraft', [ |
|
| 84 | 84 | 'default' => false, |
| 85 | 85 | 'value' => true, |
| 86 | 86 | 'getter_method' => 'isDraft' |
| 87 | 87 | ]], |
| 88 | - [ 'contact',[ |
|
| 88 | + ['contact', [ |
|
| 89 | 89 | 'value' => new OrganizationContact() |
| 90 | 90 | ]], |
| 91 | - [ 'description',[ |
|
| 91 | + ['description', [ |
|
| 92 | 92 | 'default' => null, |
| 93 | 93 | 'value' => 'Some Description' |
| 94 | 94 | ]], |
| 95 | - [ 'parent',[ |
|
| 95 | + ['parent', [ |
|
| 96 | 96 | 'default' => null, |
| 97 | 97 | 'value' => $parent, |
| 98 | 98 | ]], |
| 99 | - [ 'hiringOrganizations',[ |
|
| 99 | + ['hiringOrganizations', [ |
|
| 100 | 100 | 'default' => null, |
| 101 | 101 | ]], |
| 102 | 102 | |
| 103 | - [ 'employees',[ |
|
| 103 | + ['employees', [ |
|
| 104 | 104 | '@default' => ArrayCollection::class, |
| 105 | 105 | 'value' => new ArrayCollection() |
| 106 | 106 | ]], |
| 107 | 107 | |
| 108 | - [ 'jobs',[ |
|
| 108 | + ['jobs', [ |
|
| 109 | 109 | 'default' => null, |
| 110 | 110 | ]], |
| 111 | 111 | |
| 112 | - [ 'user',[ |
|
| 112 | + ['user', [ |
|
| 113 | 113 | 'value' => new User() |
| 114 | 114 | ]], |
| 115 | 115 | |
| 116 | - [ 'template',[ |
|
| 116 | + ['template', [ |
|
| 117 | 117 | 'value' => new Template(), |
| 118 | 118 | ]], |
| 119 | 119 | |
| 120 | - [ 'workflowSettings',[ |
|
| 120 | + ['workflowSettings', [ |
|
| 121 | 121 | 'value' => new WorkflowSettings(), |
| 122 | 122 | ]], |
| 123 | 123 | |
| 124 | - [ 'profileSetting',[ |
|
| 124 | + ['profileSetting', [ |
|
| 125 | 125 | 'value' => Organization::PROFILE_ALWAYS_ENABLE, |
| 126 | 126 | 'default' => null, |
| 127 | 127 | ]], |
| 128 | 128 | |
| 129 | - [ 'hydrator', [ |
|
| 129 | + ['hydrator', [ |
|
| 130 | 130 | '@default' => EntityHydrator::class |
| 131 | 131 | ]] |
| 132 | 132 | |
@@ -48,7 +48,7 @@ |
||
| 48 | 48 | |
| 49 | 49 | public function testGetUriWithAndWithoutFilename() |
| 50 | 50 | { |
| 51 | - $filename="My File.png"; |
|
| 51 | + $filename = "My File.png"; |
|
| 52 | 52 | $this->target->setName($filename); |
| 53 | 53 | $this->assertSame($this->target->getUri(), '/file/Organizations.OrganizationImage/'.$this->target->getId().'/'.urlencode($this->target->getName())); |
| 54 | 54 | |
@@ -78,7 +78,7 @@ |
||
| 78 | 78 | $builder->expects($this->any()) |
| 79 | 79 | ->method('field') |
| 80 | 80 | ->willReturnMap([ |
| 81 | - ['permissions.view',$builder] |
|
| 81 | + ['permissions.view', $builder] |
|
| 82 | 82 | ]) |
| 83 | 83 | ; |
| 84 | 84 | |