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

Bug in eZContentOperationCollection::registerSearchObject() in eZ Publish 4.1.3

    Details

      Description

      found a minor bug in eZContentOperationCollection::registerSearchObject()

      here

              if ( $delayedIndexing == 'enabled' )
              {
                  $db = eZDB::instance();
                  $rows = $db->arrayQuery( "SELECT param FROM ezpending_actions WHERE action='index_object' AND param = '$objectID'" );
                  if ( count( $rows ) == 0 )
                  {
                      $db->query( "INSERT INTO ezpending_actions( action, param ) VALUES ( 'index_object', '$objectID' )" );
                  return;
                  }
              }
      

      the "return" should be outside the "if ( count..." because when an object is already pending for indexation, it never enters inside this if, causing a realtime indexation, not delayed, and this is causing timeout errors when publishing content.

      Steps to reproduce
      • enable delayedindexing
      • publish some content -> a pending action is added (this is ok)
      • before runing update index cron, edit and publish the same object -> you get realtime indexation in this case (this is not desired with delayed indexing enabled)

        Issue Links

          Activity

          Hide
          (inactive) Gunnstein Lye added a comment -

          This bug was introduced with the enhancement issue #12456 in 4.1.0, and does not affect 4.0.

          Show
          (inactive) Gunnstein Lye added a comment - This bug was introduced with the enhancement issue #12456 in 4.1.0, and does not affect 4.0.
          Hide
          (inactive) Gunnstein Lye added a comment -

          Fixed in
          trunk (4.2.0Beta1) Rev. 23911
          stable/4.1 (4.1.4) Rev. 23912

          Show
          (inactive) Gunnstein Lye added a comment - Fixed in trunk (4.2.0Beta1) Rev. 23911 stable/4.1 (4.1.4) Rev. 23912
          Hide
          ezrobot added a comment -

          This issue has been automatically closed due to the lack of activity over a long period of time. It is very likely that it is obsolete, but if you think it is still valid, do not hesitate to reopen it and mention why.

          Show
          ezrobot added a comment - This issue has been automatically closed due to the lack of activity over a long period of time. It is very likely that it is obsolete, but if you think it is still valid, do not hesitate to reopen it and mention why.

            People

            • Assignee:
              (inactive) Gunnstein Lye
              Reporter:
              Disabled User
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: