@@ -65,14 +65,14 @@ discard block |
||
65 | 65 | ->disableOriginalConstructor()->getMock(); |
66 | 66 | } |
67 | 67 | if (in_array('mailerPlugin', $mocks)) { |
68 | - $this->mailerPluginMock = $this->getMockBuilder('\Core\Controller\Plugin\Mailer') |
|
68 | + $this->mailerPluginMock = $this->getMockBuilder('\Core\Controller\Plugin\Mailer') |
|
69 | 69 | ->disableOriginalConstructor()->getMock(); |
70 | 70 | } |
71 | 71 | |
72 | 72 | if ($inject) { |
73 | 73 | foreach ($mocks as $key) { |
74 | - $setter = 'set' . $key; |
|
75 | - $this->target->$setter($this->{$key . 'Mock'}); |
|
74 | + $setter = 'set'.$key; |
|
75 | + $this->target->$setter($this->{$key.'Mock'}); |
|
76 | 76 | } |
77 | 77 | } |
78 | 78 | } |
@@ -107,9 +107,9 @@ discard block |
||
107 | 107 | { |
108 | 108 | $this->setupMocks($mockKey); |
109 | 109 | |
110 | - $setter = 'set' . ucfirst($mockKey); |
|
111 | - $getter = 'get' . ucfirst($mockKey); |
|
112 | - $value = $this->{$mockKey . 'Mock'}; |
|
110 | + $setter = 'set'.ucfirst($mockKey); |
|
111 | + $getter = 'get'.ucfirst($mockKey); |
|
112 | + $value = $this->{$mockKey.'Mock'}; |
|
113 | 113 | |
114 | 114 | $this->assertSame($this->target, $this->target->$setter($value), 'Fluent interface broken!'); |
115 | 115 | $this->assertSame($value, $this->target->$getter()); |
@@ -123,7 +123,7 @@ discard block |
||
123 | 123 | */ |
124 | 124 | public function testGetterThrowExceptionIfDependencyMissing($getterName) |
125 | 125 | { |
126 | - $getter = "get" . $getterName; |
|
126 | + $getter = "get".$getterName; |
|
127 | 127 | |
128 | 128 | $this->target->$getter(); |
129 | 129 | } |
@@ -80,11 +80,11 @@ |
||
80 | 80 | $services->expects($this->exactly(5)) |
81 | 81 | ->method('get') |
82 | 82 | ->will($this->returnValueMap(array( |
83 | - array('ValidatorManager', $validators), // get ha signature ($name, $usePeeringManagers = true) |
|
83 | + array('ValidatorManager', $validators), // get ha signature ($name, $usePeeringManagers = true) |
|
84 | 84 | array('translator', $translator), |
85 | 85 | array('repositories', $repositories), |
86 | 86 | array('Auth/UserTokenGenerator', $tokenGenerator), |
87 | - array('ControllerPluginManager',$pluginManager), |
|
87 | + array('ControllerPluginManager', $pluginManager), |
|
88 | 88 | ))); |
89 | 89 | |
90 | 90 | /* |
@@ -62,7 +62,7 @@ |
||
62 | 62 | ->method('get') |
63 | 63 | ->will($this->returnValueMap( |
64 | 64 | array( |
65 | - array('repositories',$repositories), |
|
65 | + array('repositories', $repositories), |
|
66 | 66 | array('AuthenticationService', $auth) |
67 | 67 | ) |
68 | 68 | )) |
@@ -34,8 +34,8 @@ |
||
34 | 34 | $container->expects($this->exactly(2)) |
35 | 35 | ->method('get') |
36 | 36 | ->willReturnMap([ |
37 | - ['Core/RepositoryService',$container], |
|
38 | - ['Organizations/Organization',$orgRepo] |
|
37 | + ['Core/RepositoryService', $container], |
|
38 | + ['Organizations/Organization', $orgRepo] |
|
39 | 39 | ]) |
40 | 40 | ; |
41 | 41 | $factory = new InviteEmployeeControllerFactory(); |
@@ -47,12 +47,12 @@ |
||
47 | 47 | $container->expects($this->any()) |
48 | 48 | ->method('get') |
49 | 49 | ->willReturnMap([ |
50 | - ['repositories',$container], |
|
51 | - ['Organizations/Organization',$repository], |
|
52 | - ['translator',$translator], |
|
53 | - ['Organizations\ImageFileCache\Manager',$imageFileCacheManager], |
|
54 | - ['Jobs/Job',$jobRepository], |
|
55 | - ['Jobs/JobboardSearchOptions',$options] |
|
50 | + ['repositories', $container], |
|
51 | + ['Organizations/Organization', $repository], |
|
52 | + ['translator', $translator], |
|
53 | + ['Organizations\ImageFileCache\Manager', $imageFileCacheManager], |
|
54 | + ['Jobs/Job', $jobRepository], |
|
55 | + ['Jobs/JobboardSearchOptions', $options] |
|
56 | 56 | ]) |
57 | 57 | ; |
58 | 58 |
@@ -60,7 +60,7 @@ |
||
60 | 60 | { |
61 | 61 | $user = $this->getMockForAbstractClass('\Auth\Entity\UserInterface'); |
62 | 62 | |
63 | - $makeArray = function ($options) use ($user) { |
|
63 | + $makeArray = function($options) use ($user) { |
|
64 | 64 | $options['user'] = $user; |
65 | 65 | return array($options, array_merge(array('user' => $user, 'token' => false, 'template' => 'organizations/mail/invite-employee'), $options)); |
66 | 66 | }; |
@@ -55,7 +55,7 @@ |
||
55 | 55 | |
56 | 56 | public function testGetUriWithAndWithoutFilename() |
57 | 57 | { |
58 | - $filename="My File.png"; |
|
58 | + $filename = "My File.png"; |
|
59 | 59 | $this->target->setName($filename); |
60 | 60 | $this->assertSame($this->target->getUri(), '/file/Organizations.OrganizationImage/'.$this->target->getId().'/'.urlencode($this->target->getName())); |
61 | 61 | $this->target->setName(null); |
@@ -115,7 +115,7 @@ discard block |
||
115 | 115 | $args = array($args); |
116 | 116 | } |
117 | 117 | |
118 | - for ($i=0,$c=count($func); $i<$c; $i+=1) { |
|
118 | + for ($i = 0, $c = count($func); $i < $c; $i += 1) { |
|
119 | 119 | $f = $func[$i]; |
120 | 120 | $a = $args[$i]; |
121 | 121 | |
@@ -144,7 +144,7 @@ discard block |
||
144 | 144 | $expected = array($expected); |
145 | 145 | } |
146 | 146 | |
147 | - for ($i=0, $c=count($func); $i<$c; $i+=1) { |
|
147 | + for ($i = 0, $c = count($func); $i < $c; $i += 1) { |
|
148 | 148 | $f = $func[$i]; |
149 | 149 | $a = $args[$i]; |
150 | 150 | $e = $expected[$i]; |
@@ -176,7 +176,7 @@ discard block |
||
176 | 176 | $perms = new Permissions(); |
177 | 177 | $contact = new OrganizationContact(); |
178 | 178 | $image = new OrganizationImage(); |
179 | - $externalId='myReference'; |
|
179 | + $externalId = 'myReference'; |
|
180 | 180 | $template = new Template(); |
181 | 181 | $template->setLabelBenefits('These are your Benefits'); |
182 | 182 | $workflowSettings = new WorkflowSettings(); |
@@ -61,74 +61,74 @@ |
||
61 | 61 | $parent = new Organization(); |
62 | 62 | $permissions = $this->createMock(Permissions::class); |
63 | 63 | return [ |
64 | - [ 'externalId', '1234' ], |
|
65 | - [ 'name', [ 'value'=> '', 'ignore_setter' => true] ], |
|
66 | - [ 'name', [ 'pre' => 'setupGetName', 'value' => 'Test', 'ignore_setter' => true] ], |
|
67 | - [ 'organizationName' , [ |
|
68 | - '@value' => [ '\Organizations\Entity\OrganizationName', [ 'Test' ] ], |
|
64 | + ['externalId', '1234'], |
|
65 | + ['name', ['value'=> '', 'ignore_setter' => true]], |
|
66 | + ['name', ['pre' => 'setupGetName', 'value' => 'Test', 'ignore_setter' => true]], |
|
67 | + ['organizationName', [ |
|
68 | + '@value' => ['\Organizations\Entity\OrganizationName', ['Test']], |
|
69 | 69 | 'post' => 'organizationNamePostAsserts' |
70 | 70 | ]], |
71 | - [ 'organizationName', [ |
|
71 | + ['organizationName', [ |
|
72 | 72 | 'pre' => 'setupOrganizationName', |
73 | - '@value' => [ '\Organizations\Entity\OrganizationName', [ 'TestNew' ] ], |
|
73 | + '@value' => ['\Organizations\Entity\OrganizationName', ['TestNew']], |
|
74 | 74 | 'post' => 'assertOrganizationNameCounter' |
75 | 75 | ]], |
76 | - [ 'permissions',[ |
|
76 | + ['permissions', [ |
|
77 | 77 | 'value' => $permissions |
78 | 78 | ]], |
79 | - [ 'image',[ |
|
79 | + ['image', [ |
|
80 | 80 | 'value' => new OrganizationImage() |
81 | 81 | ]], |
82 | - [ 'images',[ |
|
82 | + ['images', [ |
|
83 | 83 | 'value' => new ImageSet(), |
84 | 84 | ]], |
85 | - [ 'isDraft',[ |
|
85 | + ['isDraft', [ |
|
86 | 86 | 'default' => false, |
87 | 87 | 'value' => true, |
88 | 88 | 'getter_method' => 'isDraft' |
89 | 89 | ]], |
90 | - [ 'contact',[ |
|
90 | + ['contact', [ |
|
91 | 91 | 'value' => new OrganizationContact() |
92 | 92 | ]], |
93 | - [ 'description',[ |
|
93 | + ['description', [ |
|
94 | 94 | 'default' => null, |
95 | 95 | 'value' => 'Some Description' |
96 | 96 | ]], |
97 | - [ 'parent',[ |
|
97 | + ['parent', [ |
|
98 | 98 | 'default' => null, |
99 | 99 | 'value' => $parent, |
100 | 100 | ]], |
101 | - [ 'hiringOrganizations',[ |
|
101 | + ['hiringOrganizations', [ |
|
102 | 102 | 'default' => null, |
103 | 103 | ]], |
104 | 104 | |
105 | - [ 'employees',[ |
|
105 | + ['employees', [ |
|
106 | 106 | '@default' => ArrayCollection::class, |
107 | 107 | 'value' => new ArrayCollection() |
108 | 108 | ]], |
109 | 109 | |
110 | - [ 'jobs',[ |
|
110 | + ['jobs', [ |
|
111 | 111 | 'default' => null, |
112 | 112 | ]], |
113 | 113 | |
114 | - [ 'user',[ |
|
114 | + ['user', [ |
|
115 | 115 | 'value' => new User() |
116 | 116 | ]], |
117 | 117 | |
118 | - [ 'template',[ |
|
118 | + ['template', [ |
|
119 | 119 | 'value' => new Template(), |
120 | 120 | ]], |
121 | 121 | |
122 | - [ 'workflowSettings',[ |
|
122 | + ['workflowSettings', [ |
|
123 | 123 | 'value' => new WorkflowSettings(), |
124 | 124 | ]], |
125 | 125 | |
126 | - [ 'profileSetting',[ |
|
126 | + ['profileSetting', [ |
|
127 | 127 | 'value' => Organization::PROFILE_ALWAYS_ENABLE, |
128 | 128 | 'default' => null, |
129 | 129 | ]], |
130 | 130 | |
131 | - [ 'hydrator', [ |
|
131 | + ['hydrator', [ |
|
132 | 132 | '@default' => EntityHydrator::class |
133 | 133 | ]] |
134 | 134 |