Feature #32543
closed
Some improvements for notification mail - new marker, list for multiple elements
Added by Sonja Schubert almost 13 years ago.
Updated over 9 years ago.
Description
It would be nice to have some improvements for the workspace notification mails sent by version extension.
- Nice list of elements in email if multiple elements were sent to next stage in one run
- new marker ELEMENT_NUMBER so numerable the list of elements
- new pageTsConfig option cropPathTitlesAfter to crop titles in email after number of chars
I attached the modified t3x of the version extension with a tested solution. I know that it is not nice to just attach the t3x and do not create patches. I am very sorry but I have no time this week to create these patches, but I will do this later.
Nevertheless some feedback if you like these features in general would be nice.
Files
- Status changed from New to Accepted
Sounds good... I'll try to bring up the patch for it to Gerrit for you.
- Assignee set to Tolleiv Nietsch
- Target version set to 4.7.0
It seems that right now things only just accidentally go well when changing the stage of mutliple records:
In tx_version_tcemain:
When multiple records are selected, $elementName is formatted like this: "pages:100, pages:101, pages:102" etc..
So this line:
list($elementTable, $elementUid) = explode(':', $elementName);
Creates an $elementUid: "100, pages:101, pages:102"
Luckily then comes this line:
$elementUid = intval($elementUid);
I always wondered why the ###ELEMENT_NAME### held multiple records and all other markers didn't :-)
Hi Sonja,
I have moved your changes to a separately installable extension using XCLASS.
I have also improved your fix on a number of points:
- Merged your code for multiple records with the existing code
- Also fixed the behavior of the old markers
- Made it compatible with both 4.5 and 4.6
As in your fix, this extension also includes this patch: https://review.typo3.org/#change,4519
This fix should be fully compatible with all old style markers and the old TSConfig options. I haven't fully tested all scenarios however.
Tolleiv, I could make a patch out of this and push it to Gerrit. Would that be a good idea, or would you rather have a look at it yourself?
- Target version deleted (
4.7.0)
- Status changed from Accepted to Under Review
I have created a proper patch for 6.0. I have submitted it as a feature, even though I think the behavior is actually broken in earlier versions too. But let's first get this patch into 6.0 and then see what can be backported. Patch for issue #26615 is committed in a separate patch.
- Category changed from New Features to Workspaces
- Project changed from 624 to TYPO3 Core
- Category changed from Workspaces to Workspaces
- Status changed from Under Review to Needs Feedback
Hi,
as this issue is very old. Is this feature missing in newer versions of TYPO3 CMS (6.2.9)?
If yes, can you provide a newer patch against master by using Gerrit? http://wiki.typo3.org/CWT
- Target version set to 7.5
- Status changed from Needs Feedback to Closed
- Assignee deleted (
Tolleiv Nietsch)
No feedback within the last 90 days => closing this issue.
If you think that this is the wrong decision or experience this issue again, then please write to the mailing list typo3.teams.bugs with issue number and an explanation or open a new ticket and add a relation to this ticket number.
Also available in: Atom
PDF