Page MenuHomePhabricator

Minor fix for `lipsum generate`
ClosedPublic

Authored by joshuaspence on Feb 28 2015, 11:16 AM.
Tags
None
Referenced Files
F15451025: D11908.id28696.diff
Fri, Mar 28, 7:45 PM
F15429364: D11908.id28696.diff
Mon, Mar 24, 1:47 AM
F15428846: D11908.id28696.diff
Sun, Mar 23, 10:47 PM
F15409084: D11908.id28686.diff
Wed, Mar 19, 3:15 AM
F15403542: D11908.id28696.diff
Tue, Mar 18, 3:46 AM
F15401338: D11908.diff
Mon, Mar 17, 5:09 PM
F15377398: D11908.id28696.diff
Thu, Mar 13, 8:46 AM
F15331974: D11908.id28696.diff
Fri, Mar 7, 3:40 PM
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.