@@ -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 |