Closed Bug 53875 Opened 24 years ago Closed 24 years ago

stringBundleBindings.xml uses obsolete <argument> element

Categories

(Core :: XBL, defect, P3)

defect

Tracking

()

RESOLVED FIXED

People

(Reporter: jrgmorrison, Assigned: jag+mozbugs)

References

()

Details

(Keywords: regression, Whiteboard: [nsbeta3-][rtm-])

Attachments

(1 file)

stringBundleBindings.xml uses obsolete <argument> element, which breaks this
XBL. This needs to be changed to the new <parameter> syntax to make this work
again. nominating: nsbeta3,rtm as this is a simple, safe fix, to make code
work again, with minimum risk (one-line XBL change to conform to new syntax).

  8     <implementation name="XStringBundle">
  9       
 10       <method name="getString">
 11         <argument name="aStringKey"/>
 12         <body>
 13         <![CDATA[
 14           try {
 15
We don't need this for beta 3.  Why should we stop ship for this?  What problem
does it cause?
Whiteboard: [nsbeta3-]
Well, it happens to stop me from using this sexy xbl in the filepicker.
Of course, I could just use the regular stringbundle stuff, but this is way
sexier (no need to include strres.js everywhere meaning less bloat). And the fix
seems to be s/argument/parameter/, for which I'll attach a patch.
keywords, cc
Keywords: patch, review
nav triage team:
we don't see how this effects the user, rtm-
Whiteboard: [nsbeta3-] → [nsbeta3-][rtm-]
Target Milestone: --- → Future
Taking this.
Assignee: ben → disttsc
Target Milestone: Future → ---
r=hyatt
Status: NEW → ASSIGNED
Keywords: review
Checked in on the trunk. Since this bug is nsbeta3-, rtm-, I'm going to mark it
fixed.
duh, really marking it now.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: