[TYPO3-core] RFC #11433: touch(): Utime failed in install tool

Martin Kutschker masi-no at spam-typo3.org
Tue Sep 29 15:40:32 CEST 2009


Ernesto Baschny [cron IT] schrieb:
> Martin Kutschker schrieb:
>> Ernesto Baschny [cron IT] schrieb:
>>> Steffen Gebert schrieb:
>>>> On Tue, 29 Sep 2009 13:46:01 +0200, Frans Saris <franssaris at gmail.com>
>>>> wrote:
>>>>
>>>>> still wrong variable name: $enableInstallTool => $enableInstallToolFile
>>>> grml.. not my day today, really not :D
>>>>
>>>> Attached v4.
>>>>
>>>> This time I checked weather modification time is really updated (and
>>>> works - if file created by web server user).
>>>>
>>>> Thanks for your patience ;)
>>> I like the idea of solving this, as we also have this "problem" from
>>> time to time, and it is just annoying.
>>>
>>> But the fix is not correct, because checking for "is_writeable" is not
>>> really interesting for touching a file. You can only touch a file that
>>> you own (or if you are root), so even if the file is "world writeable"
>>> it is still not toucheable if it is owned by root (or by the FTP user).
>>>
>>> So the most simply solution would be to ignore the warning with "@".
>> if (!@touch($path)) {
>>  // notify user, log, raise hell - umh - exception, etc
>> }
> 
> Yes, but this would be the same as we had now (warning). Of course
> better that it is not a PHP warning.

I thought that was the isse. Someone wnated the PHP warning to go away. That is what the original
patch did, anway.

Masi


More information about the TYPO3-team-core mailing list