Page MenuHomePhabricator

Minor fix for `lipsum generate`
ClosedPublic

Authored by joshuaspence on Feb 28 2015, 11:16 AM.
Tags
None
Referenced Files
F13277469: D11908.id28686.diff
May 31 2024, 11:02 AM
F13238754: D11908.id28696.diff
May 21 2024, 9:07 PM
F13238753: D11908.id28686.diff
May 21 2024, 9:07 PM
F13238723: D11908.id.diff
May 21 2024, 9:01 PM
F13238681: D11908.diff
May 21 2024, 8:38 PM
Unknown Object (File)
Apr 25 2024, 1:47 AM
Unknown Object (File)
Apr 24 2024, 9:57 PM
Unknown Object (File)
Apr 16 2024, 5:15 AM
Subscribers

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
rP644b05ec08ff: Minor fix for `lipsum generate`
Summary

I was hitting the following error when attempting to use ./bin/lipsum generate:

[2015-02-28 21:55:00] ERROR 2: array_rand(): Second argument has to be between 1 and the number of elements in the array at [/usr/src/phabricator/src/applications/pholio/lipsum/PhabricatorPholioMockTestDataGenerator.php:93]
arcanist(head=master, ref.master=d8182cf55d50), phabricator(head=master, ref.master=658175925931), phutil(head=master, ref.master=260e6105dee5)
  #0 array_rand(array, integer) called at [<phabricator>/src/applications/pholio/lipsum/PhabricatorPholioMockTestDataGenerator.php:93]
  #1 PhabricatorPholioMockTestDataGenerator::generateImages() called at [<phabricator>/src/applications/pholio/lipsum/PhabricatorPholioMockTestDataGenerator.php:30]
  #2 PhabricatorPholioMockTestDataGenerator::generate() called at [<phabricator>/src/applications/lipsum/management/PhabricatorLipsumGenerateWorkflow.php:67]
  #3 PhabricatorLipsumGenerateWorkflow::infinitelyGenerate(array) called at [<phabricator>/src/applications/lipsum/management/PhabricatorLipsumGenerateWorkflow.php:36]
  #4 PhabricatorLipsumGenerateWorkflow::execute(PhutilArgumentParser) called at [<phutil>/src/parser/argument/PhutilArgumentParser.php:396]
  #5 PhutilArgumentParser::parseWorkflowsFull(array) called at [<phutil>/src/parser/argument/PhutilArgumentParser.php:292]
  #6 PhutilArgumentParser::parseWorkflows(array) called at [<phabricator>/scripts/lipsum/manage_lipsum.php:21]
Test Plan

Ran ./bin/lipsum generate and received less errors.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

joshuaspence retitled this revision from to Minor fix for `lipsum generate`.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Feb 28 2015, 9:52 PM
This revision was automatically updated to reflect the committed changes.