mirror of
https://gerrit.wikimedia.org/r/mediawiki/extensions/CiteThisPage
synced 2024-11-28 01:10:00 +00:00
Fix regression -- don't forbid all names which contain integers, but rather those which are them.
Note that that's still a bad check probably; the ID issue is that a digit isn't a valid first char.
This commit is contained in:
parent
35147ec2bb
commit
57a761acb5
2
Cite.php
2
Cite.php
|
@ -159,7 +159,7 @@ function wfCite() {
|
||||||
return $this->error( 'cite_error_ref_no_key' );
|
return $this->error( 'cite_error_ref_no_key' );
|
||||||
}
|
}
|
||||||
|
|
||||||
if( preg_match( '/[0-9]+/', $key ) ) {
|
if( preg_match( '/^[0-9]+$/', $key ) ) {
|
||||||
# Numeric names mess up the resulting id's, potentially produ-
|
# Numeric names mess up the resulting id's, potentially produ-
|
||||||
# cing duplicate id's in the XHTML. The Right Thing To Do
|
# cing duplicate id's in the XHTML. The Right Thing To Do
|
||||||
# would be to mangle them, but it's not really high-priority
|
# would be to mangle them, but it's not really high-priority
|
||||||
|
|
Loading…
Reference in a new issue