[TYPO3-core] RFC: #14430: disable_controls = list makes fields 'write_once'
Alexander Stehlik
alexander.stehlik at googlemail.com
Mon Feb 7 08:36:16 CET 2011
Hi Steffen,
I finally had some time to look at this issue.
I tested the patch and the idea seems to work. I modified it a bit to
move the code to the right location. Since this change is only relevant
for file upload fields, i moved it to
t3lib_TCEforms::getSingleField_typeGroup.
There are two things I am not sure about:
Should this behaviour appy for all file upload fields that have maxitems
set to 1 or only to those that additionally have the delete button disabled?
At the moment this patch only changes the behaviour for the direct file
upload in the Backend form. If you use the Element Browser, you are
still not able to change a file if delete button is hidden. Is this the
desired behaviour? If not, another solution is needed. Maybe a hook in
the fieldChanged function that can call additional JS code if required.
Please tell me what you think of it and I will provide another patch if
needed.
Kind regards,
Alex
Am 08.01.2011 14:03, schrieb Steffen Gebert:
> Hi Alexander,
>
> > 2. In my opinion it has a better usability, because you don't have to
> > delete the image first and than upload a new one but you can
> > directly upload a new file and replace the existing one.
>
> I would like to see this behavior as the default one - it's more logical
> and easier for the user.
> Adding a configuration option is IMHO a no-go, not only because of the
> different behavior.
>
> The patch didn't apply anymore to trunk. I also changed it a bit to a)
> not rely on the configuration option and b) prevent duplicated code.
>
> However it doesn't really work.
>
> Could you please have a look? As we're very close to the next release, I
> would ask you to come up with this again after the release of 4.5.
>
> Thanks!
>
> Kind regards
> Steffen
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: typo3-14432b-config_replace_single-v4.diff
Type: text/x-patch
Size: 833 bytes
Desc: not available
URL: <http://lists.typo3.org/pipermail/typo3-team-core/attachments/20110207/fb09ffdd/attachment.bin>
More information about the TYPO3-team-core
mailing list