CLI script help output is incorrect

Bug #1496616 reported by Robert Lyon
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Low
Robert Lyon
1.10
Fix Released
Low
Unassigned
1.9
Fix Released
Low
Unassigned
15.04
Fix Released
Low
Unassigned
15.10
Fix Released
Low
Robert Lyon

Bug Description

when running sudo php reset_password.php in htdocs/admin/cli/ without setting any arguments it returns the help information but indicates "Usage: cli.php" which is wrong, it should be "Usage: reset_password.php"

Scripts need a way to pass in their file name to cli.php so we can display the correct file name.

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "master" branch: https://reviews.mahara.org/5301

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/5301
Committed: https://git.nzoss.org.nz/mahara/mahara/commit/4062091083148337b81976007ab1ce5c69ccaa87
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit 4062091083148337b81976007ab1ce5c69ccaa87
Author: Aaron Wells <email address hidden>
Date: Thu Sep 17 17:10:38 2015 +1200

Bug 1496616: Show the right script name in cli.php help text

behatnotneeded: Can't behat test CLI scripts

Change-Id: Ic83251491c745bb4375d40510139365a5e004800

Robert Lyon (robertl-9)
Changed in mahara:
status: In Progress → Fix Committed
Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "15.04_STABLE" branch: https://reviews.mahara.org/5336

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Patch for "1.10_STABLE" branch: https://reviews.mahara.org/5337

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Patch for "1.9_STABLE" branch: https://reviews.mahara.org/5338

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/5338
Committed: https://git.nzoss.org.nz/mahara/mahara/commit/1368bd17b29e9b3958c647962d0764891287d48e
Submitter: Robert Lyon (<email address hidden>)
Branch: 1.9_STABLE

commit 1368bd17b29e9b3958c647962d0764891287d48e
Author: Aaron Wells <email address hidden>
Date: Thu Sep 17 17:10:38 2015 +1200

Bug 1496616: Show the right script name in cli.php help text

behatnotneeded: Can't behat test CLI scripts

Change-Id: Ic83251491c745bb4375d40510139365a5e004800
(cherry picked from commit 4062091083148337b81976007ab1ce5c69ccaa87)

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Reviewed: https://reviews.mahara.org/5337
Committed: https://git.nzoss.org.nz/mahara/mahara/commit/e9c09920799ef78fb3315c4fdd7bd4a4cf5d4df1
Submitter: Robert Lyon (<email address hidden>)
Branch: 1.10_STABLE

commit e9c09920799ef78fb3315c4fdd7bd4a4cf5d4df1
Author: Aaron Wells <email address hidden>
Date: Thu Sep 17 17:10:38 2015 +1200

Bug 1496616: Show the right script name in cli.php help text

behatnotneeded: Can't behat test CLI scripts

Change-Id: Ic83251491c745bb4375d40510139365a5e004800
(cherry picked from commit 4062091083148337b81976007ab1ce5c69ccaa87)

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Reviewed: https://reviews.mahara.org/5336
Committed: https://git.nzoss.org.nz/mahara/mahara/commit/47d2c4e8b4984c5f6932f694dcbb68570a44da04
Submitter: Aaron Wells (<email address hidden>)
Branch: 15.04_STABLE

commit 47d2c4e8b4984c5f6932f694dcbb68570a44da04
Author: Aaron Wells <email address hidden>
Date: Thu Sep 17 17:10:38 2015 +1200

Bug 1496616: Show the right script name in cli.php help text

behatnotneeded: Can't behat test CLI scripts

Change-Id: Ic83251491c745bb4375d40510139365a5e004800
(cherry picked from commit 4062091083148337b81976007ab1ce5c69ccaa87)

Robert Lyon (robertl-9)
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.