[TYPO3-core] RFC #12679: Pagemodule don't show clipboard paste icon when there is no content

[Reinhard Führicht]

Am 2010-07-14 11:12, schrieb Reinhard Führicht:
> Am 2010-07-13 23:12, schrieb Steffen Kamper:
>> Hi Reinhard,
>>
>> thanks for your patch.
>> While reading i see the same code as in the list module, which makes me
>> think: can't we move this code to the clipboard class? I mean the
>> copy/paste/move links. It would reduce the code and we have it at the
>> one place where it belongs to, so usage in other modules would be
>> possible too without writing hundreds of code lines (or copy&paste from
>> existing modules)
>> What do you think?
>>
>> vg Steffen
>
> Attached v9 of the patch.

Now it is attached :-)

>
> - Moved getPasteLink() + getCopyLink() + getCutLink() to t3lib_clipboard
> - Chnaged code in page module to use these methods
> - Changed code in list module to use these methods too
>
> I am not sure about the two methods getCopyLink() and getCutLink(). They
> could easily be merged to a single method. But this way it is more clear
> what they are doing.
>
> I tested the code in page module and list module using different tables.
> I hope I didn't miss a bug.
>
> Regards,
> Reinhard
>
>

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 12679_v9.diff
Type: text/x-patch
Size: 17660 bytes
Desc: not available
URL: <http://lists.typo3.org/pipermail/typo3-team-core/attachments/20100714/f2650aed/attachment-0001.bin>