Uploaded image for project: 'eZ Publish / Platform'
  1. eZ Publish / Platform
  2. EZP-21563

workflows: adding approval to media tree causes fatal error

    Details

      Description

      steps to reproduce:

      1 - create an editor user
      2 - with admin user, create a workflow with an approve event for "media" section (or "any", result is the same)
      3 - set the trigger content->publish->before to the workflow created
      4 - log in with the editor user, create a new article.
      5 - on ezOE, click the add image button to open the popup
      6 - add an external image. when the "upload local image" is clicked, a fatal error is shown in the popup.

        Issue Links

          Activity

          Hide
          Bertrand Dunogier added a comment - - edited

          No error message or trace or log data ?

          Reproduced with the described sequence. Entry in error.log:

          Internal transaction counter mismatch : 1. Should be zero.

          With DebugTransactions=enabled:

          Level 0
          {
            Began at:
            /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentupload.php:620 eZDBInterface::begin
              ->/home/bertrand/www/ezpublish-legacy/extension/ezoe/modules/ezoe/upload.php:114 eZContentUpload::handleUpload
              Level 1
              {
                Began at:
                /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclass.php:303 eZDBInterface::begin
                  ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclass.php:263 eZContentClass::instantiate
                  Level 2
                  {
                    Began at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:260 eZDBInterface::begin
                      ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclass.php:305 eZContentObject::store
                    And commited at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:263 eZDBInterface::commit
                      ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclass.php:305 eZContentObject::store
                  }
                  Level 2
                  {
                    Began at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:6545 eZDBInterface::begin
                      ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclass.php:306 eZContentObject::assignDefaultStates
                    And commited at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:6554 eZDBInterface::commit
                      ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclass.php:306 eZContentObject::assignDefaultStates
                  }
                  Level 2
                  {
                    Began at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:502 eZDBInterface::begin
                      ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclass.php:307 eZContentObject::setName
                    And commited at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:533 eZDBInterface::commit
                      ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclass.php:307 eZContentObject::setName
                  }
                  Level 2
                  {
                    Began at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:276 eZDBInterface::begin
                      ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclassattribute.php:271 eZContentObjectAttribute::store
                    And commited at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:283 eZDBInterface::commit
                      ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclassattribute.php:271 eZContentObjectAttribute::store
                  }
                  Level 2
                  {
                    Began at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:276 eZDBInterface::begin
                      ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclassattribute.php:271 eZContentObjectAttribute::store
                    And commited at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:283 eZDBInterface::commit
                      ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclassattribute.php:271 eZContentObjectAttribute::store
                  }
                  Level 2
                  {
                    Began at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:276 eZDBInterface::begin
                      ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclassattribute.php:271 eZContentObjectAttribute::store
                    And commited at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:283 eZDBInterface::commit
                      ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclassattribute.php:271 eZContentObjectAttribute::store
                  }
                  Level 2
                  {
                    Began at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:276 eZDBInterface::begin
                      ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclassattribute.php:271 eZContentObjectAttribute::store
                    And commited at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:283 eZDBInterface::commit
                      ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclassattribute.php:271 eZContentObjectAttribute::store
                  }
                  Level 2
                  {
                    Began at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:276 eZDBInterface::begin
                      ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclassattribute.php:271 eZContentObjectAttribute::store
                    And commited at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:283 eZDBInterface::commit
                      ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclassattribute.php:271 eZContentObjectAttribute::store
                  }
                And commited at:
                /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclass.php:344 eZDBInterface::commit
                  ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclass.php:263 eZContentClass::instantiate
              }
              Level 1
              {
                Began at:
                /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:276 eZDBInterface::begin
                  ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentupload.php:676 eZContentObjectAttribute::store
                And commited at:
                /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:283 eZDBInterface::commit
                  ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentupload.php:676 eZContentObjectAttribute::store
              }
              Level 1
              {
                Began at:
                /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:276 eZDBInterface::begin
                  ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentupload.php:697 eZContentObjectAttribute::store
                And commited at:
                /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:283 eZDBInterface::commit
                  ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentupload.php:697 eZContentObjectAttribute::store
              }
              Level 1
              {
                Began at:
                /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:502 eZDBInterface::begin
                  ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentupload.php:723 eZContentObject::setName
                And commited at:
                /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:533 eZDBInterface::commit
                  ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentupload.php:723 eZContentObject::setName
              }
              Level 1
              {
                Began at:
                /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:260 eZDBInterface::begin
                  ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentupload.php:724 eZContentObject::store
                  Level 2
                  {
                    Began at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:313 eZDBInterface::begin
                      ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:261 eZContentObject::storeNodeModified
                    And commited at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:318 eZDBInterface::commit
                      ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:261 eZContentObject::storeNodeModified
                  }
                And commited at:
                /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:263 eZDBInterface::commit
                  ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentupload.php:724 eZContentObject::store
              }
              Level 1
              {
                Began at:
                /home/bertrand/www/ezpublish-legacy/kernel/content/ezcontentoperationcollection.php:112 eZDBInterface::begin
                  ->???:??? eZContentOperationCollection::beginTransaction
                  Level 2
                  {
                    Began at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:260 eZDBInterface::begin
                      ->/home/bertrand/www/ezpublish-legacy/kernel/content/ezcontentoperationcollection.php:418 eZContentObject::store
                      Level 3
                      {
                        Began at:
                        /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:313 eZDBInterface::begin
                          ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:261 eZContentObject::storeNodeModified
                        And commited at:
                        /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:318 eZDBInterface::commit
                          ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:261 eZContentObject::storeNodeModified
                      }
                    And commited at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:263 eZDBInterface::commit
                      ->/home/bertrand/www/ezpublish-legacy/kernel/content/ezcontentoperationcollection.php:418 eZContentObject::store
                  }
                  Level 2
                  {
                    Began at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcollaborationitemgrouplink.php:99 eZDBInterface::begin
                      ->/home/bertrand/www/ezpublish-legacy/kernel/classes/collaborationhandlers/ezapprove/ezapprovecollaborationhandler.php:186 eZCollaborationItemGroupLink::addItem
                    And commited at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcollaborationitemgrouplink.php:103 eZDBInterface::commit
                      ->/home/bertrand/www/ezpublish-legacy/kernel/classes/collaborationhandlers/ezapprove/ezapprovecollaborationhandler.php:186 eZCollaborationItemGroupLink::addItem
                  }
                  Level 2
                  {
                    Began at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcollaborationitemgrouplink.php:99 eZDBInterface::begin
                      ->/home/bertrand/www/ezpublish-legacy/kernel/classes/collaborationhandlers/ezapprove/ezapprovecollaborationhandler.php:186 eZCollaborationItemGroupLink::addItem
                    And commited at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcollaborationitemgrouplink.php:103 eZDBInterface::commit
                      ->/home/bertrand/www/ezpublish-legacy/kernel/classes/collaborationhandlers/ezapprove/ezapprovecollaborationhandler.php:186 eZCollaborationItemGroupLink::addItem
                  }
                  Level 2
                  {
                    Began at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/eztrigger.php:244 eZDBInterface::begin
                      ->/home/bertrand/www/ezpublish-legacy/kernel/classes/eztrigger.php:222 eZTrigger::runWorkflow
                    And commited at:
                    /home/bertrand/www/ezpublish-legacy/kernel/classes/eztrigger.php:298 eZDBInterface::commit
                      ->/home/bertrand/www/ezpublish-legacy/kernel/classes/eztrigger.php:222 eZTrigger::runWorkflow
                  }
                And commited at:
                /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentupload.php:702 eZDBInterface::commit
                  ->/home/bertrand/www/ezpublish-legacy/extension/ezoe/modules/ezoe/upload.php:114 eZContentUpload::handleUpload
              }
              Level 1
              {
                Began at:
                /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:2850 eZDBInterface::begin
                  ->/home/bertrand/www/ezpublish-legacy/extension/ezoe/modules/ezoe/upload.php:182 eZContentObject::addContentObjectRelation
                And commited at:
                /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:2853 eZDBInterface::commit
                  ->/home/bertrand/www/ezpublish-legacy/extension/ezoe/modules/ezoe/upload.php:182 eZContentObject::addContentObjectRelation
              }
          }
          [ Sep 14 2013 11:08:06 ] [192.168.56.1] eZDBInterface::commit TRANSID-7f8c12e6bc3b56c780ea36d49e20b0cd:
          Transaction in progress failed due to DB error, transaction was rollbacked. Transaction ID is TRANSID-7f8c12e6bc3b56c780ea36d49e20b0cd.
          [ Sep 14 2013 11:08:07 ] [192.168.56.1] Timing Point: Module start 'ezoe'
          [ Sep 14 2013 11:08:07 ] [192.168.56.1] eZContentObject::fetch:
          Object not found (193)
          

          Show
          Bertrand Dunogier added a comment - - edited No error message or trace or log data ? Reproduced with the described sequence. Entry in error.log: Internal transaction counter mismatch : 1. Should be zero. With DebugTransactions=enabled: Level 0 { Began at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentupload.php:620 eZDBInterface::begin ->/home/bertrand/www/ezpublish-legacy/extension/ezoe/modules/ezoe/upload.php:114 eZContentUpload::handleUpload Level 1 { Began at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclass.php:303 eZDBInterface::begin ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclass.php:263 eZContentClass::instantiate Level 2 { Began at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:260 eZDBInterface::begin ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclass.php:305 eZContentObject::store And commited at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:263 eZDBInterface::commit ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclass.php:305 eZContentObject::store } Level 2 { Began at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:6545 eZDBInterface::begin ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclass.php:306 eZContentObject::assignDefaultStates And commited at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:6554 eZDBInterface::commit ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclass.php:306 eZContentObject::assignDefaultStates } Level 2 { Began at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:502 eZDBInterface::begin ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclass.php:307 eZContentObject::setName And commited at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:533 eZDBInterface::commit ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclass.php:307 eZContentObject::setName } Level 2 { Began at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:276 eZDBInterface::begin ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclassattribute.php:271 eZContentObjectAttribute::store And commited at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:283 eZDBInterface::commit ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclassattribute.php:271 eZContentObjectAttribute::store } Level 2 { Began at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:276 eZDBInterface::begin ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclassattribute.php:271 eZContentObjectAttribute::store And commited at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:283 eZDBInterface::commit ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclassattribute.php:271 eZContentObjectAttribute::store } Level 2 { Began at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:276 eZDBInterface::begin ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclassattribute.php:271 eZContentObjectAttribute::store And commited at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:283 eZDBInterface::commit ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclassattribute.php:271 eZContentObjectAttribute::store } Level 2 { Began at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:276 eZDBInterface::begin ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclassattribute.php:271 eZContentObjectAttribute::store And commited at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:283 eZDBInterface::commit ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclassattribute.php:271 eZContentObjectAttribute::store } Level 2 { Began at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:276 eZDBInterface::begin ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclassattribute.php:271 eZContentObjectAttribute::store And commited at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:283 eZDBInterface::commit ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclassattribute.php:271 eZContentObjectAttribute::store } And commited at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclass.php:344 eZDBInterface::commit ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentclass.php:263 eZContentClass::instantiate } Level 1 { Began at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:276 eZDBInterface::begin ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentupload.php:676 eZContentObjectAttribute::store And commited at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:283 eZDBInterface::commit ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentupload.php:676 eZContentObjectAttribute::store } Level 1 { Began at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:276 eZDBInterface::begin ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentupload.php:697 eZContentObjectAttribute::store And commited at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobjectattribute.php:283 eZDBInterface::commit ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentupload.php:697 eZContentObjectAttribute::store } Level 1 { Began at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:502 eZDBInterface::begin ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentupload.php:723 eZContentObject::setName And commited at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:533 eZDBInterface::commit ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentupload.php:723 eZContentObject::setName } Level 1 { Began at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:260 eZDBInterface::begin ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentupload.php:724 eZContentObject::store Level 2 { Began at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:313 eZDBInterface::begin ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:261 eZContentObject::storeNodeModified And commited at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:318 eZDBInterface::commit ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:261 eZContentObject::storeNodeModified } And commited at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:263 eZDBInterface::commit ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentupload.php:724 eZContentObject::store } Level 1 { Began at: /home/bertrand/www/ezpublish-legacy/kernel/content/ezcontentoperationcollection.php:112 eZDBInterface::begin ->???:??? eZContentOperationCollection::beginTransaction Level 2 { Began at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:260 eZDBInterface::begin ->/home/bertrand/www/ezpublish-legacy/kernel/content/ezcontentoperationcollection.php:418 eZContentObject::store Level 3 { Began at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:313 eZDBInterface::begin ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:261 eZContentObject::storeNodeModified And commited at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:318 eZDBInterface::commit ->/home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:261 eZContentObject::storeNodeModified } And commited at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:263 eZDBInterface::commit ->/home/bertrand/www/ezpublish-legacy/kernel/content/ezcontentoperationcollection.php:418 eZContentObject::store } Level 2 { Began at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcollaborationitemgrouplink.php:99 eZDBInterface::begin ->/home/bertrand/www/ezpublish-legacy/kernel/classes/collaborationhandlers/ezapprove/ezapprovecollaborationhandler.php:186 eZCollaborationItemGroupLink::addItem And commited at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcollaborationitemgrouplink.php:103 eZDBInterface::commit ->/home/bertrand/www/ezpublish-legacy/kernel/classes/collaborationhandlers/ezapprove/ezapprovecollaborationhandler.php:186 eZCollaborationItemGroupLink::addItem } Level 2 { Began at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcollaborationitemgrouplink.php:99 eZDBInterface::begin ->/home/bertrand/www/ezpublish-legacy/kernel/classes/collaborationhandlers/ezapprove/ezapprovecollaborationhandler.php:186 eZCollaborationItemGroupLink::addItem And commited at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcollaborationitemgrouplink.php:103 eZDBInterface::commit ->/home/bertrand/www/ezpublish-legacy/kernel/classes/collaborationhandlers/ezapprove/ezapprovecollaborationhandler.php:186 eZCollaborationItemGroupLink::addItem } Level 2 { Began at: /home/bertrand/www/ezpublish-legacy/kernel/classes/eztrigger.php:244 eZDBInterface::begin ->/home/bertrand/www/ezpublish-legacy/kernel/classes/eztrigger.php:222 eZTrigger::runWorkflow And commited at: /home/bertrand/www/ezpublish-legacy/kernel/classes/eztrigger.php:298 eZDBInterface::commit ->/home/bertrand/www/ezpublish-legacy/kernel/classes/eztrigger.php:222 eZTrigger::runWorkflow } And commited at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentupload.php:702 eZDBInterface::commit ->/home/bertrand/www/ezpublish-legacy/extension/ezoe/modules/ezoe/upload.php:114 eZContentUpload::handleUpload } Level 1 { Began at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:2850 eZDBInterface::begin ->/home/bertrand/www/ezpublish-legacy/extension/ezoe/modules/ezoe/upload.php:182 eZContentObject::addContentObjectRelation And commited at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentobject.php:2853 eZDBInterface::commit ->/home/bertrand/www/ezpublish-legacy/extension/ezoe/modules/ezoe/upload.php:182 eZContentObject::addContentObjectRelation } } [ Sep 14 2013 11:08:06 ] [192.168.56.1] eZDBInterface::commit TRANSID-7f8c12e6bc3b56c780ea36d49e20b0cd: Transaction in progress failed due to DB error, transaction was rollbacked. Transaction ID is TRANSID-7f8c12e6bc3b56c780ea36d49e20b0cd. [ Sep 14 2013 11:08:07 ] [192.168.56.1] Timing Point: Module start 'ezoe' [ Sep 14 2013 11:08:07 ] [192.168.56.1] eZContentObject::fetch: Object not found (193)
          Hide
          Bertrand Dunogier added a comment -

          Further analysis of the transaction stack shows that this begin seems unmatched:

          Level 0
          {
            Began at:
            /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentupload.php:620 eZDBInterface::begin
              ->/home/bertrand/www/ezpublish-legacy/extension/ezoe/modules/ezoe/upload.php:114 eZContentUpload::handleUpload
          }

          Show
          Bertrand Dunogier added a comment - Further analysis of the transaction stack shows that this begin seems unmatched: Level 0 { Began at: /home/bertrand/www/ezpublish-legacy/kernel/classes/ezcontentupload.php:620 eZDBInterface::begin ->/home/bertrand/www/ezpublish-legacy/extension/ezoe/modules/ezoe/upload.php:114 eZContentUpload::handleUpload }
          Hide
          Bertrand Dunogier added a comment - - edited

          Seems like a regression from the bugfix for EZP-19660.

          The regression was fixed during the 5.1 certification phase (EZP-20558), but in content/edit.php, not in ezoe/upload.php. I'd say the same fix needs to be applied to this file.

          Show
          Bertrand Dunogier added a comment - - edited Seems like a regression from the bugfix for EZP-19660 . The regression was fixed during the 5.1 certification phase ( EZP-20558 ), but in content/edit.php, not in ezoe/upload.php. I'd say the same fix needs to be applied to this file.
          Hide
          Bertrand Dunogier added a comment -

          How did we handle uploads from eZOE that are interrupted by an approval workflow in the past ? How did it behave ?

          Show
          Bertrand Dunogier added a comment - How did we handle uploads from eZOE that are interrupted by an approval workflow in the past ? How did it behave ?
          Hide
          Bertrand Dunogier added a comment -

          After fixing the transaction error, we get a fatal error when ezjscAjaxContent::nodeEncode( $embedObject, $params ) is called from ezoe/upload.php.

          ezjscAjaxContent::simplify() seems to be the one crashing (nothing in error.log).

          Show
          Bertrand Dunogier added a comment - After fixing the transaction error, we get a fatal error when ezjscAjaxContent::nodeEncode( $embedObject, $params ) is called from ezoe/upload.php . ezjscAjaxContent::simplify() seems to be the one crashing (nothing in error.log).
          Hide
          Bertrand Dunogier added a comment - - edited

          The error occurs @ https://github.com/ezsystems/ezpublish-legacy/blob/master/extension/ezjscore/classes/ezjscajaxcontent.php#L245. $node is undefined since the image hasn't been published (interrupted by the workflow).

          We can work around this by checking the existence of the $node variable. The embed form shows up, and the image is visible. If the embed is confirmed, the image is correctly embedded, and the content can be published without errors.

          However, the image won't show in the content where it's embedded, since it hasn't been approved, and will only show up after it gets approved (tested).

          This sounds quite acceptable to me.

          Summary of fixes:

          • clear transaction queue accordingly if publishing is interrupted in kernel/classes/ezcontentupload.php
          • prevent fatal error in ezjscore when encoding an unapproved object
          Show
          Bertrand Dunogier added a comment - - edited The error occurs @ https://github.com/ezsystems/ezpublish-legacy/blob/master/extension/ezjscore/classes/ezjscajaxcontent.php#L245 . $node is undefined since the image hasn't been published (interrupted by the workflow). We can work around this by checking the existence of the $node variable. The embed form shows up, and the image is visible. If the embed is confirmed, the image is correctly embedded, and the content can be published without errors. However, the image won't show in the content where it's embedded, since it hasn't been approved, and will only show up after it gets approved (tested). This sounds quite acceptable to me. Summary of fixes: clear transaction queue accordingly if publishing is interrupted in kernel/classes/ezcontentupload.php prevent fatal error in ezjscore when encoding an unapproved object
          Show
          Bertrand Dunogier added a comment - PR: https://github.com/ezsystems/ezpublish-legacy/pull/757 .
          Show
          Bertrand Dunogier added a comment - Merged to master: https://github.com/ezsystems/ezpublish-legacy/commit/fd5ee497d020cc89a972652e19308e58de36c773 .
          Hide
          Bertrand Dunogier added a comment -

          Re-closed it. I just needed to edit a comment...

          Show
          Bertrand Dunogier added a comment - Re-closed it. I just needed to edit a comment...
          Hide
          Bertrand Dunogier added a comment -
          Show
          Bertrand Dunogier added a comment - Bugfix/workaround reverted in master: https://github.com/ezsystems/ezpublish-legacy/commit/d355026740f39039e44e909544464c1a1fead233 . See https://jira.ez.no/browse/EZP-21599 for the final, proper fix.
          Hide
          Bertrand Dunogier added a comment - - edited

          Reopening as there is a second bug on this use case: since the uploaded object has no node yet, we get a fatal error in eZContentUpload::publishObject().

          Furthermore, a bugfix that was extracted from one of the previous fixes must be also be applied: https://jira.ez.no/browse/EZP-19077 (already linked to this issue).

          Show
          Bertrand Dunogier added a comment - - edited Reopening as there is a second bug on this use case: since the uploaded object has no node yet, we get a fatal error in eZContentUpload::publishObject() . Furthermore, a bugfix that was extracted from one of the previous fixes must be also be applied: https://jira.ez.no/browse/EZP-19077 (already linked to this issue).
          Hide
          Bertrand Dunogier added a comment -

          New pull request for the node fatal error reported in EZP-21599: https://github.com/ezsystems/ezpublish-legacy/pull/780.

          Show
          Bertrand Dunogier added a comment - New pull request for the node fatal error reported in EZP-21599 : https://github.com/ezsystems/ezpublish-legacy/pull/780 .
          Show
          Bertrand Dunogier added a comment - Merged to master: https://github.com/ezsystems/ezpublish-legacy/commit/7f3145ba239253b3a3353551e1621a5dbd68c2c4 .
          Hide
          Filipe Dobreira (Inactive) added a comment -

          QA Approved.

          Show
          Filipe Dobreira (Inactive) added a comment - QA Approved.

            People

            • Assignee:
              Unassigned
              Reporter:
              Paulo Bras (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Time Spent - 5 hours, 8 minutes Remaining Estimate - 1 hour
                1h
                Logged:
                Time Spent - 5 hours, 8 minutes Remaining Estimate - 1 hour
                5h 8m