Skip to content

Commit 30e3395

Browse files
committed
GANReviewTool: don't add |class= to non banner shells
1 parent d192369 commit 30e3395

File tree

2 files changed

+13
-34
lines changed

2 files changed

+13
-34
lines changed

GANReviewTool/modules/GANReviewWikicodeGenerator.js

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -318,14 +318,8 @@ export class GANReviewWikicodeGenerator {
318318
}
319319

320320
changeWikiProjectArticleClassToGA( talkWikicode ) {
321-
// TODO: need to rewrite this to handle the following test case: {{WikiProject Energy|importance=Mid}}. Should add |rating=GA
322-
323-
// replace existing |class=
321+
// replace existing |class=. this should handle most cases. GA talk pages usually already have a rating.
324322
talkWikicode = talkWikicode.replace( /(\|\s*class\s*=\s*)(a|b|c|start|stub|list|fa|fl)?(?=[}\s|])/gi, '$1GA' );
325-
326-
// add |class= to {{WikiProject}} templates containing no parameters
327-
talkWikicode = talkWikicode.replace( /(\{\{WikiProject [^|}]+)(\}\})/gi, '$1|class=GA$2' );
328-
329323
return talkWikicode;
330324
}
331325

GANReviewTool/tests/GANReviewWikicodeGenerator.test.js

Lines changed: 12 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -171,21 +171,6 @@ describe( 'getPassWikicodeForTalkPage(talkWikicode, reviewTitle, gaSubpageShortT
171171
`{{GA|~~~~~|topic=Media and drama|page=1|oldid=1111}}
172172
{{WikiProject Television|class=GA}}
173173
{{WikiProject LGBT studies|class=GA}}
174-
`;
175-
expect( wg.getPassWikicodeForTalkPage( talkWikicode, reviewTitle, gaSubpageShortTitle, oldid ) ).toBe( output );
176-
} );
177-
178-
test( 'Should change {{WikiProject}} template with no parameters to include |class=GA', () => {
179-
const talkWikicode =
180-
`{{GA nominee|17:35, 8 June 2022 (UTC)|nominator=[[User:Underclass King|Underclass King]] ([[User talk:Underclass King|talk]])|page=1|subtopic=Television|status=onreview|note=}}
181-
{{WikiProject Television}}
182-
`;
183-
const reviewTitle = 'Talk:Seriously, Dude, I\'m Gay/GA1';
184-
const gaSubpageShortTitle = 'Media and drama';
185-
const oldid = 1111;
186-
const output =
187-
`{{GA|~~~~~|topic=Media and drama|page=1|oldid=1111}}
188-
{{WikiProject Television|class=GA}}
189174
`;
190175
expect( wg.getPassWikicodeForTalkPage( talkWikicode, reviewTitle, gaSubpageShortTitle, oldid ) ).toBe( output );
191176
} );
@@ -1561,25 +1546,25 @@ Test Test`;
15611546
} );
15621547

15631548
describe( 'changeWikiProjectArticleClassToGA(talkWikicode)', () => {
1564-
test( 'normal', () => {
1549+
test( 'change class in {{WikiProject banner shell}}', () => {
15651550
const talkWikicode =
1566-
`{{WikiProject banner shell|blp=yes|1=
1567-
{{WikiProject Biography|living=yes|needs-infobox=no|needs-photo=yes|listas=Amamiya, Sora|class=Start|musician-work-group=yes|musician-priority=}}
1568-
{{WikiProject Anime and manga|class=Start|biography=yes|importance=low|listas=Amamiya, Sora}}
1569-
{{WikiProject Japan|class=Start|biography|yes|importance=low|listas=Amamiya, Sora}}
1570-
{{WikiProject Women in Music |class=Start |importance=Low|listas=Amamiya, Sora}}
1551+
`{{WikiProject banner shell|blp=yes|class=Start|1=
1552+
{{WikiProject Biography|living=yes|needs-infobox=no|needs-photo=yes|listas=Amamiya, Sora|musician-work-group=yes|musician-priority=}}
1553+
{{WikiProject Anime and manga|biography=yes|importance=low|listas=Amamiya, Sora}}
1554+
{{WikiProject Japan|biography|yes|importance=low|listas=Amamiya, Sora}}
1555+
{{WikiProject Women in Music |importance=Low|listas=Amamiya, Sora}}
15711556
}}`;
15721557
const output =
1573-
`{{WikiProject banner shell|blp=yes|1=
1574-
{{WikiProject Biography|living=yes|needs-infobox=no|needs-photo=yes|listas=Amamiya, Sora|class=GA|musician-work-group=yes|musician-priority=}}
1575-
{{WikiProject Anime and manga|class=GA|biography=yes|importance=low|listas=Amamiya, Sora}}
1576-
{{WikiProject Japan|class=GA|biography|yes|importance=low|listas=Amamiya, Sora}}
1577-
{{WikiProject Women in Music |class=GA |importance=Low|listas=Amamiya, Sora}}
1558+
`{{WikiProject banner shell|blp=yes|class=GA|1=
1559+
{{WikiProject Biography|living=yes|needs-infobox=no|needs-photo=yes|listas=Amamiya, Sora|musician-work-group=yes|musician-priority=}}
1560+
{{WikiProject Anime and manga|biography=yes|importance=low|listas=Amamiya, Sora}}
1561+
{{WikiProject Japan|biography|yes|importance=low|listas=Amamiya, Sora}}
1562+
{{WikiProject Women in Music |importance=Low|listas=Amamiya, Sora}}
15781563
}}`;
15791564
expect( wg.changeWikiProjectArticleClassToGA( talkWikicode ) ).toBe( output );
15801565
} );
15811566

1582-
test( 'don\'t change |class in non-WikiProject templates', () => {
1567+
test( "don't change class in non-WikiProject templates", () => {
15831568
const talkWikicode =
15841569
`{{WikiProject Anime and manga|class=Start|biography=yes|importance=low|listas=Amamiya, Sora}}
15851570
{{Random template|class=DontChangeMe}}`;

0 commit comments

Comments
 (0)