Bug #66861

Do not automatically append a "/" to the identifier of a folder

Added by Sascha Egerer over 4 years ago. Updated 11 months ago.

Status:
Closed
Priority:
Must have
Assignee:
Category:
File Abstraction Layer (FAL)
Target version:
-
Start date:
2015-05-08
Due date:
% Done:

100%

TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

When creating a Folder Object the identifier is suffixed with a / automatically which does not make sense when using a non hierarchical filesystem.

The highlevel folder api should not modify the identifier by its own.

Associated revisions

Revision d9755979 (diff)
Added by Sascha Egerer over 3 years ago

[BUGFIX] Do not append a "/" to the folder identifier

When creating a folder object the identifier is suffixed with a '/'
automatically which does not make sense when using a non hierarchical
filesystem.
The identifier must not be modified by the Folder constructor.

Resolves: #66861
Related: #65305
Releases: master, 7.6
Change-Id: I9b02767d124a7502f4170cb39a7df3dcff27fe3c
Reviewed-on: https://review.typo3.org/39371
Reviewed-by: Frans Saris <>
Tested-by: Frans Saris <>
Reviewed-by: Jigal van Hemert <>
Tested-by: Jigal van Hemert <>

Revision 1692e251 (diff)
Added by Sascha Egerer over 3 years ago

[BUGFIX] Do not append a "/" to the folder identifier

When creating a folder object the identifier is suffixed with a '/'
automatically which does not make sense when using a non hierarchical
filesystem.
The identifier must not be modified by the Folder constructor.

Resolves: #66861
Related: #65305
Releases: master, 7.6
Change-Id: I9b02767d124a7502f4170cb39a7df3dcff27fe3c
Reviewed-on: https://review.typo3.org/47749
Reviewed-by: Jigal van Hemert <>
Tested-by: Jigal van Hemert <>

Revision 27e38145 (diff)
Added by Frans Saris over 3 years ago

[FOLLOWUP][BUGFIX] Do not append a "/" to the folder identifier

Added missing rst file.

Change-Id: Id61a1d2fdc27da4837e5d0d880f8dcaad5d6e1a4
Resolves: #66861
Releases: master
Reviewed-on: https://review.typo3.org/47777
Reviewed-by: Nicole Cordes <>
Tested-by: Nicole Cordes <>

History

#1 Updated by Gerrit Code Review over 4 years ago

  • Status changed from In Progress to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/39371

#2 Updated by Gerrit Code Review over 4 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/39371

#3 Updated by Anja Leichsenring over 4 years ago

  • Sprint Focus set to Stabilization Sprint

#4 Updated by Gerrit Code Review over 4 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/39371

#5 Updated by Gerrit Code Review over 4 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/39371

#6 Updated by Gerrit Code Review about 4 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/39371

#7 Updated by Xavier Perseguers about 4 years ago

  • Target version changed from 6.2.13 to next-patchlevel
  • TYPO3 Version changed from 6.2 to 7
  • Sprint Focus deleted (Stabilization Sprint)

Will not be backported to 6.2. Thanks for your understanding.

#8 Updated by Gerrit Code Review about 4 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/39371

#9 Updated by Mathias Schreiber almost 4 years ago

  • Target version deleted (next-patchlevel)

#10 Updated by Gerrit Code Review over 3 years ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/39371

#11 Updated by Gerrit Code Review over 3 years ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/39371

#12 Updated by Gerrit Code Review over 3 years ago

Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/39371

#13 Updated by Gerrit Code Review over 3 years ago

Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/39371

#14 Updated by Gerrit Code Review over 3 years ago

Patch set 1 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47749

#15 Updated by Sascha Egerer over 3 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#16 Updated by Gerrit Code Review over 3 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47777

#17 Updated by Gerrit Code Review over 3 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47777

#18 Updated by Frans Saris over 3 years ago

  • Status changed from Under Review to Resolved

#19 Updated by Benni Mack 11 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF