Conditions | 10 |
Paths | 32 |
Total Lines | 93 |
Code Lines | 64 |
Lines | 0 |
Ratio | 0 % |
Changes | 0 |
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
1 | <?php |
||
106 | public function showVersionOne(JobApplication $application) |
||
107 | { |
||
108 | $response_poster = false; |
||
109 | $show_review = true; |
||
110 | $jobPoster = $application->job_poster; |
||
111 | |||
112 | $essential_criteria = $jobPoster->criteria->filter(function ($value, $key) { |
||
113 | return $value->criteria_type->name == 'essential' |
||
114 | && $value->skill->skill_type->name == 'hard'; |
||
115 | }); |
||
116 | $asset_criteria = $jobPoster->criteria->filter(function ($value, $key) { |
||
117 | return $value->criteria_type->name == 'asset' |
||
118 | && $value->skill->skill_type->name == 'hard'; |
||
119 | }); |
||
120 | |||
121 | // Display slightly different views on different portals. |
||
122 | $view = WhichPortal::isManagerPortal() || WhichPortal::isHrPortal() ? |
||
123 | 'manager/application_post' : 'applicant/application_preview'; |
||
124 | |||
125 | $application_view = 'common/application/view/view_layout'; |
||
126 | |||
127 | if (WhichPortal::isManagerPortal() || WhichPortal::isHrPortal()) { |
||
128 | // Load things required for review component. |
||
129 | $application->load(['veteran_status', 'citizenship_declaration', 'application_review', 'applicant.user']); |
||
130 | } |
||
131 | |||
132 | |||
133 | // If the application status is draft then get data through the applicant model. |
||
134 | // Else, grab the data from the application itself. |
||
135 | if ($application->isDraft()) { |
||
136 | $source = $application->applicant; |
||
137 | $show_review = false; |
||
138 | } else { |
||
139 | $source = $application; |
||
140 | } |
||
141 | |||
142 | $degrees = $source->degrees; |
||
143 | $courses = $source->courses; |
||
144 | $work_experiences = $source->work_experiences; |
||
145 | $skill_declarations = $source->skill_declarations; |
||
146 | $references = $source->references; |
||
147 | $work_samples = $source->work_samples; |
||
148 | |||
149 | $custom_breadcrumbs = [ |
||
150 | 'home' => route('home'), |
||
151 | 'applications' => route('applications.index'), |
||
152 | 'application' => '', |
||
153 | ]; |
||
154 | |||
155 | if (WhichPortal::isManagerPortal() || WhichPortal::isHrPortal()) { |
||
156 | $custom_breadcrumbs = [ |
||
157 | 'home' => route('home'), |
||
158 | 'jobs' => route(WhichPortal::prefixRoute('jobs.index')), |
||
159 | $jobPoster->title => route(WhichPortal::prefixRoute('jobs.summary'), $jobPoster), |
||
160 | 'applications' => route(WhichPortal::prefixRoute('jobs.applications'), $jobPoster), |
||
161 | 'application' => '', |
||
162 | ]; |
||
163 | } |
||
164 | |||
165 | return view( |
||
166 | $view, |
||
167 | [ |
||
168 | 'application_view_template' => $application_view, |
||
169 | // Localized strings. |
||
170 | 'post' => Lang::get('manager/application_post'), // Change text |
||
171 | 'is_manager_view' => WhichPortal::isManagerPortal(), |
||
172 | 'is_hr_portal' => WhichPortal::isHrPortal(), |
||
173 | // Application Template Data. |
||
174 | 'application_template' => Lang::get('applicant/application_template'), |
||
175 | 'citizenship_declaration_template' => Lang::get('common/citizenship_declaration'), |
||
176 | 'veteran_status_template' => Lang::get('common/veteran_status'), |
||
177 | // Job Data. |
||
178 | 'job' => $jobPoster, |
||
179 | // Skills Data. |
||
180 | 'skills' => Skill::all(), |
||
181 | 'skill_template' => Lang::get('common/skills'), |
||
182 | 'reference_template' => Lang::get('common/references'), |
||
183 | 'sample_template' => Lang::get('common/work_samples'), |
||
184 | 'essential_criteria' => $essential_criteria, |
||
185 | 'asset_criteria' => $asset_criteria, |
||
186 | // Applicant Data. |
||
187 | 'applicant' => $application->applicant, |
||
188 | 'job_application' => $application, |
||
189 | 'degrees' => $degrees, |
||
190 | 'courses' => $courses, |
||
191 | 'work_experiences' => $work_experiences, |
||
192 | 'skill_declarations' => $skill_declarations, |
||
193 | 'references' => $references, |
||
194 | 'work_samples' => $work_samples, |
||
195 | 'review_statuses' => ReviewStatus::all(), |
||
196 | 'custom_breadcrumbs' => $custom_breadcrumbs, |
||
197 | 'response_poster' => $response_poster, |
||
198 | 'show_review' => $show_review, |
||
199 | ] |
||
225 |