helmut /
forms
| Conditions | 4 |
| Paths | 4 |
| Total Lines | 31 |
| Code Lines | 17 |
| Lines | 0 |
| Ratio | 0 % |
| Tests | 0 |
| CRAP Score | 20 |
| Changes | 0 | ||
| 1 | <?php |
||
| 14 | public function onSubmitted($form) |
||
| 15 | { |
||
| 16 | $request = $form->getRequest(); |
||
| 17 | |||
| 18 | if ($request->get('feedback_trigger_'.$form->id)) { |
||
| 19 | |||
| 20 | $field = $form->field($request->get('name')); |
||
| 21 | |||
| 22 | if ( ! is_null($field)) { |
||
| 23 | |||
| 24 | $field->setFromRequest($request); |
||
| 25 | |||
| 26 | $field->runValidations(); |
||
| 27 | |||
| 28 | $response = [ |
||
| 29 | 'id' => $field->id, |
||
| 30 | 'valid' => $field->valid(), |
||
| 31 | 'error_id' => 'feedback_errors_'.$field->id, |
||
| 32 | 'error' => '', |
||
| 33 | ]; |
||
| 34 | |||
| 35 | if ( ! $field->valid()) { |
||
| 36 | $response['error'] = $form->renderFieldErrors($field); |
||
| 37 | } |
||
| 38 | |||
| 39 | header('Content-Type: application/json'); |
||
| 40 | echo json_encode($response); |
||
| 41 | exit(); |
||
|
|
|||
| 42 | } |
||
| 43 | } |
||
| 44 | } |
||
| 45 | |||
| 46 | } |
An exit expression should only be used in rare cases. For example, if you write a short command line script.
In most cases however, using an
exitexpression makes the code untestable and often causes incompatibilities with other libraries. Thus, unless you are absolutely sure it is required here, we recommend to refactor your code to avoid its usage.