Bug #14509

File module (aka fileadmin) "redirect" and "target" problems

Added by Wolfgang Klinger almost 15 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Should have
Category:
Backend API
Target version:
Start date:
2005-01-22
Due date:
% Done:

100%

TYPO3 Version:
4.0
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:
Remote Sprint

Description

Try to create a folder that already exists. You'll get a warning message and the mentioned "Continue" button. If you take a look at the HTML of this page you can see that the link points now either to "file_list.php" alone (without any parameters) or with parameters added url encoded (had both cases on two different servers)

Ok, the second case can be fixed with the attached patch (simply call urldecode before using the file path);
I'll investigate in the other case(s)...

But now on to the next one ;-) (no, it's not over)

If you try to create e.g. three (already existing) folders you'll get a blank page, as "document.location=''" (empty) is set!!

This is related to the following bug I think: http://bugs.typo3.org/bug_view_page.php?bug_id=0000637
(somehow) ;-)
(issue imported from #M709)

0000709-file_list.diff View (149 Bytes) Administrator Admin, 2005-01-22 12:56


Related issues

Related to TYPO3 Core - Bug #65305: Folder::getSubfolder() doesn't use the file drivers correctly Closed 2015-05-25
Blocked by TYPO3 Core - Bug #59475: Folder Object is missing getFile Closed 2014-06-11

Associated revisions

Revision 44fd59e4 (diff)
Added by Michael Oehlhof over 4 years ago

[BUGFIX] Message when trying to create existing folder

Show a flash message when trying to create a folder with a name that
already exists.

Releases: master
Resolves: #14509
Change-Id: Ia293ea61a3a892103c9b82affe5f527f1c05b39e
Reviewed-on: http://review.typo3.org/36733
Reviewed-by: Nicole Cordes <>
Tested-by: Nicole Cordes <>
Reviewed-by: Georg Ringer <>
Tested-by: Georg Ringer <>
Reviewed-by: Stefan Froemken <>
Tested-by: Stefan Froemken <>

History

#1 Updated by Mathias Schreiber over 13 years ago

TYPO3 Version too old.
The reported version is no longer maintained.

If this bug still persists in 4.0.1, please re-open it again and set the field "Product Version" to the corresponding version.

#2 Updated by Wolfgang Klinger over 13 years ago

----------
If you try to create e.g. three (already existing) folders you'll get a blank page, as "document.location=''" (empty) is set!!
----------
is still valid

#3 Updated by Alexander Opitz over 6 years ago

  • Status changed from New to Needs Feedback
  • Target version deleted (0)
  • PHP Version deleted (4)

Does this happen with TYPO3 CMS 4.5 or with 6.0?

#4 Updated by Riccardo De Contardi over 6 years ago

Tried with TYPO3 4.5.20:
Filelist module
1. created a folder
2. tried to create a folder with the same name
result: error message:

1: File or directory "/var/lib/typo3-sites/dummy-4.5L_test/fileadmin/videos/temp" existed already!

And no further folder created

3. created 4 folders
4. tried to create 4 folders with the same names
result: 4 error messages (like the previous one) and no further folder created

Tried the same with TYPO3 6.1.0
results: no folder has been created, but... no error message has been showed (why??) :S

#5 Updated by Alexander Opitz over 6 years ago

  • Status changed from Needs Feedback to New
  • Is Regression set to No

#6 Updated by Riccardo De Contardi about 5 years ago

Tried with TYPO3 6.2.5:
Filelist module
1. created a folder
2. tried to create a folder with the same name

Result: no error message shown

#7 Updated by Michael Oehlhof almost 5 years ago

  • Target version set to 7.1 (Cleanup)

#8 Updated by Gerrit Code Review almost 5 years ago

  • Status changed from New 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/36733

#9 Updated by Gerrit Code Review almost 5 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/36733

#10 Updated by Gerrit Code Review almost 5 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/36733

#11 Updated by Gerrit Code Review almost 5 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/36733

#12 Updated by Gerrit Code Review almost 5 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/36733

#13 Updated by Benni Mack over 4 years ago

  • Target version changed from 7.1 (Cleanup) to 7.4 (Backend)

#14 Updated by Michael Oehlhof over 4 years ago

  • Sprint Focus set to Remote Sprint

#15 Updated by Michael Oehlhof over 4 years ago

  • Assignee set to Michael Oehlhof

#16 Updated by Gerrit Code Review over 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/36733

#17 Updated by Michael Oehlhof over 4 years ago

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

#18 Updated by Riccardo De Contardi about 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF