| @@ -96,7 +96,7 @@ | ||
| 96 | 96 |      { | 
| 97 | 97 | $parsed = array_merge([ | 
| 98 | 98 | 'data-type' => 'numeric', | 
| 99 | - 'info-type' => empty($meta['type'])? 'generic' : $meta['type'], | |
| 99 | + 'info-type' => empty($meta['type']) ? 'generic' : $meta['type'], | |
| 100 | 100 | 'strict' => true, | 
| 101 | 101 | ], $meta); | 
| 102 | 102 | |
| @@ -213,7 +213,9 @@ | ||
| 213 | 213 | */ | 
| 214 | 214 | protected function toDefault($value, array $meta = []) | 
| 215 | 215 |      { | 
| 216 | - if ( ! empty($value)) return $value; | |
| 216 | +        if ( ! empty($value)) { | |
| 217 | + return $value; | |
| 218 | + } | |
| 217 | 219 | |
| 218 | 220 |          if ( ! empty($meta['def'])) { | 
| 219 | 221 | $value = $meta['def']; | 
| @@ -85,7 +85,7 @@ discard block | ||
| 85 | 85 | * | 
| 86 | 86 | * @param mixed $value | 
| 87 | 87 | * @param array $data | 
| 88 | - * @return mixed | |
| 88 | + * @return integer | |
| 89 | 89 | */ | 
| 90 | 90 | protected function mutateDetailCompanyDocumentType( | 
| 91 | 91 | $value, | 
| @@ -113,7 +113,7 @@ discard block | ||
| 113 | 113 | * | 
| 114 | 114 | * @param mixed $value | 
| 115 | 115 | * @param array $data | 
| 116 | - * @return mixed | |
| 116 | + * @return integer | |
| 117 | 117 | */ | 
| 118 | 118 | protected function mutateDetailDocumentType( | 
| 119 | 119 | $value, | 
| @@ -133,7 +133,9 @@ discard block | ||
| 133 | 133 | $value, | 
| 134 | 134 | array $data = [] | 
| 135 | 135 |      ) { | 
| 136 | - if (empty($data['account'])) return $value; | |
| 136 | +        if (empty($data['account'])) { | |
| 137 | + return $value; | |
| 138 | + } | |
| 137 | 139 | |
| 138 | 140 | return $value ?: $data['account']; | 
| 139 | 141 | } | 
| @@ -149,7 +151,9 @@ discard block | ||
| 149 | 151 | $value, | 
| 150 | 152 | array $data = [] | 
| 151 | 153 |      ) { | 
| 152 | - if (empty($data['account'])) return $value; | |
| 154 | +        if (empty($data['account'])) { | |
| 155 | + return $value; | |
| 156 | + } | |
| 153 | 157 | |
| 154 | 158 | return $value ?: $data['accountDv']; | 
| 155 | 159 | } | 
| @@ -165,7 +169,9 @@ discard block | ||
| 165 | 169 | $value, | 
| 166 | 170 | array $data = [] | 
| 167 | 171 |      ) { | 
| 168 | - if (empty($data['branch'])) return $value; | |
| 172 | +        if (empty($data['branch'])) { | |
| 173 | + return $value; | |
| 174 | + } | |
| 169 | 175 | |
| 170 | 176 | return $value ?: $data['branch']; | 
| 171 | 177 | } | 
| @@ -181,7 +187,9 @@ discard block | ||
| 181 | 187 | $value, | 
| 182 | 188 | array $data = [] | 
| 183 | 189 |      ) { | 
| 184 | - if (empty($data['branch'])) return $value; | |
| 190 | +        if (empty($data['branch'])) { | |
| 191 | + return $value; | |
| 192 | + } | |
| 185 | 193 | |
| 186 | 194 | return $value ?: $data['branchDv']; | 
| 187 | 195 | } | 
| @@ -35,7 +35,7 @@ | ||
| 35 | 35 | * | 
| 36 | 36 | * @param mixed $value | 
| 37 | 37 | * @param array $data | 
| 38 | - * @return mixed | |
| 38 | + * @return integer | |
| 39 | 39 | */ | 
| 40 | 40 | protected function mutateDetailLateInterestFlag( | 
| 41 | 41 | $value, | 
| @@ -21,7 +21,7 @@ discard block | ||
| 21 | 21 | * | 
| 22 | 22 | * @param mixed $value | 
| 23 | 23 | * @param array $data | 
| 24 | - * @return mixed | |
| 24 | + * @return integer | |
| 25 | 25 | */ | 
| 26 | 26 | protected function mutateDetailCompanyDocumentType( | 
| 27 | 27 | $value, | 
| @@ -49,7 +49,7 @@ discard block | ||
| 49 | 49 | * | 
| 50 | 50 | * @param mixed $value | 
| 51 | 51 | * @param array $data | 
| 52 | - * @return mixed | |
| 52 | + * @return integer | |
| 53 | 53 | */ | 
| 54 | 54 | protected function mutateDetailLateInterestFlag( | 
| 55 | 55 | $value, | 
| @@ -63,7 +63,7 @@ discard block | ||
| 63 | 63 | * | 
| 64 | 64 | * @param mixed $value | 
| 65 | 65 | * @param array $data | 
| 66 | - * @return mixed | |
| 66 | + * @return integer | |
| 67 | 67 | */ | 
| 68 | 68 | protected function mutateDetailDocumentType( | 
| 69 | 69 | $value, | 
| @@ -20,7 +20,7 @@ discard block | ||
| 20 | 20 | * Mutates an account on detail. | 
| 21 | 21 | * | 
| 22 | 22 | * @param mixed $value | 
| 23 | - * @return mixed | |
| 23 | + * @return string | |
| 24 | 24 | */ | 
| 25 | 25 | protected function mutateDetailAccount($value) | 
| 26 | 26 |      { | 
| @@ -49,7 +49,7 @@ discard block | ||
| 49 | 49 | * @param mixed $value | 
| 50 | 50 | * @param array $data | 
| 51 | 51 | * @param array $meta | 
| 52 | - * @return mixed | |
| 52 | + * @return string | |
| 53 | 53 | */ | 
| 54 | 54 | protected function mutateDetailBranch( | 
| 55 | 55 | $value, | 
| @@ -80,7 +80,7 @@ discard block | ||
| 80 | 80 | * | 
| 81 | 81 | * @param mixed $value | 
| 82 | 82 | * @param array $data | 
| 83 | - * @return mixed | |
| 83 | + * @return integer | |
| 84 | 84 | */ | 
| 85 | 85 | protected function mutateDetailCompanyDocumentType( | 
| 86 | 86 | $value, | 
| @@ -108,7 +108,7 @@ discard block | ||
| 108 | 108 | * | 
| 109 | 109 | * @param mixed $value | 
| 110 | 110 | * @param array $data | 
| 111 | - * @return mixed | |
| 111 | + * @return integer | |
| 112 | 112 | */ | 
| 113 | 113 | protected function mutateDetailDocumentType( | 
| 114 | 114 | $value, | 
| @@ -121,7 +121,7 @@ discard block | ||
| 121 | 121 | * Mutates a guarantee contract on detail. | 
| 122 | 122 | * | 
| 123 | 123 | * @param mixed $value | 
| 124 | - * @return mixed | |
| 124 | + * @return string | |
| 125 | 125 | */ | 
| 126 | 126 | protected function mutateDetailGuaranteeContract($value) | 
| 127 | 127 |      { | 
| @@ -150,7 +150,7 @@ discard block | ||
| 150 | 150 | * @param mixed $value | 
| 151 | 151 | * @param array $data | 
| 152 | 152 | * @param array $meta | 
| 153 | - * @return mixed | |
| 153 | + * @return string | |
| 154 | 154 | */ | 
| 155 | 155 | protected function mutateDetailReceiveBranch( | 
| 156 | 156 | $value, | 
| @@ -180,7 +180,7 @@ discard block | ||
| 180 | 180 | * Mutates a branch on header. | 
| 181 | 181 | * | 
| 182 | 182 | * @param mixed $value | 
| 183 | - * @return mixed | |
| 183 | + * @return string | |
| 184 | 184 | */ | 
| 185 | 185 | protected function mutateHeaderBranch($value) | 
| 186 | 186 |      { | 
| @@ -38,7 +38,9 @@ discard block | ||
| 38 | 38 | $value, | 
| 39 | 39 | array $data = [] | 
| 40 | 40 |      ) { | 
| 41 | - if (empty($data['accountDv'])) return $value; | |
| 41 | +        if (empty($data['accountDv'])) { | |
| 42 | + return $value; | |
| 43 | + } | |
| 42 | 44 | |
| 43 | 45 | return $value ?: $data['accountDv']; | 
| 44 | 46 | } | 
| @@ -139,7 +141,9 @@ discard block | ||
| 139 | 141 | $value, | 
| 140 | 142 | array $data = [] | 
| 141 | 143 |      ) { | 
| 142 | - if (empty($data['guaranteeContract'])) return $value; | |
| 144 | +        if (empty($data['guaranteeContract'])) { | |
| 145 | + return $value; | |
| 146 | + } | |
| 143 | 147 | |
| 144 | 148 | return $value ?: substr($data['guaranteeContract'], -1); | 
| 145 | 149 | } | 
| @@ -198,7 +202,9 @@ discard block | ||
| 198 | 202 | $value, | 
| 199 | 203 | array $data = [] | 
| 200 | 204 |      ) { | 
| 201 | - if (empty($data['branch'])) return $value; | |
| 205 | +        if (empty($data['branch'])) { | |
| 206 | + return $value; | |
| 207 | + } | |
| 202 | 208 | |
| 203 | 209 | return empty($data['branchDv']) ? $value : $data['branchDv']; | 
| 204 | 210 | } | 
| @@ -214,7 +220,9 @@ discard block | ||
| 214 | 220 | $value, | 
| 215 | 221 | array $data = [] | 
| 216 | 222 |      ) { | 
| 217 | - if (empty($data['account'])) return $value; | |
| 223 | +        if (empty($data['account'])) { | |
| 224 | + return $value; | |
| 225 | + } | |
| 218 | 226 | |
| 219 | 227 | return $value ?: $data['account']; | 
| 220 | 228 | } | 
| @@ -230,7 +238,9 @@ discard block | ||
| 230 | 238 | $value, | 
| 231 | 239 | array $data = [] | 
| 232 | 240 |      ) { | 
| 233 | - if (empty($data['account'])) return $value; | |
| 241 | +        if (empty($data['account'])) { | |
| 242 | + return $value; | |
| 243 | + } | |
| 234 | 244 | |
| 235 | 245 | return $value ?: $data['accountDv']; | 
| 236 | 246 | } | 
| @@ -166,7 +166,9 @@ discard block | ||
| 166 | 166 | $value, | 
| 167 | 167 | array $data = [] | 
| 168 | 168 |      ) { | 
| 169 | - if (empty($data['account'])) return $value; | |
| 169 | +        if (empty($data['account'])) { | |
| 170 | + return $value; | |
| 171 | + } | |
| 170 | 172 | |
| 171 | 173 | return $value ?: $data['account']; | 
| 172 | 174 | } | 
| @@ -182,7 +184,9 @@ discard block | ||
| 182 | 184 | $value, | 
| 183 | 185 | array $data = [] | 
| 184 | 186 |      ) { | 
| 185 | - if (empty($data['accountDv'])) return $value; | |
| 187 | +        if (empty($data['accountDv'])) { | |
| 188 | + return $value; | |
| 189 | + } | |
| 186 | 190 | |
| 187 | 191 | return $value ?: $data['accountDv']; | 
| 188 | 192 | } | 
| @@ -85,7 +85,7 @@ discard block | ||
| 85 | 85 | * | 
| 86 | 86 | * @param mixed $value | 
| 87 | 87 | * @param array $data | 
| 88 | - * @return mixed | |
| 88 | + * @return integer | |
| 89 | 89 | */ | 
| 90 | 90 | protected function mutateDetailCompanyDocumentType( | 
| 91 | 91 | $value, | 
| @@ -113,7 +113,7 @@ discard block | ||
| 113 | 113 | * | 
| 114 | 114 | * @param mixed $value | 
| 115 | 115 | * @param array $data | 
| 116 | - * @return mixed | |
| 116 | + * @return integer | |
| 117 | 117 | */ | 
| 118 | 118 | protected function mutateDetailDocumentType( | 
| 119 | 119 | $value, | 
| @@ -85,7 +85,7 @@ discard block | ||
| 85 | 85 | * | 
| 86 | 86 | * @param mixed $value | 
| 87 | 87 | * @param array $data | 
| 88 | - * @return mixed | |
| 88 | + * @return integer | |
| 89 | 89 | */ | 
| 90 | 90 | protected function mutateDetailCompanyDocumentType( | 
| 91 | 91 | $value, | 
| @@ -113,7 +113,7 @@ discard block | ||
| 113 | 113 | * | 
| 114 | 114 | * @param mixed $value | 
| 115 | 115 | * @param array $data | 
| 116 | - * @return mixed | |
| 116 | + * @return integer | |
| 117 | 117 | */ | 
| 118 | 118 | protected function mutateDetailDocumentType( | 
| 119 | 119 | $value, | 
| @@ -70,9 +70,13 @@ discard block | ||
| 70 | 70 | $value, | 
| 71 | 71 | array $data = [] | 
| 72 | 72 |      ) { | 
| 73 | - if ($data['occurrenceCode'] == 11) return Caixa::INSTRUCTION_PROTEST; | |
| 73 | +        if ($data['occurrenceCode'] == 11) { | |
| 74 | + return Caixa::INSTRUCTION_PROTEST; | |
| 75 | + } | |
| 74 | 76 | |
| 75 | - if ($data['occurrenceCode'] == 12) return Caixa::INSTRUCTION_DEVOLUTION; | |
| 77 | +        if ($data['occurrenceCode'] == 12) { | |
| 78 | + return Caixa::INSTRUCTION_DEVOLUTION; | |
| 79 | + } | |
| 76 | 80 | |
| 77 | 81 | return $value; | 
| 78 | 82 | } | 
| @@ -90,7 +94,9 @@ discard block | ||
| 90 | 94 |      ) { | 
| 91 | 95 | $returnValue = $value == '' && $data['occurrenceCode'] == 9; | 
| 92 | 96 | |
| 93 | - if ($returnValue) return $value; | |
| 97 | +        if ($returnValue) { | |
| 98 | + return $value; | |
| 99 | + } | |
| 94 | 100 | |
| 95 | 101 |          return $value ?: $data['expiration']->add(new \DateInterval('P1D')); | 
| 96 | 102 | } | 
| @@ -3,7 +3,6 @@ | ||
| 3 | 3 | namespace SmartCNAB\Services\Returning; | 
| 4 | 4 | |
| 5 | 5 | use StdClass; | 
| 6 | - | |
| 7 | 6 | use SmartCNAB\Support\Bank; | 
| 8 | 7 | use SmartCNAB\Support\Picture; | 
| 9 | 8 | use SmartCNAB\Support\File\Returning as SupportReturning; | 
| @@ -197,9 +197,13 @@ | ||
| 197 | 197 | */ | 
| 198 | 198 | public function motives($occurrenceCode = null) | 
| 199 | 199 |      { | 
| 200 | - if ($occurrenceCode == '99') return []; | |
| 200 | +        if ($occurrenceCode == '99') { | |
| 201 | + return []; | |
| 202 | + } | |
| 201 | 203 | |
| 202 | - if ( ! $occurrenceCode) return $this->rejectionCodes(); | |
| 204 | +        if ( ! $occurrenceCode) { | |
| 205 | + return $this->rejectionCodes(); | |
| 206 | + } | |
| 203 | 207 | |
| 204 | 208 | $occurrenceCode = str_pad($occurrenceCode, 2, 0, STR_PAD_LEFT); | 
| 205 | 209 | |