bwssytems / domuslink

web-based frontend for Heyu
http://domus.link.co.pt/
1 stars 1 forks source link

«Output style broken» on utility if no command & arguments are provided #2

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Go to ./admin/utility.php
2. Don't select any command anddon't write any argumment
3. Click execute

What is the expected output? What do you see instead?
The expect output should be an error inside the output box but instead it
redirects to error.php

What version of the product are you using? On what operating system?
1.0.2

Please provide any additional information below.
Its not a big deal but should be fixed anyway

Original issue reported on code.google.com by sergio9...@gmail.com on 22 Feb 2010 at 10:35

GoogleCodeExporter commented 9 years ago
The utility.php should check for empty commands and arguments. It is using the
default error catch php for domuslink.

Original comment by bwsamuels@gmail.com on 22 Feb 2010 at 2:45

GoogleCodeExporter commented 9 years ago

Original comment by bwsamuels@gmail.com on 22 Feb 2010 at 2:46

GoogleCodeExporter commented 9 years ago
I know that but the error is still part of the output thats why I think it 
shouldn't
redirect to another page. Also if the error is shown on the output area it's 
easier
for the user to correct it.

Original comment by sergio9...@gmail.com on 22 Feb 2010 at 2:58

GoogleCodeExporter commented 9 years ago
Agreed, As my previous comment stated, we should change it. I was only 
re-iterating
the actual path that was being taken so that others know what we want to change.

Original comment by bwsamuels@gmail.com on 22 Feb 2010 at 3:09

GoogleCodeExporter commented 9 years ago

Original comment by bwsamuels@gmail.com on 7 Mar 2010 at 3:26

GoogleCodeExporter commented 9 years ago
Change the call to lib func execute command to return error as a string instead 
of
calling gen_error display by setting noerror flag to true.

Original comment by bwsamuels@gmail.com on 27 Mar 2010 at 7:22

GoogleCodeExporter commented 9 years ago
This issue was closed by revision r647.

Original comment by bwsamuels@gmail.com on 27 Mar 2010 at 7:44