| Total Complexity | 60 | 
| Total Lines | 240 | 
| Duplicated Lines | 0 % | 
| Changes | 1 | ||
| Bugs | 0 | Features | 0 | 
Complex classes like FakeRecordGeneratorTask often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes.
Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.
While breaking up the class, it is a good idea to analyze how other classes use FakeRecordGeneratorTask, and based on these observations, apply Extract Interface, too.
| 1 | <?php  | 
            ||
| 21 | class FakeRecordGeneratorTask extends BuildTask  | 
            ||
| 22 | { | 
            ||
| 23 | use BuildTaskTools;  | 
            ||
| 24 | |||
| 25 | protected $title = "Fake Record Generator";  | 
            ||
| 26 | protected $description = 'Generate fake records for a given class';  | 
            ||
| 27 | private static $segment = 'FakeRecordGeneratorTask';  | 
            ||
| 28 | |||
| 29 | public function run($request)  | 
            ||
| 159 | }  | 
            ||
| 160 | }  | 
            ||
| 161 | |||
| 162 | protected function getRandomValueFromType($type, $name, $record)  | 
            ||
| 163 |     { | 
            ||
| 164 |         $type = explode('(', $type); | 
            ||
| 165 |         switch ($type[0]) { | 
            ||
| 166 | case 'Varchar':  | 
            ||
| 167 | case DBVarchar::class:  | 
            ||
| 168 | $length = 50;  | 
            ||
| 169 |                 if (count($type) > 1) { | 
            ||
| 170 | $length = (int) $type[1];  | 
            ||
| 171 | }  | 
            ||
| 172 |                 if ($name == 'CountryCode' || $name == 'Nationality') { | 
            ||
| 173 | return FakeDataProvider::countryCode();  | 
            ||
| 174 |                 } elseif ($name == 'PostalCode' || $name == 'Postcode') { | 
            ||
| 175 | $addr = FakeDataProvider::address();  | 
            ||
| 176 | return $addr['Postcode'];  | 
            ||
| 177 |                 } elseif ($name == 'Locality' || $name == 'City') { | 
            ||
| 178 | $addr = FakeDataProvider::address();  | 
            ||
| 179 | return $addr['City'];  | 
            ||
| 180 |                 } elseif ($name == 'URLSegment' || $name == 'Slug') { | 
            ||
| 181 | return null; // let autogeneration happen  | 
            ||
| 182 | }  | 
            ||
| 183 | return FakeDataProvider::words(3, 7);  | 
            ||
| 184 | case 'Date':  | 
            ||
| 185 | case 'DateTime':  | 
            ||
| 186 | case DBDate::class:  | 
            ||
| 187 |                 return FakeDataProvider::date(strtotime('-1 year'), strtotime('+1 year')); | 
            ||
| 188 | case 'Boolean':  | 
            ||
| 189 | case DBBoolean::class:  | 
            ||
| 190 | return FakeDataProvider::boolean();  | 
            ||
| 191 | case 'Enum':  | 
            ||
| 192 | case 'NiceEnum':  | 
            ||
| 193 | case DBEnum::class:  | 
            ||
| 194 | /* @var $enum Enum */  | 
            ||
| 195 | $enum = $record->dbObject($name);  | 
            ||
| 196 | return FakeDataProvider::pick(array_values($enum->enumValues()));  | 
            ||
| 197 | case 'Int':  | 
            ||
| 198 | case DBInt::class:  | 
            ||
| 199 | return rand(1, 10);  | 
            ||
| 200 | case 'Currency':  | 
            ||
| 201 | case DBCurrency::class:  | 
            ||
| 202 | return FakeDataProvider::fprand(20, 100, 2);  | 
            ||
| 203 | case 'HTMLText':  | 
            ||
| 204 | case DBHTMLText::class:  | 
            ||
| 205 | return FakeDataProvider::paragraphs(3, 7);  | 
            ||
| 206 | case 'Text':  | 
            ||
| 207 | case DBText::class:  | 
            ||
| 208 | return FakeDataProvider::sentences(3, 7);  | 
            ||
| 209 | default:  | 
            ||
| 210 | $dbObject = $record->dbObject($name);  | 
            ||
| 211 |                 if ($dbObject && $dbObject->hasMethod('fillFake')) { | 
            ||
| 212 | return $dbObject->fillFake();  | 
            ||
| 213 | }  | 
            ||
| 214 | return null;  | 
            ||
| 215 | }  | 
            ||
| 216 | }  | 
            ||
| 217 | |||
| 218 | protected function createMembersFromApi($how_many)  | 
            ||
| 219 |     { | 
            ||
| 220 | $data = FakeDataProvider::randomUser(['result' => $how_many]);  | 
            ||
| 221 |         foreach ($data as $res) { | 
            ||
| 222 |             try { | 
            ||
| 223 | $rec = Member::create();  | 
            ||
| 224 | $rec->Gender = $res['gender'];  | 
            ||
| 225 | $rec->FirstName = ucwords($res['name']['first']);  | 
            ||
| 226 | $rec->Surname = ucwords($res['name']['last']);  | 
            ||
| 227 | $rec->Salutation = ucwords($res['name']['title']);  | 
            ||
| 228 | $rec->Address = $res['location']['street'];  | 
            ||
| 229 | $rec->Locality = $res['location']['city'];  | 
            ||
| 230 | $rec->PostalCode = $res['location']['postcode'];  | 
            ||
| 231 | $rec->BirthDate = $res['dob'];  | 
            ||
| 232 | $rec->Created = $res['registered'];  | 
            ||
| 233 | $rec->Phone = $res['phone'];  | 
            ||
| 234 | $rec->Cell = $res['cell'];  | 
            ||
| 235 | $rec->Nationality = $res['nat'];  | 
            ||
| 236 | $rec->Email = $res['email'];  | 
            ||
| 237 | |||
| 238 | $image_data = file_get_contents($res['picture']['large']);  | 
            ||
| 239 | $image = FakeDataProvider::storeFakeImage($image_data, basename($res['picture']['large']), 'Avatars');  | 
            ||
| 240 | $rec->AvatarID = $image->ID;  | 
            ||
| 241 | |||
| 242 | $id = $rec->write();  | 
            ||
| 243 | |||
| 244 | $rec->changePassword($res['login']['password']);  | 
            ||
| 245 | |||
| 246 |                 if ($rec->hasMethod('fillFake')) { | 
            ||
| 247 | $rec->fillFake();  | 
            ||
| 248 | }  | 
            ||
| 249 | $id = $rec->write();  | 
            ||
| 250 | |||
| 251 |                 $this->message("New record with id $id", "created"); | 
            ||
| 252 |             } catch (Exception $ex) { | 
            ||
| 253 | $this->message($ex->getMessage(), "error");  | 
            ||
| 254 | }  | 
            ||
| 255 | }  | 
            ||
| 256 | }  | 
            ||
| 257 | |||
| 258 | public function isEnabled()  | 
            ||
| 261 | }  | 
            ||
| 262 | }  | 
            ||
| 263 | 
The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g.
excluded_paths: ["lib/*"], you can move it to the dependency path list as follows:For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths