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

ezcontentobject::removeThis() has dead code

    XMLWordPrintable

Details

    • Icon: Improvement Improvement
    • Resolution: Unresolved
    • Icon: Low Low
    • Customer request
    • 5.2, 5.3.3, 5.4.1.1
    • Legacy stack
    • None

    Description

      the ezcontentobject::removeThis() has a block of code that is not used.

      the block in question is an "else if(){}" that starts here:
      https://github.com/ezsystems/ezpublish-legacy/blob/master/kernel/classes/ezcontentobject.php#L1979
      there is even buggy code inside, but it's never used. there is not a single call of removeThis() with a nodeId parameter.

      this code throws warnings in some IDEs about the unistantiated var $additionalNodes.

      the whole block should be removed.

      Attachments

        Activity

          People

            Unassigned Unassigned
            paulo.bras-obsolete@ez.no Paulo Bras (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated: