The default bug view has changed. See this FAQ.

Drop-down 'View:' menu displays 'undefined' when you try and use Tags, or Custom Views

RESOLVED FIXED in seamonkey2.1a1

Status

SeaMonkey
MailNews: Message Display
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: Ben, Assigned: Zane U. Ji)

Tracking

({fixed-seamonkey2.0.1})

Trunk
seamonkey2.1a1
x86
Windows XP
fixed-seamonkey2.0.1

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b3pre) Gecko/20081202 SeaMonkey/2.0a2
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b3pre) Gecko/20081202 SeaMonkey/2.0a2

If you try and use any of the following View message options: Tags, Custom Views


(View menu, then Messages submenu) or the drop-down view menu, and then close e-mail client, then re-open seamonkeys e-mail client, the drop-down 'View:' menu displays the word 'undefined'

It should also be noted, that if you choose any of the Tags (e.g. 'Important'), the drop-down 'View:' menu will immediately display the tag you chose, e.g.
'Important'

If you then close the e-mail client, and then re-open the e-mail client, the drop-down 'View:' menu will then display the word 'undefined'

Unlike the Custom Views option which will make the drop-down 'View:' menu display the word 'undefined' immediately.


If you make a New personal Custom View, and give a certain criteria, then choose to use your custom view, it will work, even though the 'View:'
drop-down menu displays the word 'undefined'.


The only e-mail that will display is the one that meets the custom view requirement i set, even though the drop-down       

'View:' menu displays the word 'undefined' in it.

Reproducible: Always

Steps to Reproduce:
To Reproduce Symptoms:

1. Send yourself an e-mail that has the words (for example) 'i really like technicians because they know a lot' in the subject line.

2. For your 'Inbox' Folder, create a new custom view called 'Bens Custom' from the drop-down 'View:' menu, and use the following criteria:

Goto Customize, then goto New, and create the following:



Drop-down menu 1:     Drop-down menu 2:     Data Entry Field:

Subject               Contains              technicians



3. goto the drop-down 'View:' menu, then to the 'Custom Views' sub-menu, and choose 'Bens Custom'

4. Quit seamonkey e-mail client (or seamonkey completely), and re-enter seamonkeys e-mail client


Actual Results:  
A. The drop-down 'View:' menu will display 'undefined' in it.
B. Only the E-mail that has the word 'technicians' in the subject line is displayed even though the drop-down 'View:' menu displays the word 'undefined'

Expected Results:  
Expected the drop-down 'View:' menu to show the name of the chosen Tag, or the Custom View options name (including Newly Created Custom Views)

Sometimes if you click a couple of times (sometimes between 1 to 31 times) between the e-mail Folder you set the View Criteria up on and the Inbox and/or Drafts folder with the mouse, the drop-down 'View:' menu might suddenly correctly
display the actual name of the Custom View when you have clicked on the e-mail folder that you decided to set the custom view up on (in this case E-mail folder = 'Inbox', Custom View  = 'Bens Custom').

NOTE: Not sure wether this constitutes a potential privacy and/or a security
      issue. User may want to use view message filter to filter out certain
      e-mails so as to not have other people in public areas see certain
      e-mail addresses or names of people, but does not mind if public can see
      certain e-mail addresses. E-mail names and addressess could have
      sensitive information not allowed to be seen by others.
I doubt this needs to remain a hidden security bug.  The |undefined| being displayed is probably from JavaScript land and is probably not a memory safety problem.  I'll let someone else confirm, though.
Group: core-security
Version: unspecified → Trunk

Comment 2

8 years ago
Confirming on SeaMonkey trunk.
Status: UNCONFIRMED → NEW
Ever confirmed: true
(Assignee)

Comment 3

8 years ago
I found out how to solve the problem. However, I need more days to test my patch.
(Assignee)

Comment 4

8 years ago
Created attachment 408404 [details] [diff] [review]
Don't mistake an attribute for a property
[Checkin: Comment 6]
[comm-1.9.1 Checkin: Comment 7]
Attachment #408404 - Flags: superreview?(bienvenu)
Attachment #408404 - Flags: review?(bienvenu)

Updated

8 years ago
Assignee: nobody → ZaneUJi
Status: NEW → ASSIGNED

Comment 5

8 years ago
Comment on attachment 408404 [details] [diff] [review]
Don't mistake an attribute for a property
[Checkin: Comment 6]
[comm-1.9.1 Checkin: Comment 7]

r+moa=me
Both bug and patch are SeaMonkey only, thus no need to bother the TB folks. :)

Thanks for patching this!
Attachment #408404 - Flags: superreview?(bienvenu)
Attachment #408404 - Flags: superreview+
Attachment #408404 - Flags: review?(bienvenu)
Attachment #408404 - Flags: review+

Updated

8 years ago
Keywords: checkin-needed

Updated

8 years ago
Attachment #408404 - Flags: approval-seamonkey2.0.1+
Comment on attachment 408404 [details] [diff] [review]
Don't mistake an attribute for a property
[Checkin: Comment 6]
[comm-1.9.1 Checkin: Comment 7]


http://hg.mozilla.org/comm-central/rev/436dd822f88b
Attachment #408404 - Attachment description: Don't mistake an attribute for a property → Don't mistake an attribute for a property [Checkin: Comment 6]
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Whiteboard: [c-n: c-1.9.1]
Target Milestone: --- → seamonkey2.1a1

Comment 7

8 years ago
Comment on attachment 408404 [details] [diff] [review]
Don't mistake an attribute for a property
[Checkin: Comment 6]
[comm-1.9.1 Checkin: Comment 7]

http://hg.mozilla.org/releases/comm-1.9.1/rev/050bea4b1e87
Attachment #408404 - Attachment description: Don't mistake an attribute for a property [Checkin: Comment 6] → Don't mistake an attribute for a property [Checkin: Comment 6] [comm-1.9.1 Checkin: Comment 7]

Updated

8 years ago
Keywords: checkin-needed → fixed-seamonkey2.0.1
Whiteboard: [c-n: c-1.9.1]
You need to log in before you can comment on or make changes to this bug.