r32492 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r32491‎ | r32492 | r32493 >
Date:10:09, 27 March 2008
Author:raymond
Status:old
Tags:
Comment:
* Fix a PHP notice when <references group="foo" /> is used but "foo" is not defined by any <ref>. New message 'cite_error_references_invalid_group'
* Add new message 'cite_error_references_invalid_parameters_group' to clarify error message when $wgAllowCiteGroups=true (default)
* Rephrase 'cite_error_references_no_backlink_label' a bit to make it clearer that 'cite_references_link_many_format_backlink_labels' is a MediaWiki message
Modified paths:
  • /trunk/extensions/Cite/Cite.i18n.php (modified) (history)
  • /trunk/extensions/Cite/Cite.php (modified) (history)

Diff [purge]

Index: trunk/extensions/Cite/Cite.i18n.php
@@ -1,6 +1,6 @@
22 <?php
33 /**
4 - * Internationalisation file for Ci;te extension.
 4+ * Internationalisation file for Cite extension.
55 *
66 * @addtogroup Extensions
77 */
@@ -18,16 +18,17 @@
1919 'cite_error_stack_invalid_input' => 'Internal error; invalid stack key. This should never occur.',
2020
2121 # User errors
22 - 'cite_error' => 'Cite error: $1',
23 - 'cite_error_ref_numeric_key' => 'Invalid <code>&lt;ref&gt;</code> tag; name cannot be a simple integer, use a descriptive title',
24 - 'cite_error_ref_no_key' => 'Invalid <code>&lt;ref&gt;</code> tag; refs with no content must have a name',
25 - 'cite_error_ref_too_many_keys' => 'Invalid <code>&lt;ref&gt;</code> tag; invalid names, e.g. too many',
26 - 'cite_error_ref_no_input' => 'Invalid <code>&lt;ref&gt;</code> tag; refs with no name must have content',
27 - 'cite_error_references_invalid_input' => 'Invalid <code>&lt;references&gt;</code> tag; no input is allowed, use
28 -<code>&lt;references /&gt;</code>',
29 - 'cite_error_references_invalid_parameters' => 'Invalid <code>&lt;references&gt;</code> tag; no parameters are allowed, use <code>&lt;references /&gt;</code>',
30 - 'cite_error_references_no_backlink_label' => "Ran out of custom backlink labels, define more in the \"''cite_references_link_many_format_backlink_labels''\" message",
31 - 'cite_error_references_no_text' => 'Invalid <code>&lt;ref&gt;</code> tag; no text was provided for refs named <code>$1</code>',
 22+ 'cite_error' => 'Cite error: $1',
 23+ 'cite_error_ref_numeric_key' => 'Invalid <code>&lt;ref&gt;</code> tag; name cannot be a simple integer, use a descriptive title',
 24+ 'cite_error_ref_no_key' => 'Invalid <code>&lt;ref&gt;</code> tag; refs with no content must have a name',
 25+ 'cite_error_ref_too_many_keys' => 'Invalid <code>&lt;ref&gt;</code> tag; invalid names, e.g. too many',
 26+ 'cite_error_ref_no_input' => 'Invalid <code>&lt;ref&gt;</code> tag; refs with no name must have content',
 27+ 'cite_error_references_invalid_input' => 'Invalid <code>&lt;references&gt;</code> tag; no input is allowed, use <code>&lt;references /&gt;</code>',
 28+ 'cite_error_references_invalid_parameters' => 'Invalid <code>&lt;references&gt;</code> tag; no parameters are allowed, use <code>&lt;references /&gt;</code>',
 29+ 'cite_error_references_invalid_parameters_group' => 'Invalid <code>&lt;references&gt;</code> tag; parameter group is allowed only, use <code>&lt;references /&gt;</code> or <code>&lt;references group="..." /&gt;</code>',
 30+ 'cite_error_references_invalid_group' => 'Invalid <code>&lt;references group="$1" /&gt;</code> tag; group name "$1" not defined in <code>&lt;ref&gt;</code>',
 31+ 'cite_error_references_no_backlink_label' => 'Ran out of custom backlink labels, define more in the <nowiki>[[MediaWiki:Cite references link many format backlink labels]]</nowiki> message',
 32+ 'cite_error_references_no_text' => 'Invalid <code>&lt;ref&gt;</code> tag; no text was provided for refs named <code>$1</code>',
3233
3334 /*
3435 Output formatting
@@ -207,19 +208,21 @@
208209 * @author Raimond Spekking
209210 */
210211 $messages['de'] = array(
211 - 'cite_desc' => 'Ergänzt für Quellennachweise die <tt><nowiki><ref[ name=id]></nowiki></tt> und <tt><nowiki><references /></nowiki></tt>-Tags',
212 - 'cite_croak' => 'Fehler im Referenz-System. $1: $2',
213 - 'cite_error_key_str_invalid' => 'Interner Fehler: ungültiger $str und/oder $key. Dies sollte eigentlich gar nicht passieren können.',
214 - 'cite_error_stack_invalid_input' => 'Interner Fehler: ungültiger „name“-stack. Dies sollte eigentlich gar nicht passieren können.',
215 - 'cite_error' => 'Referenz-Fehler $1',
216 - 'cite_error_ref_numeric_key' => 'Ungültige <tt>&lt;ref&gt;</tt>-Verwendung: „name“ darf kein reiner Zahlenwert sein, benutze einen beschreibenden Namen.',
217 - 'cite_error_ref_no_key' => 'Ungültige <tt>&lt;ref&gt;</tt>-Verwendung: „ref“ ohne Inhalt muss einen Namen haben.',
218 - 'cite_error_ref_too_many_keys' => 'Ungültige <tt>&lt;ref&gt;</tt>-Verwendung: „name“ ist ungültig oder zu lang.',
219 - 'cite_error_ref_no_input' => 'Ungültige <tt>&lt;ref&gt;</tt>-Verwendung: „ref“ ohne Namen muss einen Inhalt haben.',
220 - 'cite_error_references_invalid_input' => 'Ungültige <tt>&lt;references&gt;</tt>-Verwendung: Es ist kein zusätzlicher Text erlaubt, verwende ausschließlich <tt><nowiki><references /></nowiki></tt>.',
221 - 'cite_error_references_invalid_parameters' => 'Ungültige <tt>&lt;references&gt;</tt>-Verwendung: Es sind keine zusätzlichen Parameter erlaubt, verwende ausschließlich <tt><nowiki><references /></nowiki></tt>.',
222 - 'cite_error_references_no_backlink_label' => 'Eine Referenz der Form <tt>&lt;ref name="…"/&gt;</tt> wird öfter benutzt als Buchstaben vorhanden sind. Ein Administrator muss <nowiki>[[MediaWiki:Cite references link many format backlink labels]]</nowiki> um weitere Buchstaben/Zeichen ergänzen.',
223 - 'cite_error_references_no_text' => 'Ungültiger <tt>&lt;ref&gt;</tt>-Tag; es wurde kein Text für das Ref mit dem Namen <tt>$1</tt> angegeben.',
 212+ 'cite_desc' => 'Ergänzt für Quellennachweise die <tt><nowiki><ref[ name=id]></nowiki></tt> und <tt><nowiki><references /></nowiki></tt>-Tags',
 213+ 'cite_croak' => 'Fehler im Referenz-System. $1: $2',
 214+ 'cite_error_key_str_invalid' => 'Interner Fehler: ungültiger $str und/oder $key. Dies sollte eigentlich gar nicht passieren können.',
 215+ 'cite_error_stack_invalid_input' => 'Interner Fehler: ungültiger „name“-stack. Dies sollte eigentlich gar nicht passieren können.',
 216+ 'cite_error' => 'Referenz-Fehler: $1',
 217+ 'cite_error_ref_numeric_key' => 'Ungültige <tt>&lt;ref&gt;</tt>-Verwendung: „name“ darf kein reiner Zahlenwert sein, benutze einen beschreibenden Namen.',
 218+ 'cite_error_ref_no_key' => 'Ungültige <tt>&lt;ref&gt;</tt>-Verwendung: „ref“ ohne Inhalt muss einen Namen haben.',
 219+ 'cite_error_ref_too_many_keys' => 'Ungültige <tt>&lt;ref&gt;</tt>-Verwendung: „name“ ist ungültig oder zu lang.',
 220+ 'cite_error_ref_no_input' => 'Ungültige <tt>&lt;ref&gt;</tt>-Verwendung: „ref“ ohne Namen muss einen Inhalt haben.',
 221+ 'cite_error_references_invalid_input' => 'Ungültige <tt>&lt;references&gt;</tt>-Verwendung: Es ist kein zusätzlicher Text erlaubt, verwende ausschließlich <tt><nowiki><references /></nowiki></tt>.',
 222+ 'cite_error_references_invalid_parameters' => 'Ungültige <tt>&lt;references&gt;</tt>-Verwendung: Es sind keine zusätzlichen Parameter erlaubt, verwende ausschließlich <tt><nowiki><references /></nowiki></tt>.',
 223+ 'cite_error_references_invalid_parameters_group' => 'Ungültige <code>&lt;references&gt;</code>-Verwendung: Nur der Parameter „group“ ist erlaubt, verwende <tt>&lt;references /&gt;</tt> oder <tt>&lt;references group="…" /&gt;</tt>',
 224+ 'cite_error_references_invalid_group' => 'Ungültige <code>&lt;references group="$1" /&gt;</code>-Verwendung: Gruppenname „$1“ wurde in keinem <tt>&lt;ref&gt;</tt> definiert',
 225+ 'cite_error_references_no_backlink_label' => 'Eine Referenz der Form <tt>&lt;ref name="…"/&gt;</tt> wird öfter benutzt als Buchstaben vorhanden sind. Ein Administrator muss <nowiki>[[MediaWiki:Cite references link many format backlink labels]]</nowiki> um weitere Buchstaben/Zeichen ergänzen.',
 226+ 'cite_error_references_no_text' => 'Ungültiger <tt>&lt;ref&gt;</tt>-Tag; es wurde kein Text für das Ref mit dem Namen <tt>$1</tt> angegeben.',
224227 );
225228
226229 /** Lower Sorbian (Dolnoserbski)
Index: trunk/extensions/Cite/Cite.php
@@ -350,21 +350,27 @@
351351 global $wgAllowCiteGroups;
352352
353353 $this->mParser = $parser;
354 -
355 - if ( $str !== null )
 354+
 355+ if ( $str !== null ) {
356356 return $this->error( 'cite_error_references_invalid_input' );
 357+ }
357358
358 -
359359 if ( isset( $argv['group'] ) and $wgAllowCiteGroups) {
360360 $group = $argv['group'];
361361 unset ($argv['group']);
362 -
363362 }
364 -
365 - if ( count( $argv ) )
366 - return $this->error( 'cite_error_references_invalid_parameters' );
367 - else
368 - return $this->referencesFormat($group);
 363+
 364+ if ( count( $argv ) ) {
 365+ if( $wgAllowCiteGroups ) {
 366+ return $this->error( 'cite_error_references_invalid_parameters_group' );
 367+ } else {
 368+ return $this->error( 'cite_error_references_invalid_parameters' );
 369+ }
 370+ } elseif ( $wgAllowCiteGroups && !isset( $this->mRefs[$group] ) ) {
 371+ return $this->error( 'cite_error_references_invalid_group', htmlspecialchars( $group ) );
 372+ } else {
 373+ return $this->referencesFormat( $group );
 374+ }
369375 }
370376
371377 /**

Follow-up revisions

RevisionCommit summaryAuthorDate
r33003Revert part of r32492 -- don't spew errors for <references/> when no <ref>s a...brion01:22, 9 April 2008

Status & tagging log