TYPO3 Core

I have added two diffs.
The first one makes the icons in the new shortcut menu reflected the actual record type. So the if you have a shortcut to at news category record, you will have the news category icon instead of the pencil icon.

The second one does the exact same, plus
changes to the shortcut label when shortcuts are created.
So if you create a shortcut to a 'Website User' record on the page 'storage folder' the label will be: 'Edit Website User (storage folder)'.
If you create a shortcut to a blank 'Website User' record on the page 'storage folder', the label will be: 'Create Website User (storage folder)'.
The result is a more descriptive shortcut label, but also a longer label. So it might impose some problems with the current menu width....

bug_7267_2a.diff as a addition for bug_7267_2.diff
I have also added a diff on /typo3/sysext/lang/locallang_misc.xml
I have added the label 'shortcut_create'.

Hi Stig, I cleaned up your patch a bit. However when editing a content element in Web->page and set a shortcut to it I don't get a shortcut icon... could you please check that?

EDIT: Other then that: good job!

Hi Ingo
Thanks for checking out the patch.
I will look into that as soon as possible.
Probably either sunday or monday.

Hi Ingo
Your cleansing of my patch seems very reasonable.
The Web->page bug in the patch is corrected easy.
You added a check on the $shortcut['recordid'] if there was a comma. If that was the case you remove it.
But $shortcut['recordid'] need to be used in its original form on the next line.
So I just moved your "comma check" below.

Btw. do you know what is the intention with this comma?

No idea, I only can imagine that it's related to the edit-multiple-records-at-once feature the list view offers.

checked and it works now.

Only one issue left: the icons are the "old" original ones and not the ones from skins like t3skin.

ok, solved by adding

$icon = t3lib_iconWorks::skinImg($this->backPath, $icon, '', 1);

at the end of the first case in getShortcutIcon()

Ok.
About the comma. It is for a comma separated list when editing multiple records.
So I suggest that we change:
'WHERE' => 'uid = '.$recordid.' '.$permissionClause.
to
'WHERE' => 'uid IN ('.$recordid.') '.$permissionClause.
also in getShortcutIcon()

unfortunately that doesn't seem to work when creating a shortcut to "editing a content element" in the page module.

EDIT:
could you provide a new patch with your proposed changes, maybe I've overseen something...

just noticed: even a shortcut to editing multiple records at once should work in the case we only strip the last comma

I attached what works for me now

I will probably not have time to look more at this until next week.

committed to trunk