[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

Other related posts: