Tickets :: [#9853] More intelligent part naming if name is not specified in the message

6.0.0-beta1

11/4/25

Summary	More intelligent part naming if name is not specified in the message
Queue	IMP
Queue Version	Git master
Type	Enhancement
State	Resolved
Priority	1. Low
Owners	slusarz (at) horde (dot) org
Requester	slusarz (at) horde (dot) org
Created	04/08/2011 (5324 days ago)
Due
Updated	04/11/2011 (5321 days ago)
Assigned
Resolved	04/11/2011 (5321 days ago)
Milestone
Patch	No

04/11/2011 06:53:45 PM	Michael Slusarz	Comment #12 State ⇒ Resolved	Reply to this comment
I'd really like to move the naming determination to within the MIME Viewers itself, so they can override as necessary. But this is only feasible once we allow on-demand loading of content (Request #9878). So I'm going to close this ticket and make note of this over there.

04/11/2011 08:16:50 AM	Jan Schneider	Comment #11	Reply to this comment
Am I missing something here? It is a text part - either a text/plain part or text/html part will be displayed. Eh, yes, of course that's correct. I guess I expected "Text" vs. "HTML", which would be "Plain" vs. "HTML" really.

04/11/2011 05:40:37 AM	Git Commit	Comment #10	Reply to this comment
Changes have been made in Git for this ticket: ~~Bug #9853~~: Second parameter is optional 2 files changed, 4 insertions(+), 3 deletions(-) http://git.horde.org/horde-git/-/commit/a0b3fadfc3a4de866c1d976028f05bba2e5758ca

04/11/2011 05:33:36 AM	Michael Slusarz	Comment #9	Reply to this comment
One small bug: mixed parts are named like "Text part" even though there is only text/html and application/mp3 in it. Is this right? Here is another example showing incorrectly "Text part". Am I missing something here? It is a text part - either a text/plain part or text/html part will be displayed.

04/10/2011 02:56:27 PM	Git Commit	Comment #8	Reply to this comment
Changes have been made in Git for this ticket: ~~Bug #9853~~: Add missing parameter to getPartName() 2 files changed, 6 insertions(+), 6 deletions(-) http://git.horde.org/horde-git/-/commit/ce13baf7139b9edd8298c900c7ca67e2add5eb18

04/10/2011 01:24:30 PM	Jan Schneider	Comment #7 New Attachment: bug9853.eml	Reply to this comment
One small bug: mixed parts are named like "Text part" even though there is only text/html and application/mp3 in it. Is this right? Here is another example showing incorrectly "Text part".

04/10/2011 01:20:33 PM	Jan Schneider	Comment #6	Reply to this comment
Feedback welcome. I like it.

04/10/2011 10:17:49 AM	vilius (at) lnk (dot) lt	Comment #5	Reply to this comment
Also some errors from the logs: 2011-04-10T13:01:50+03:00 WARN: HORDE [imp] PHP ERROR: Missing argument 2 for IMP_Contents::getPartName(), called in /var/www/html/horde/imp/view.php on line 132 and defined [pid 30757 on line 1098 of "/var/www/html/horde/imp/lib/Contents.php"] 2011-04-10T13:01:50+03:00 NOTICE: HORDE [imp] PHP ERROR: Undefined variable: descrip [pid 30757 on line 1101 of "/var/www/html/horde/imp/lib/Contents.php"]

04/10/2011 09:55:03 AM	vilius (at) lnk (dot) lt	Comment #4	Reply to this comment
One small bug: mixed parts are named like "Text part" even though there is only text/html and application/mp3 in it. Is this right?

04/08/2011 07:50:21 PM	Michael Slusarz	Comment #3 Assigned to Michael Slusarz State ⇒ Feedback	Reply to this comment
Feedback welcome.

04/08/2011 07:50:07 PM	Git Commit	Comment #2	Reply to this comment
Changes have been made in Git for this ticket: ~~Request #9853~~: More intelligent part naming if name is not specified in the message 4 files changed, 37 insertions(+), 12 deletions(-) http://git.horde.org/horde-git/-/commit/52c16c8b1e222fb18aa95334e019333b2ae2942b

04/08/2011 07:49:12 PM	Michael Slusarz	Comment #1 State ⇒ Accepted Patch ⇒ No Milestone ⇒ Queue ⇒ IMP Summary ⇒ More intelligent part naming if name is not specified in the message Type ⇒ Enhancement Priority ⇒ 1. Low	Reply to this comment
Replaces old "Unnamed Message Part"