[racktables-users] Re: A brand new shiny pre-release is ready to be tested
- From: Denis Yeldandi <dyeldandi@xxxxxxxxxx>
- To: racktables-users@xxxxxxxxxxxxx
- Date: Tue, 10 Mar 2009 12:09:10 +0300
Mateusz,
here's another bug? tried to add file (txt) to object + comment on this url http://racktables.org/demo/temporal/?page=object&tab=files&object_id=22
Yep, that is a bug.
Some more strange things, might be related. http://racktables.org/demo/temporal/?page=rackspace&tab=edit click save button on any row
Yep, that's another nasty typo. I've fixed both and updated the demo code. Thank you for the report! Denis. Mateusz Blaszczyk wrote:
Oh, just detected a possible bug within the Files section. When looking ata file (e.g. http://racktables.org/demo/temporal/?page=file&tab=default&file_id=1 ),and pressing on the "save changes" button without actually having entered a new filename on "upload replacement", this gets sent and the file getskicked off (e.g. 0 bytes after that).Excellent spot! That actually revealed another 2 bugs with files. I justfixed all that and updated the demo code. One bug report helped fix 3 bugs,well done!here's another bug? tried to add file (txt) to object + comment on this url http://racktables.org/demo/temporal/?page=object&tab=files&object_id=22 after clicking save:http://racktables.org/demo/temporal/process.php?page=object&tab=files&op=addFileIt spits this: Uncaught exception: PDOExceptionSQLSTATE[23000]: Integrity constraint violation: 1048 Column 'contents' cannot be null (23000)at file /var/www/vhosts/racktables.org/demo/temporal/inc/revdatabase.php, line 713Array ( [0] => Array ([file] => /var/www/vhosts/racktables.org/demo/temporal/inc/revdatabase.php[line] => 713 [function] => execute [class] => PDOStatement [type] => -> [args] => Array ( ) ) [1] => Array ([file] => /var/www/vhosts/racktables.org/demo/temporal/inc/database.php[line] => 3496 [function] => insert [class] => Database [type] => :: [args] => Array ( [0] => Array ( [name] => aaaa.txt [type] => text/plain [size] => 4441 [atime] => 20090310102754[contents] => [comment] => Power requirements for New-York network UPS) [1] => File ) ) [2] => Array ([file] => /var/www/vhosts/racktables.org/demo/temporal/inc/ophandlers.php[line] => 1645 [function] => commitAddFile [args] => Array ( [0] => aaaa.txt [1] => text/plain [2] => 4441 [3] => Resource id #22 [4] => Power requirements for New-York network UPS ) ) [3] => Array ([file] => /var/www/vhosts/racktables.org/demo/temporal/process.php[line] => 33 [function] => addFileToEntity [args] => Array ( ) ) ) HTH
- References:
- [racktables-users] Re: A brand new shiny pre-release is ready to be tested
- From: Mateusz Blaszczyk
- [racktables-users] Re: A brand new shiny pre-release is ready to be tested
Other related posts:
- » [racktables-users] A brand new shiny pre-release is ready to be tested - Denis Yeldandi
- » [racktables-users] Re: A brand new shiny pre-release is ready to be tested - Frank Altpeter
- » [racktables-users] Re: A brand new shiny pre-release is ready to be tested - Denis Yeldandi
- » [racktables-users] Re: A brand new shiny pre-release is ready to be tested - Mateusz Blaszczyk
- » [racktables-users] Re: A brand new shiny pre-release is ready to be tested - Mateusz Blaszczyk
- » [racktables-users] Re: A brand new shiny pre-release is ready to be tested - Mateusz Blaszczyk
- » [racktables-users] Re: A brand new shiny pre-release is ready to be tested - Denis Yeldandi
- » [racktables-users] Re: A brand new shiny pre-release is ready to be tested - Denis Yeldandi
- » [racktables-users] Re: A brand new shiny pre-release is ready to be tested - Denis Ovsienko
- » [racktables-users] Re: A brand new shiny pre-release is ready to be tested - Mateusz Blaszczyk
- » [racktables-users] Re: A brand new shiny pre-release is ready to be tested - Denis Yeldandi
- » [racktables-users] Re: A brand new shiny pre-release is ready to be tested - Denis Ovsienko
- » [racktables-users] Re: A brand new shiny pre-release is ready to be tested - Mateusz Blaszczyk
- » [racktables-users] Re: A brand new shiny pre-release is ready to be tested - Luis Fernando Lacayo