Closed Bug 238675 Opened 20 years ago Closed 20 years ago

templatization went amuck in "reassign_to_empty" error

Categories

(Bugzilla :: User Interface, defect)

Product:
Bugzilla
Component:
User Interface
Type:
defect
Priority:
Not set
Severity:
minor

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Bugzilla 2.18

People

(Reporter: timeless, Assigned: apeiron)

References

Details

Attachments

(1 file, 1 obsolete file)

Here's an updated patch based on current CVS.
936 bytes, patch
justdave
: review+
Details | Diff | Splinter Review 
626   [% ELSIF error == "reassign_to_empty" %]
627     [% title = "Illegal Reassignment" %]
628     You cannot reassign to [% terms.abug %] to nobody. Unless you
629     intentionally cleared out the "Reassign [% terms.bug %] to"
630     field, [% Param("browserbugmessage") %]

reassign to a bug to nobody.

um sure. fixme.
Attached patch fixme'd. (obsolete) — — Splinter Review 

    
  
Assignee: myk → apeiron
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows XP → All
Hardware: PC → All
Target Milestone: --- → Bugzilla 2.18

    
  
Status: NEW → ASSIGNED
Flags: approval?

    
  

        Attachment #144777 -
        Flags: review+

    
    

    
  
Comment on attachment 144777 [details] [diff] [review]
fixme'd.

What if we went ahead and improved this phrase? The reason I'm saying this is
that "nobody" is now ambiguous given that certain sites (bmo for instance) use
nobody@foo.com accounts as bug assignees. My suggestion:

'To reassign [% terms.abug %], you must provide an address for the new owner.
If you did not intentionally clear out the "Reassign...'

        Attachment #144777 -
        Flags: review-

    
    

    
  
Comment on attachment 144777 [details] [diff] [review]
fixme'd.

Per kiko's comments.

        Attachment #144777 -
        Flags: review+
Flags: approval?

    
    

    
  
Chris, do you have a few minutes to update this, or would you like someone else
to tackle it?

    
    

    
  


  
This patch reflects changes inspired by comments made after I posted the first
patch.

        Attachment #148367 -
        Flags: review+
Flags: approval+

    
  

        Attachment #144777 -
        Attachment is obsolete: true

    
    

    
  
Checking in user-error.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/global/user-error.html.tmpl,v
 <--  user-error.html.tmpl
new revision: 1.61; previous revision: 1.60
done
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED

    
    

    
  
*** Bug 251288 has been marked as a duplicate of this bug. ***
QA Contact: matty_is_a_geek → default-qa

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: