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: