Closed
Bug 208561
Opened 23 years ago
Closed 22 years ago
simple grammar error in xslt failure message
Categories
(Core :: XSLT, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: e_mayilme, Assigned: peterv)
Details
Attachments
(1 file)
|
2.25 KB,
patch
|
axel
:
review+
peterv
:
superreview+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.4) Gecko/20030529
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.4) Gecko/20030529
"Parsing a XSLT stylesheet failed."
correctly should be writtten as
"Parsing an XSLT stylesheet failed."
(x is pronounced with a beginning vowel sound--an is used before all words with
a beginning bowel sound)
Reproducible: Always
Steps to Reproduce:
1.
2.
3.
| Reporter | ||
Comment 1•23 years ago
|
||
oops: an is used before all words with a beginning _vowel_ sound.
Comment 2•23 years ago
|
||
Grammatical errors are Severity: trivial.
Reporter: Can you provide a public test case?
Severity: normal → trivial
| Reporter | ||
Comment 3•23 years ago
|
||
try this page:
http://www.providenceproject.com/fmxml/fmpro.xml
I don't know if the xml is right or not, but it does show the grammatical
mistake in the error message (Mozilla 1.4 rc1 on MacOSX). Perhaps grep -r 'a
XSLT' in the Mozilla tree would find the source file, and changing it to "an
XSLT" would then be easy.
| Reporter | ||
Comment 4•22 years ago
|
||
Here are patches to a few files with the spelling mistake:
I checked out the whole CVS to do this, so pls don't ignore.
The patch is for these files:
mozilla/extensions/transformiix/resources/buster/ReadMe
mozilla/extensions/transformiix/resources/xslt.properties
Those were the first I found when grepping, so I don't know if there are more.
nb: I could not find (quickly) a guideline for the correct patch format, so I
used -pru. Should someone add a bug to have this be mentioned directly in the
first Bugzilla fact sheet? (Forget it: I just found it, "cvs diff -u >
mypatch.diff," although it indirectly assumes one is using CVS.)
Updated•22 years ago
|
Attachment #126262 -
Flags: review+
| Assignee | ||
Comment 5•22 years ago
|
||
Comment on attachment 126262 [details] [diff] [review]
a patch for the grammar error, i.e, "a xslt" / "a xul"
>+The buster is an XUL interface to the conformance tests shipped as part of
XUL is pronounced "zool" so I am not sure about this change.
Attachment #126262 -
Flags: superreview+
Comment 6•22 years ago
|
||
Hixie it's "a XUL document" with "an XBL binding"
Hixie i say that as the editor of both those specs :-)
Hixie (well, one of the editors)
checked in, thanx for the fix.
Status: UNCONFIRMED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•