@@ -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 |
@@ -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 |
@@ -83,7 +83,7 @@ discard block |
||
83 | 83 | |
84 | 84 | $this->fileManager = $this->createMock(FileManager::class); |
85 | 85 | |
86 | - $this->listener = new ApplicationListener($this->cacheManager,$this->fileManager); |
|
86 | + $this->listener = new ApplicationListener($this->cacheManager, $this->fileManager); |
|
87 | 87 | } |
88 | 88 | |
89 | 89 | /** |
@@ -166,7 +166,7 @@ discard block |
||
166 | 166 | { |
167 | 167 | $baseUrl = '/base'; |
168 | 168 | $path = '/some/uri'; |
169 | - $uri = $baseUrl . $path; |
|
169 | + $uri = $baseUrl.$path; |
|
170 | 170 | $this->event->setError(Application::ERROR_ROUTER_NO_MATCH); |
171 | 171 | |
172 | 172 | $this->request->expects($this->once()) |
@@ -201,7 +201,7 @@ discard block |
||
201 | 201 | { |
202 | 202 | $id = 'someId'; |
203 | 203 | $ext = 'ext'; |
204 | - $name = 'filename.' . $ext; |
|
204 | + $name = 'filename.'.$ext; |
|
205 | 205 | $path = sprintf('d/I/%s.%s', $id, $ext); |
206 | 206 | $resource = 'someResource'; |
207 | 207 | |
@@ -238,7 +238,7 @@ discard block |
||
238 | 238 | */ |
239 | 239 | public function testMatchUri($uri, $expected) |
240 | 240 | { |
241 | - $this->assertSame($expected, $this->manager->matchUri($this->options->getUriPath() . $uri)); |
|
241 | + $this->assertSame($expected, $this->manager->matchUri($this->options->getUriPath().$uri)); |
|
242 | 242 | } |
243 | 243 | |
244 | 244 | /** |