[racktables-users] Re: A brand new shiny pre-release is ready to be tested
- From: Mateusz Blaszczyk <blahu77@xxxxxxxxx>
- To: racktables-users@xxxxxxxxxxxxx
- Date: Tue, 10 Mar 2009 08:30:05 +0000 (IST)
Oh, just detected a possible bug within the Files section. When looking at a 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 gets kicked off (e.g. 0 bytes after that).Excellent spot! That actually revealed another 2 bugs with files. I just fixed 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=addFile It spits this: Uncaught exception: PDOException SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'contents' cannot be null (23000) at file /var/www/vhosts/racktables.org/demo/temporal/inc/revdatabase.php, line 713 Array ( [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
--
pgp-key 0x1C655CAB
- Follow-Ups:
- [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
- From: Denis Yeldandi
- [racktables-users] Re: A brand new shiny pre-release is ready to be tested
- References:
- [racktables-users] Re: A brand new shiny pre-release is ready to be tested
- From: Denis Yeldandi
- [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