cs: Enable risky rule random_api_migration.

This commit is contained in:
Pol Dellaiera 2021-11-30 11:55:02 +01:00
parent 91d12c4a96
commit a43784c3a1
No known key found for this signature in database
GPG Key ID: D476DFE9C67467CA
13 changed files with 27 additions and 27 deletions

View File

@ -62,7 +62,7 @@ $riskyRules = [
'php_unit_expectation' => false, 'php_unit_expectation' => false,
'php_unit_mock' => false, 'php_unit_mock' => false,
'php_unit_namespaced' => false, 'php_unit_namespaced' => false,
'random_api_migration' => false, // 'random_api_migration' => false,
// 'static_lambda' => false, // 'static_lambda' => false,
'php_unit_construct' => false, 'php_unit_construct' => false,
// 'psr_autoloading' => false, // 'psr_autoloading' => false,

View File

@ -48,7 +48,7 @@ class LoadActivity extends AbstractFixture implements OrderedFixtureInterface
->findAll(); ->findAll();
foreach ($persons as $person) { foreach ($persons as $person) {
$activityNbr = rand(0, 3); $activityNbr = mt_rand(0, 3);
for ($i = 0; $i < $activityNbr; ++$i) { for ($i = 0; $i < $activityNbr; ++$i) {
$activity = $this->newRandomActivity($person); $activity = $this->newRandomActivity($person);
@ -73,7 +73,7 @@ class LoadActivity extends AbstractFixture implements OrderedFixtureInterface
// ->setAttendee($this->faker->boolean()) // ->setAttendee($this->faker->boolean())
for ($i = 0; rand(0, 4) > $i; ++$i) { for ($i = 0; mt_rand(0, 4) > $i; ++$i) {
$reason = $this->getRandomActivityReason(); $reason = $this->getRandomActivityReason();
if (null !== $reason) { if (null !== $reason) {

View File

@ -102,7 +102,7 @@ class LoadOption extends AbstractFixture implements OrderedFixtureInterface
$manager->persist($parent); $manager->persist($parent);
//Load children //Load children
$expected_nb_children = rand(10, 50); $expected_nb_children = mt_rand(10, 50);
for ($i = 0; $i < $expected_nb_children; ++$i) { for ($i = 0; $i < $expected_nb_children; ++$i) {
$companyName = $this->fakerFr->company; $companyName = $this->fakerFr->company;
@ -142,7 +142,7 @@ class LoadOption extends AbstractFixture implements OrderedFixtureInterface
$manager->persist($parent); $manager->persist($parent);
//Load children //Load children
$expected_nb_children = rand(10, 50); $expected_nb_children = mt_rand(10, 50);
for ($i = 0; $i < $expected_nb_children; ++$i) { for ($i = 0; $i < $expected_nb_children; ++$i) {
$manager->persist($this->createChildOption($parent, [ $manager->persist($this->createChildOption($parent, [

View File

@ -37,7 +37,7 @@ class ChoicesListType extends AbstractType
$formData = $form->getData(); $formData = $form->getData();
if (null === $formData['slug']) { if (null === $formData['slug']) {
$slug = uniqid(rand(), true); $slug = uniqid(mt_rand(), true);
$data['slug'] = $slug; $data['slug'] = $slug;
$event->setData($data); $event->setData($data);

View File

@ -78,7 +78,7 @@ class DocGeneratorTemplateController extends AbstractController
$fileContent = fopen($tmpfname2, 'r'); // the generated file content $fileContent = fopen($tmpfname2, 'r'); // the generated file content
$genDocName = 'doc_' . sprintf('%010d', rand()) . '.docx'; $genDocName = 'doc_' . sprintf('%010d', mt_rand()) . '.docx';
$getUrlGen = $tempUrlGenerator->generate( $getUrlGen = $tempUrlGenerator->generate(
'PUT', 'PUT',

View File

@ -40,7 +40,7 @@ class LoadParticipation extends AbstractFixture implements OrderedFixtureInterfa
for ($i = 0; $i < $expectedNumber; ++$i) { for ($i = 0; $i < $expectedNumber; ++$i) {
$event = (new Event()) $event = (new Event())
->setDate($this->faker->dateTimeBetween('-2 years', '+6 months')) ->setDate($this->faker->dateTimeBetween('-2 years', '+6 months'))
->setName($this->faker->words(rand(2, 4), true)) ->setName($this->faker->words(mt_rand(2, 4), true))
->setType($this->getReference(LoadEventTypes::$refs[array_rand(LoadEventTypes::$refs)])) ->setType($this->getReference(LoadEventTypes::$refs[array_rand(LoadEventTypes::$refs)]))
->setCenter($center) ->setCenter($center)
->setCircle( ->setCircle(
@ -72,7 +72,7 @@ class LoadParticipation extends AbstractFixture implements OrderedFixtureInterfa
/** @var \Chill\PersonBundle\Entity\Person $person */ /** @var \Chill\PersonBundle\Entity\Person $person */
foreach ($people as $person) { foreach ($people as $person) {
$nb = rand(0, 3); $nb = mt_rand(0, 3);
for ($i = 0; $i < $nb; ++$i) { for ($i = 0; $i < $nb; ++$i) {
$event = $events[array_rand($events)]; $event = $events[array_rand($events)];

View File

@ -402,7 +402,7 @@ class ExportController extends AbstractController
'alias' => $alias, 'alias' => $alias,
]; ];
unset($parameters['_token']); unset($parameters['_token']);
$key = md5(uniqid(rand(), false)); $key = md5(uniqid(mt_rand(), false));
$this->redis->setEx($key, 3600, serialize($parameters)); $this->redis->setEx($key, 3600, serialize($parameters));

View File

@ -67,7 +67,7 @@ class LoadAddressReferences extends AbstractFixture implements ContainerAwareInt
$ar->setRefId($this->faker->numerify('ref-id-######')); $ar->setRefId($this->faker->numerify('ref-id-######'));
$ar->setStreet($this->faker->streetName); $ar->setStreet($this->faker->streetName);
$ar->setStreetNumber(rand(0, 199)); $ar->setStreetNumber(mt_rand(0, 199));
$ar->setPoint($this->getRandomPoint()); $ar->setPoint($this->getRandomPoint());
$ar->setPostcode($this->getReference( $ar->setPostcode($this->getReference(
LoadPostalCodes::$refs[array_rand(LoadPostalCodes::$refs)] LoadPostalCodes::$refs[array_rand(LoadPostalCodes::$refs)]
@ -87,8 +87,8 @@ class LoadAddressReferences extends AbstractFixture implements ContainerAwareInt
{ {
$lonBrussels = 4.35243; $lonBrussels = 4.35243;
$latBrussels = 50.84676; $latBrussels = 50.84676;
$lon = $lonBrussels + 0.01 * rand(-5, 5); $lon = $lonBrussels + 0.01 * mt_rand(-5, 5);
$lat = $latBrussels + 0.01 * rand(-5, 5); $lat = $latBrussels + 0.01 * mt_rand(-5, 5);
return Point::fromLonLat($lon, $lat); return Point::fromLonLat($lon, $lat);
} }

View File

@ -121,12 +121,12 @@ class LoadCustomFields extends AbstractFixture implements
// select a set of people and add data // select a set of people and add data
foreach ($personIds as $id) { foreach ($personIds as $id) {
// add info on 1 person on 2 // add info on 1 person on 2
if (rand(0, 1) === 1) { if (mt_rand(0, 1) === 1) {
/** @var Person $person */ /** @var Person $person */
$person = $manager->getRepository(Person::class)->find($id); $person = $manager->getRepository(Person::class)->find($id);
$person->setCFData([ $person->setCFData([
'remarques' => $this->createCustomFieldText() 'remarques' => $this->createCustomFieldText()
->serialize($faker->text(rand(150, 250)), $this->customFieldText), ->serialize($faker->text(mt_rand(150, 250)), $this->customFieldText),
'document-d-identite' => $this->createCustomFieldChoice() 'document-d-identite' => $this->createCustomFieldChoice()
->serialize([$choices[array_rand($choices)]], $this->customFieldChoice), ->serialize([$choices[array_rand($choices)]], $this->customFieldChoice),
]); ]);

View File

@ -491,10 +491,10 @@ class LoadPeople extends AbstractFixture implements OrderedFixtureInterface, Con
return (new Address()) return (new Address())
->setStreetAddress1($this->faker->streetAddress) ->setStreetAddress1($this->faker->streetAddress)
->setStreetAddress2( ->setStreetAddress2(
rand(0, 9) > 5 ? $this->faker->streetAddress : '' mt_rand(0, 9) > 5 ? $this->faker->streetAddress : ''
) )
->setPoint( ->setPoint(
rand(0, 9) > 5 ? $this->getRandomPoint() : null mt_rand(0, 9) > 5 ? $this->getRandomPoint() : null
) )
->setPostcode($this->getReference( ->setPostcode($this->getReference(
LoadPostalCodes::$refs[array_rand(LoadPostalCodes::$refs)] LoadPostalCodes::$refs[array_rand(LoadPostalCodes::$refs)]
@ -511,8 +511,8 @@ class LoadPeople extends AbstractFixture implements OrderedFixtureInterface, Con
{ {
$lonBrussels = 4.35243; $lonBrussels = 4.35243;
$latBrussels = 50.84676; $latBrussels = 50.84676;
$lon = $lonBrussels + 0.01 * rand(-5, 5); $lon = $lonBrussels + 0.01 * mt_rand(-5, 5);
$lat = $latBrussels + 0.01 * rand(-5, 5); $lat = $latBrussels + 0.01 * mt_rand(-5, 5);
return Point::fromLonLat($lon, $lat); return Point::fromLonLat($lon, $lat);
} }

View File

@ -65,15 +65,15 @@ class LoadCustomField extends AbstractFixture implements OrderedFixtureInterface
]; ];
for ($i = 0; 25 >= $i; ++$i) { for ($i = 0; 25 >= $i; ++$i) {
$cFType = $cFTypes[rand(0, count($cFTypes) - 1)]; $cFType = $cFTypes[mt_rand(0, count($cFTypes) - 1)];
$customField = (new CustomField()) $customField = (new CustomField())
->setSlug("cf_report_{$i}") ->setSlug("cf_report_{$i}")
->setType($cFType['type']) ->setType($cFType['type'])
->setOptions($cFType['options']) ->setOptions($cFType['options'])
->setName(['fr' => "CustomField {$i}"]) ->setName(['fr' => "CustomField {$i}"])
->setOrdering(rand(0, 1000) / 1000) ->setOrdering(mt_rand(0, 1000) / 1000)
->setCustomFieldsGroup($this->getReference('cf_group_report_' . (rand(0, 3)))); ->setCustomFieldsGroup($this->getReference('cf_group_report_' . (mt_rand(0, 3))));
$manager->persist($customField); $manager->persist($customField);
} }

View File

@ -84,7 +84,7 @@ class LoadReports extends AbstractFixture implements OrderedFixtureInterface, Co
$report = (new Report()) $report = (new Report())
->setPerson($person) ->setPerson($person)
->setCFGroup( ->setCFGroup(
rand(0, 10) > 5 ? mt_rand(0, 10) > 5 ?
$this->getReference('cf_group_report_logement') : $this->getReference('cf_group_report_logement') :
$this->getReference('cf_group_report_education') $this->getReference('cf_group_report_education')
) )
@ -107,7 +107,7 @@ class LoadReports extends AbstractFixture implements OrderedFixtureInterface, Co
//set date. 30% of the dates are 2015-05-01 //set date. 30% of the dates are 2015-05-01
$expectedDate = new DateTime('2015-01-05'); $expectedDate = new DateTime('2015-01-05');
if (rand(0, 100) < 30) { if (mt_rand(0, 100) < 30) {
$report->setDate($expectedDate); $report->setDate($expectedDate);
} else { } else {
$report->setDate($this->faker->dateTimeBetween('-1 year', 'now') $report->setDate($this->faker->dateTimeBetween('-1 year', 'now')
@ -151,7 +151,7 @@ class LoadReports extends AbstractFixture implements OrderedFixtureInterface, Co
$selectedPeople = []; $selectedPeople = [];
foreach ($people as $person) { foreach ($people as $person) {
if (rand(0, 100) < $percentage) { if (mt_rand(0, 100) < $percentage) {
$selectedPeople[] = $person; $selectedPeople[] = $person;
} }
} }
@ -179,7 +179,7 @@ class LoadReports extends AbstractFixture implements OrderedFixtureInterface, Co
$picked = []; $picked = [];
if ($multiple) { if ($multiple) {
$numberSelected = rand(1, count($choices) - 1); $numberSelected = mt_rand(1, count($choices) - 1);
for ($i = 0; $i < $numberSelected; ++$i) { for ($i = 0; $i < $numberSelected; ++$i) {
$picked[] = $this->pickChoice($choices); $picked[] = $this->pickChoice($choices);

View File

@ -187,6 +187,6 @@ class TimelineProviderTest extends WebTestCase
} }
} }
return $groups[rand(0, count($groups) - 1)]; return $groups[mt_rand(0, count($groups) - 1)];
} }
} }