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

Get rid of eZDir::path()

    XMLWordPrintable

Details

    • Icon: Bug Bug
    • Resolution: Unresolved
    • Icon: Medium Medium
    • None
    • 4.1.4, 4.2.0, 4.3.0alpha1
    • Misc
    • None

    Description

      It slows down everything (profiling shows 10-15% of time elapsed in this function) and can easily be replaced:

      eZDir::path( array( $this->RootDir, $this->FileName ) );
      

      can just be:

      $this->RootDir . "/" . $this->FileName;
      

      PHP plays very well with / even on Windows OS.
      We shouldn't care about a path being ended by a dir separator or not, if unsure => just add it. It is safe to use multiple dir separators like in: "var//cache///"

      Attachments

        Activity

          People

            e8318ba6-e4ae-477c-9116-36c073bd11a3@accounts.ibexa.co Patrick Allaert
            patrick.allaert patrick.allaert
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated: