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

ezcontentobject::removeThis() has dead code

    Details

    • Type: Improvement Improvement
    • Status: Open
    • Priority: Low Low
    • Resolution: Unresolved
    • Affects Version/s: 5.2, 5.3.3, 5.4.1.1
    • Fix Version/s: Customer request
    • Component/s: Legacy stack
    • Labels:
      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.

        Activity

        There are no comments yet on this issue.

          People

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

            Dates

            • Created:
              Updated: