TYPO3 Core

Patch set 1 of change Ie645dca8b7605896b52bc0982586121cef4dd991 has been pushed to the review server.
It is available at http://review.typo3.org/3963

Commit message:

[FEATURE] Refactor t3lib_div::getUrl()

* Move non HTTP(S) parts of cURL into own method
* Deprecate this method
* Use t3lib_http_Request for HTTP(S) handling
* Move file handling into own method

Addionally fixed deprecation notizes and some spelling mistakes
around getUrl().

Now included left out deprecation warnings in content_default.php.

Change-Id: Ie645dca8b7605896b52bc0982586121cef4dd991
Resolves: #28626

• I do not think we should care for non HTTP(S) URLs. If anyone still needs this functionality, he should move this out of t3lib_div until TYPO3 4.8. I won't.
• I tested this with current linkvalidator, however please point me to any spot where the code now behaves differently.
• This should fix a bunch of known bugs/problems too.

Patch set 2 of change Ie645dca8b7605896b52bc0982586121cef4dd991 has been pushed to the review server.
It is available at http://review.typo3.org/3963

Patch set 3 of change Ie645dca8b7605896b52bc0982586121cef4dd991 has been pushed to the review server.
It is available at http://review.typo3.org/3963

Please change from under review to accepted and from 4.7 to 4.8.

I have no time to work on this and feature freeze is soon.

The problem is the behavior of the current function. Depending on the arguments and the environment (cURL?) it behaves totally different.

This is hard to refactor and there are many points to consider. So the discussion of the (abandoned) change.

This is still on my list, but not in focus as I consider this not to be urgent.