Project

General

Profile

Actions

Bug #14509

closed

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

Added by Wolfgang Klinger almost 20 years ago. Updated about 7 years ago.

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

100%

Estimated time:
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)


Files

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

Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Bug #65305: Folder::getSubfolder() doesn't use the file drivers correctlyClosed2015-05-25

Actions
Blocked by TYPO3 Core - Bug #59475: Folder Object is missing getFileClosedSteffen Ritter2014-06-11

Actions
Actions #1

Updated by Mathias Schreiber over 18 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.

Actions #2

Updated by Wolfgang Klinger over 18 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

Actions #3

Updated by Alexander Opitz over 11 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?

Actions #4

Updated by Riccardo De Contardi over 11 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

Actions #5

Updated by Alexander Opitz about 11 years ago

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

Updated by Riccardo De Contardi about 10 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

Actions #7

Updated by Michael Oehlhof almost 10 years ago

  • Target version set to 7.1 (Cleanup)
Actions #8

Updated by Gerrit Code Review almost 10 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

Actions #9

Updated by Gerrit Code Review almost 10 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

Actions #10

Updated by Gerrit Code Review almost 10 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

Actions #11

Updated by Gerrit Code Review almost 10 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

Actions #12

Updated by Gerrit Code Review almost 10 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

Actions #13

Updated by Benni Mack over 9 years ago

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

Updated by Michael Oehlhof over 9 years ago

  • Sprint Focus set to Remote Sprint
Actions #15

Updated by Michael Oehlhof over 9 years ago

  • Assignee set to Michael Oehlhof
Actions #16

Updated by Gerrit Code Review over 9 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

Actions #17

Updated by Michael Oehlhof over 9 years ago

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

Updated by Riccardo De Contardi about 7 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF