@@ -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 | /** |
@@ -143,7 +143,7 @@ discard block |
||
| 143 | 143 | try { |
| 144 | 144 | /* @var $handler \Organizations\Controller\Plugin\GetOrganizationHandler */ |
| 145 | 145 | $handler = $this->plugin('Organizations/GetOrganizationHandler'); |
| 146 | - $org = $handler->process($this->params(), true); |
|
| 146 | + $org = $handler->process($this->params(), true); |
|
| 147 | 147 | } catch (MissingParentOrganizationException $e) { |
| 148 | 148 | return $this->getErrorViewModel('no-parent'); |
| 149 | 149 | } catch (NotFoundException $e) { |
@@ -154,7 +154,7 @@ discard block |
||
| 154 | 154 | ]; |
| 155 | 155 | } |
| 156 | 156 | |
| 157 | - $container = $this->getFormular($org); |
|
| 157 | + $container = $this->getFormular($org); |
|
| 158 | 158 | |
| 159 | 159 | if (isset($formIdentifier) && $request->isPost()) { |
| 160 | 160 | /* @var $form \Laminas\Form\FormInterface */ |
@@ -180,10 +180,10 @@ discard block |
||
| 180 | 180 | $form = $container->get($formIdentifier); |
| 181 | 181 | $form->setData(array_merge($postData, $filesData)); |
| 182 | 182 | if (!isset($form)) { |
| 183 | - throw new \RuntimeException('No form found for "' . $formIdentifier . '"'); |
|
| 183 | + throw new \RuntimeException('No form found for "'.$formIdentifier.'"'); |
|
| 184 | 184 | } |
| 185 | 185 | |
| 186 | - if('organizationLogo' == $formIdentifier){ |
|
| 186 | + if ('organizationLogo' == $formIdentifier) { |
|
| 187 | 187 | return $this->handleLogoUpload($form, $filesData); |
| 188 | 188 | } |
| 189 | 189 | |
@@ -49,17 +49,17 @@ discard block |
||
| 49 | 49 | /** |
| 50 | 50 | * Always enabled even if there are no active jobs |
| 51 | 51 | */ |
| 52 | - const PROFILE_ALWAYS_ENABLE = 'always'; |
|
| 52 | + const PROFILE_ALWAYS_ENABLE = 'always'; |
|
| 53 | 53 | |
| 54 | 54 | /** |
| 55 | 55 | * Hide if there are no jobs available |
| 56 | 56 | */ |
| 57 | - const PROFILE_ACTIVE_JOBS = 'active-jobs'; |
|
| 57 | + const PROFILE_ACTIVE_JOBS = 'active-jobs'; |
|
| 58 | 58 | |
| 59 | 59 | /** |
| 60 | 60 | * Always disabled profile |
| 61 | 61 | */ |
| 62 | - const PROFILE_DISABLED = 'disabled'; |
|
| 62 | + const PROFILE_DISABLED = 'disabled'; |
|
| 63 | 63 | |
| 64 | 64 | /** |
| 65 | 65 | * Event name of post construct event. |
@@ -317,7 +317,7 @@ discard block |
||
| 317 | 317 | */ |
| 318 | 318 | public function getParent($returnSelf = false) |
| 319 | 319 | { |
| 320 | - return $this->parent ? : ($returnSelf ? $this : null); |
|
| 320 | + return $this->parent ?: ($returnSelf ? $this : null); |
|
| 321 | 321 | } |
| 322 | 322 | |
| 323 | 323 | /** |
@@ -548,7 +548,7 @@ discard block |
||
| 548 | 548 | */ |
| 549 | 549 | public function getPermissionsResourceId() |
| 550 | 550 | { |
| 551 | - return 'organization:' . $this->getId(); |
|
| 551 | + return 'organization:'.$this->getId(); |
|
| 552 | 552 | } |
| 553 | 553 | |
| 554 | 554 | /** |
@@ -614,7 +614,7 @@ discard block |
||
| 614 | 614 | */ |
| 615 | 615 | public function getImages(): ?ImageSetInterface |
| 616 | 616 | { |
| 617 | - if(is_null($this->images)){ |
|
| 617 | + if (is_null($this->images)) { |
|
| 618 | 618 | $this->images = new ImageSet(); |
| 619 | 619 | } |
| 620 | 620 | return $this->images; |
@@ -32,9 +32,9 @@ |
||
| 32 | 32 | |
| 33 | 33 | public function getUri(): string |
| 34 | 34 | { |
| 35 | - $id = (string)$this->id; |
|
| 36 | - $name = (string)$this->name; |
|
| 37 | - return '/'.trim('file/Organizations.OrganizationImage/' . $id . "/" . urlencode($name), '/'); |
|
| 35 | + $id = (string) $this->id; |
|
| 36 | + $name = (string) $this->name; |
|
| 37 | + return '/'.trim('file/Organizations.OrganizationImage/'.$id."/".urlencode($name), '/'); |
|
| 38 | 38 | } |
| 39 | 39 | |
| 40 | 40 | } |
| 41 | 41 | \ No newline at end of file |