Module talk:Episode list/Archive 1
This is an archive of past discussions about Module:Episode list. Do not edit the contents of this page. If you wish to start a new discussion or revive an old one, please do so on the current talk page. |
Archive 1 |
Template-protected edit request on 14 July 2014
This edit request to Module:Episode list has been answered. Set the |answered= or |ans= parameter to no to reactivate your request. |
In response to User_talk:The_Mol_Man#Template:Episode_list, a small change I made to the wikitext() function should fix it. It's found in this revision of the sand box (it's safe to just replace the module with the code on the /sandbox). I have no idea what was causing the issue in the first place. It appears to have been some issue with how Lua is creating the wikitext. moluɐɯ 20:04, 14 July 2014 (UTC)
- A similar edit was made without knowledge of this request moluɐɯ 20:09, 14 July 2014 (UTC)
- That was me. The reason is didn't work is because the parser does not generate
<p>...</p>
tags unless the text starts on a new line. That is what I added. Your hack was quite over the top. Just remember that anything in wikitext() is sent straight to the parser and should therefor behave exactly as wikitext.-- [[User:Edokter]] {{talk}}
20:14, 14 July 2014 (UTC)- My hack also included something that would have decreased the spacing between the headers, though he seems to not have a problem with that anymore judging by my talk page. Other than that, it was just your edit with some redundant <p> tags. moluɐɯ 20:21, 14 July 2014 (UTC)
- That was me. The reason is didn't work is because the parser does not generate
Template-protected edit request on 5 December 2014
This edit request to Module:Episode list has been answered. Set the |answered= or |ans= parameter to no to reactivate your request. |
This module fails to place a space between the Title (which is rendered in quotes) and RTitle, in cases where the call includes both parameters. For example, the following strings are rendered at List of South Park episodes#Shorts, sketches, and other appearances (note the lack of a space after the closing quotation mark):
- "Magic: The Gathering"promotional bumper
- "Kenny's Dead"music video
- "The Aristocrats"sketch
While my Lua experience is modest, I suggest either of the following changes:
- ...to...
title_string = title_string..'"'..args.Title..'"'
- , or change...
title_string = title_string..'"'..args.Title..'" '
- ...to...
title_string = title_string..args.RTitle
title_string = title_string..' '..args.RTitle
Thanks in advance! – voidxor (talk | contrib) 23:10, 5 December 2014 (UTC)
- Above examples are crude. A better way is to only add a space when both args are present.
-- [[User:Edokter]] {{talk}}
00:05, 6 December 2014 (UTC)
True, so how about we insert the following at line #141 instead?
if args.Title and args.RTitle then
title_string = title_string..' '
end
– voidxor (talk | contrib) 05:59, 6 December 2014 (UTC)
- Not done: please make your requested changes to the module's sandbox first; see WP:TESTCASES. Test your proposal in the sandbox and then reactivate this request when you've fully tested it. Thanks. — {{U|Technical 13}} (e • t • c) 06:55, 17 December 2014 (UTC)
- Implemented in sandbox. Can someone test?
-- [[User:Edokter]] {{talk}}
10:29, 18 December 2014 (UTC)
- Implemented in sandbox. Can someone test?
- I setup a test by having Template:Episode list/sandbox invoke Module:Episode list/sandbox. Examples are already setup at Template:Episode list/testcases, and the only changes appear in the reference tests. Everything appears to be working properly. The only question is, are we willing to have a space between the closing quotation mark of an episode title, and the citation for that title—in cases where the RTitle parameter is used for a title reference instead of a raw title? – voidxor (talk | contrib) 00:00, 19 December 2014 (UTC)
- I had not considered that... But in those cases (as wel as the cases shown above), one can use RTitle for everything at once. The only downside being having to add the quotes yourself. In which case, so we still need the added spaces?
-- [[User:Edokter]] {{talk}}
09:46, 20 December 2014 (UTC)
- I had not considered that... But in those cases (as wel as the cases shown above), one can use RTitle for everything at once. The only downside being having to add the quotes yourself. In which case, so we still need the added spaces?
- My opinion at this time is that adding a space (as you have done in the sandbox) is not a perfect solution, but it's better than the status quo. I'd go ahead and add it, unless somebody else chimes in with a better proposal. – voidxor (talk | contrib) 02:28, 21 December 2014 (UTC)
- Done.
-- [[User:Edokter]] {{talk}}
13:24, 5 January 2015 (UTC)
- Done.
- This change needs to be reverted ASAP. There can be no space between Title and RTitle or AltTitle and RAltTitle. In the vast majority of cases, RTitle and RAltTitle are used to add references to episodes and the change now puts a space between the text and the reference, which breaches WP:REFPUNCT. This is essentially the same change that Edokter made to {{Episode list}} back in 2012,[1] and it had to be reverted then as well.[2] In the odd case where a space is needed, the workaround has always been to code a non-breaking space as the first character in the RTitle or RAltTitle fields. As it stands now, when a new episode is added, the RTitle field has to be used insted of Title and quotes must be manually added. Normal practice is to remove references after episodes have aired, generally by removing the RTitle field and reference. Now, it's necessary to remove the reference manually leaving the RTitle field. Then, somebody has to clean the table up by changing RTitle to Title and removing the quotes. It's a ridiculous situation at best. The motivation behind Voidxor's request seems to be based on a misunderstanding of how {{Episode list}} is used. The fixes for List of South Park episodes#Shorts, sketches, and other appearances, the examples that were given, are relatively simple.[3] We shouldn't modify a template and make hundreds, possibly thousands, of articles no longer compliant with WP:REFPUNCT to fix such a simple issue in a single article. --AussieLegend (✉) 08:13, 6 January 2015 (UTC)
- I've reverted the change for now. Please continue to discuss if there is a better way to achieve the intended behaviour. — Martin (MSGJ · talk) 16:50, 6 January 2015 (UTC)
- This change needs to be reverted ASAP. There can be no space between Title and RTitle or AltTitle and RAltTitle. In the vast majority of cases, RTitle and RAltTitle are used to add references to episodes and the change now puts a space between the text and the reference, which breaches WP:REFPUNCT. This is essentially the same change that Edokter made to {{Episode list}} back in 2012,[1] and it had to be reverted then as well.[2] In the odd case where a space is needed, the workaround has always been to code a non-breaking space as the first character in the RTitle or RAltTitle fields. As it stands now, when a new episode is added, the RTitle field has to be used insted of Title and quotes must be manually added. Normal practice is to remove references after episodes have aired, generally by removing the RTitle field and reference. Now, it's necessary to remove the reference manually leaving the RTitle field. Then, somebody has to clean the table up by changing RTitle to Title and removing the quotes. It's a ridiculous situation at best. The motivation behind Voidxor's request seems to be based on a misunderstanding of how {{Episode list}} is used. The fixes for List of South Park episodes#Shorts, sketches, and other appearances, the examples that were given, are relatively simple.[3] We shouldn't modify a template and make hundreds, possibly thousands, of articles no longer compliant with WP:REFPUNCT to fix such a simple issue in a single article. --AussieLegend (✉) 08:13, 6 January 2015 (UTC)
- Note: Is there a way to check if
args.RTitle
starts with<ref
? If so, we can have it not add the space in cases where that is true and reduce the number of spaces before refs. Maybe we can get some help from Jackmcbarn or Mr. S on this. :) — {{U|Technical 13}} (e • t • c) 12:50, 4 January 2015 (UTC) — {{U|Technical 13}} (e • t • c) 13:04, 5 January 2015 (UTC)- @Technical 13: Jackmcbarn (talk) 18:21, 6 January 2015 (UTC)
if args.RTitle:find("\127UNIQ[0-9a-f]+%-[Rr][Ee][Ff]%-[0-9A-F]+%-QINU\127") == 1 then -- ...
- @Technical 13:
- Back to the drawing board... Can someone give an example of why the space was wanted in the first place?
-- [[User:Edokter]] {{talk}}
17:45, 6 January 2015 (UTC)- I've already addressed that in my post above. Voidxor gave some examples where the fix was rather simple.[4] There is an alternate method.[5] In the "Magic: The Gathering" example,
|Title=
should never have been used. "Magic: The Gathering" should only be italicised, not italicised and quoted (the Title and AltTitle fields automatically place the episode title inside quotation marks). In both fixes, the resolution is the same. For the other two, you can either use both Title and RTitle and add a non-breaking space at the beginning of the RTitle field, or just use the RTitle field and add the quotes manually. The non-breaking space method is normally used when "(Part x)" is not part of the episode title. The examples provided by Voidxor are unusual cases. --AussieLegend (✉) 18:52, 6 January 2015 (UTC) - Noting this edit at ANI, most editors seem to think that "RTitle" means "RefTitle" and the instructions for that field say "Unformatted parameter that can be used to add a reference after "
Title
", or can be used as a "raw title" to replace "Title
" completely. Future episodes should include a reference in this field to comply with Wikipedia:Verifiability." The emphasis is on using it for a reference, with the ability to use it for other purposes as a bonus. Regarding the question that was asked in another edit, "How many more uses require the space that the initial edit catered for?", RTitle is usually only used for references. Once an episode has aired, the reference and the RTitle field is generally removed, as the episode serves as the primary reference. {{Episode list}} is used on 6,163 pages, while {{Episode list/sublist}} is used on 2,875. I don't think there's a way of counting actual uses but it has been estimated that there could be 120,000 or more uses of the template. I'd be surprised if RTitle was used more than 2,000-3,000 times and not all of them require the nbsp workaround. --AussieLegend (✉) 19:03, 6 January 2015 (UTC)- RTitle is also use a lot for subtiltes, like 'part 2' that should not be included in the title quotes. That is where the original request stems from.
-- [[User:Edokter]] {{talk}}
19:21, 6 January 2015 (UTC)- Yes, I said in my post that's what it's used for, but this is likely to be less than a couple of thousand instances, out of 120,000 uses of the template. I'm a bit confused about "the original request". Voidxor's request didn't mention parts, the examples given were about extra text that has nothing to do with parts. --AussieLegend (✉) 19:30, 6 January 2015 (UTC)
- That was just an example. The question is, does the use of references outnumber the use of other extra text? I ask because the use of references is usually temporary, while the latter is permanent. That is why I proposed and dedicated parameter for references.
-- [[User:Edokter]] {{talk}}
09:03, 8 January 2015 (UTC)
- That was just an example. The question is, does the use of references outnumber the use of other extra text? I ask because the use of references is usually temporary, while the latter is permanent. That is why I proposed and dedicated parameter for references.
- Yes, I said in my post that's what it's used for, but this is likely to be less than a couple of thousand instances, out of 120,000 uses of the template. I'm a bit confused about "the original request". Voidxor's request didn't mention parts, the examples given were about extra text that has nothing to do with parts. --AussieLegend (✉) 19:30, 6 January 2015 (UTC)
- RTitle is also use a lot for subtiltes, like 'part 2' that should not be included in the title quotes. That is where the original request stems from.
- I've already addressed that in my post above. Voidxor gave some examples where the fix was rather simple.[4] There is an alternate method.[5] In the "Magic: The Gathering" example,
- Okay, so based on the results in Module talk:Episode list/testcases based on adding:
if args.Title and args.RTitle and args.RTitle:find("\127UNIQ[0-9a-f]+%-[Rr][Ee][Ff]%-[0-9A-F]+%-QINU\127") == nil then
- to Module:Episode list/sandbox we can get a space if there is both a Title and RTitle and RTitle does not start with
<ref
. Is that acceptable to everyone (AussieLegend — Edokter — Jackmcbarn — MSGJ — Voidxor:) and do what is needed? — {{U|Technical 13}} (e • t • c) 00:12, 8 January 2015 (UTC)
- I'd be happy with that change. Thanks again, Technical 13! – voidxor (talk | contrib) 06:17, 8 January 2015 (UTC)
- Only concern is that some existing RTitles start with a non-breaking space such as before (Part x) where a line break is not desired after the Title in quotes. Adding a space for all but ref tags will also put a space before the non-breaking space character and permit a line break that is not desired. Geraldo Perez (talk) 06:25, 8 January 2015 (UTC)
- I really think that this is fixing a problem that doesn't really need fixing. --AussieLegend (✉) 06:46, 8 January 2015 (UTC)
- It needs fixing, but I want a better solution by introducing a dedicated ref parameter;
|TitleRef=
(and|AltTitleRef=
).-- [[User:Edokter]] {{talk}}
09:03, 8 January 2015 (UTC)- As someone who spends far too much time cleaning up TV articles containing episodes lists, I have to disagree. The main use of RTitle is as the reference field. The second main use is for containing "(Part x)" when that isn't part of the episode title and we add nbsp so that the title and the part number remain attached. Other than that, we generally use the RTitle field exclusively for custom text, as in the case of the "Magic: The Gathering" example given above, but that doesn't really get a lot of use. On my wish list of things to change in {{Episode list}}, "fixing" this "problem" doesn't even rate a mention. If we were talking about
|Viewers=
defaulting to "N/a" or "TBA" when the fields is empty or adding a reference field for|Viewers=
, I might get excited. We had some very long discussions in 2012, starting at Template talk:Episode list/Archive 3#Requested edits and finishing somewhere near the end of Template talk:Episode list/Archive 4 but even then it wasn't an issue. Working out how to automatically insert a space on the very odd occasion that one is needed is really unnecessary effort. --AussieLegend (✉) 11:01, 8 January 2015 (UTC)- It may have been unnecessary effort, but that effort has already been expended and there is a proposal here that does just that. Would you rather use what is available and then develop further or waste that effort and discourage a new Lua coder that may have been interested in trying to help you achieve your other desired results? — {{U|Technical 13}} (e • t • c) 13:45, 8 January 2015 (UTC)
- Edokter, if you want to introduce new parameters, please start a new discussion below so that there is a clean start to that discussion (feel free to mention this one in it) and so that this one doesn't get TL;DR. In the mean time, would this solution I have proposed be a step forward in achieving the desired behavior? — {{U|Technical 13}} (e • t • c) 13:45, 8 January 2015 (UTC)
- As someone who spends far too much time cleaning up TV articles containing episodes lists, I have to disagree. The main use of RTitle is as the reference field. The second main use is for containing "(Part x)" when that isn't part of the episode title and we add nbsp so that the title and the part number remain attached. Other than that, we generally use the RTitle field exclusively for custom text, as in the case of the "Magic: The Gathering" example given above, but that doesn't really get a lot of use. On my wish list of things to change in {{Episode list}}, "fixing" this "problem" doesn't even rate a mention. If we were talking about
- It needs fixing, but I want a better solution by introducing a dedicated ref parameter;
- Geraldo, that sounds to me like there are existing RTitles that are using a hack which would break the general formating and be generally undesirable. Can you show me where this hack is in place? I need to see some examples as to why someone shouldn't go through and remove all of those from the template usage. — {{U|Technical 13}} (e • t • c) 13:45, 8 January 2015 (UTC)
- Technical 13, I couldn't find any examples where preventing a line break in the title would have been prevented with a nbsp, most of the title columns are wide enough anyway. I guess only issue is this change would force a requirement to remove the hack from existing articles to prevent a double space from appearing. If the check for the ref tag could be expanded to also check for the nbsp, we wouldn't need to update the articles. Geraldo Perez (talk) 15:10, 8 January 2015 (UTC)
- Geraldo, I would honestly rather remove the hack and have no problem with doing it myself aided by WP:AWB. If there turns out to be an overwhelming large number of these, I can put in a request a WP:BRFA to do it with User:T13bot. — {{U|Technical 13}} (e • t • c) 15:18, 8 January 2015 (UTC)
- The module is used on over 9,000 pages with up to 120,000 instances of the module all up. Finding and replacing all of the instances would be an enormous effort, even with AWB. Many pages don't contain any instances so it's a lot of doing nothing. --AussieLegend (✉) 15:38, 8 January 2015 (UTC)
- I couldn't find any exemplars with a quick search but commas also can't have spaces in front of them. This is a plausible initial characters for values in the raw title attribute. M-dashes are not supposed to be spaced either but that is iffy after a quote. Also for raw titles if the title attribute is empty. I think the value of automatically adding a space is outweighed by the the generally understood rule that template attribute values can't start with a space so if one is required, use a nbsp. This is how editors have been trained so far and existing documentation reflects. Geraldo Perez (talk) 15:59, 8 January 2015 (UTC)
- I really think that this is fixing a problem that doesn't really need fixing. --AussieLegend (✉) 06:46, 8 January 2015 (UTC)
- Only concern is that some existing RTitles start with a non-breaking space such as before (Part x) where a line break is not desired after the Title in quotes. Adding a space for all but ref tags will also put a space before the non-breaking space character and permit a line break that is not desired. Geraldo Perez (talk) 06:25, 8 January 2015 (UTC)
- 7th Heaven (season 1)
- 7th Heaven (season 10)
- 7th Heaven (season 11)
- 7th Heaven (season 2)
- 7th Heaven (season 3)
- 7th Heaven (season 4)
- 7th Heaven (season 6)
- 7th Heaven (season 7)
- 7th Heaven (season 8)
- 7th Heaven (season 9)
- Cold Case (season 6)
- Cold Case (season 7)
- Degrassi (season 10)
- Degrassi (season 11)
- Degrassi (season 12)
- Degrassi (season 13)
- Degrassi: The Next Generation (season 2)
- Degrassi: The Next Generation (season 4)
- Degrassi: The Next Generation (season 5)
- Degrassi: The Next Generation (season 6)
- Degrassi: The Next Generation (season 7)
- Degrassi: The Next Generation (season 8)
- Degrassi: The Next Generation (season 9)
- ER (season 13)
- Family Guy (season 4)
- Family Guy (season 6)
- Ghost Whisperer (season 1)
- Ghost Whisperer (season 2)
- Ghost Whisperer (season 3)
- Hannah Montana (season 2)
- Hannah Montana (season 3)
- List of 2point4 children episodes
- List of Aaron Stone episodes
- List of Are You Being Served? episodes
- List of Camelot episodes
- List of Degrassi episodes
- List of Diff'rent Strokes episodes
- List of Duel Masters episodes
- List of Eureka episodes
- List of Game Grumps serials
- List of Good Luck Charlie episodes
- List of Grand Designs episodes
- List of Human Target episodes
- List of Journeyman episodes
- List of Junior MasterChef Pinoy Edition episodes
- List of Lab Rats episodes
- List of Mister Ed episodes
- List of My Babysitter's a Vampire episodes
- List of Odd One In episodes
- List of Pair of Kings episodes
- List of R. L. Stine's The Haunting Hour episodes
- List of Reno 911! episodes
- List of South Park episodes
- List of Spooksville episodes
- List of Steptoe and Son episodes
- List of Tenkai Knights episodes
- List of That Girl episodes
- List of The Incredible Hulk (1978 TV series) episodes
- List of Ultimate Spider-Man episodes
- List of Victorious episodes
- List of Whitney episodes
- List of Winx Club episodes
- List of Zeke and Luther episodes
- Raising Hope (season 2)
- Sonny with a Chance (season 2)
- Spike Team
- Teenage Mutant Ninja Turtles (1987 TV series) (season 3)
- Template talk:Episode list/Archive 5
- Template:Episode list/testcases
- The Civil War (TV series)
- The New Adventures of Old Christine (season 1)
- The New Adventures of Old Christine (season 2)
- The New Adventures of Old Christine (season 4)
- The New Adventures of Old Christine (season 5)
- The River (U.S. TV series)
- The Roosevelts (film)
- The Wannabes (TV series)
- The War (2007 TV series)
- The West (documentary)
- Under Cover (TV series)
- User:Gersende/sandbox 24
- User:Matthewedwards/Sandbox/Episodes
- User:Matthewedwards/Sandbox/List of 2point4 Children episodes
- User:Pokemon Ranger is Amazing 123/sandbox
- Warehouse 13 (season 2)
- Warehouse 13 (season 3)
- Wikipedia talk:Featured list candidates/Archive 8
- Winx Club (season 1)
- Winx Club (season 2)
- Winx Club (season 3)
- Winx Club (season 4)
- Wizards of Waverly Place (season 1)
A request to module editors
This module is used in {{Episode list}}, the primary user of which is WikiProject Television. Generally, few end users monitor template pages anyway, but even less monitor modules such as this. Prior to making any changes to this module it would be appreciated if some notification is made at the template talk page, or at WT:TV so end users can be aware of what is happening. This will avoid situations as that which happened in January when undiscussed changes resulted in the WP:REFPUNCT violation above. It was only because I have template editing experience that I thought to check the module page. Other editors had no idea what was going on. Had I not still had the page on my watchlist, Rich Farmbrough's request would likely have gone unanswered by somebody who actually uses the template. --AussieLegend (✉) 19:08, 31 March 2015 (UTC)
No episode title
An editor has asked for help doing an episode guide for a series with no titles. The module does not support empty title columns because it processes "Title" specially, as it is an agglutination of fields, and the check for that column does not check that it will not be empty. I have introduced such a check, which considers all four arguments which can comprise the title column, in the sandbox.
Are there any other issues that prevent this from being made live?
All the best: Rich Farmbrough, 17:24, 31 March 2015 (UTC).
- I eventually found the request that you referred to, at least I assume it's this. Typically, we require citations for all future episodes and these are normally placed in
|RTitle=
, which appears in|Title=
. If one adds a citation without an episode this will result in an error:
Episode | Directed by | Written by | Original air date | |
---|---|---|---|---|
3–6 | [1] | TBA | TBA | April 29, 2015 |
- It's for that reason
|Title=
is required. Where an episode has no title, which seems common on UK programmes, we typically use "Series x, episode y" or something similar as the title. (See List of Top Gear episodes, List of Primeval episodes for examples). In the event that an episode article is eventually created, it's in this field that the link to the article is normally included. I can understand the issues that the editor is having, but the resolution seems inconsistent with standard practice throughout WikiProject Television, which is the primary user of this module. --AussieLegend (✉) 18:53, 31 March 2015 (UTC)- This seems to be a hack - making up episode titles to fit a technical issue. If no title column is used the ref can be added to any appropriate column, either explicitly or through an additional parameter. (Note that the only reason RTitle exists is to avoid having the reference inside the quote marks - there is nothing particularly "referency" about the Title column, rather the contrary.)
Episode | Directed by | Written by | Original air date |
---|---|---|---|
3-6[1] | TBA | TBA | April 29, 2015 |
- I considered throwing an error if an illogical combination of Title, RTitle, AltTitle and RAltTitle are used, but I see that these parameters are designed to be abused as "raw titles". It would be better to have separate parameters for "raw titles" as it would allow the space issue to be fixed properly, and allow referencing and a raw title to coexist. That, however, is beyond the scope of this change.
- All the best: Rich Farmbrough, 19:59, 31 March 2015 (UTC).
- Yes, the reference can be added to any column but the convention is to use the RTitle field, and that's the way it has been done for years. Its use has previously been discussed, at FAR, at resulting discussions at the template talk page that resulted in changes to the template, and at WT:TV. Changing that convention is beyond the scope of this change too. Adding a reference to
|EpisodeNumber=
is not recommended, as it affects linking to the section. In your example above, [[#3-6]] no longer works. Neither does [[#3-6<sup>[1]</sup>]]. --AussieLegend (✉) 21:27, 31 March 2015 (UTC)
- Yes, the reference can be added to any column but the convention is to use the RTitle field, and that's the way it has been done for years. Its use has previously been discussed, at FAR, at resulting discussions at the template talk page that resulted in changes to the template, and at WT:TV. Changing that convention is beyond the scope of this change too. Adding a reference to
Thee is no auto link to sections, and manually they can be done trivially:
Episode | Directed by | Written by | Original air date |
---|---|---|---|
No–title | TBA | TBA | April 29, 2015 |
#No episode title[1] | TBA | TBA | April 29, 2015 |
Also references are widely used in other columns, even from the handful of pages I have seen.
But this is really beside the point. The proposed change does not break any existing tables. If someone were to attempt to add a reference to a non-existing title, they should expect the same problem as if they were to use any other column not defined for that particular series.
All the best: Rich Farmbrough, 19:55, 3 April 2015 (UTC).
- I'm not sure what you're referring to when you say "The[r]e is no auto link to sections". I was talking about the anchor provided by
|EpisodeNumber=
. For example see List of NCIS episodes#ep67, which links to the episode "Bloodbath". It's not possible to link to the episode that you've shown as "#No episode title[1]" Yes, references are used in other columns but when they reference the whole episode, or even a significant portion of it, they are supposed to go in RTitle. The claim that "If someone were to attempt to add a reference to a non-existing title, they should expect the same problem as if they were to use any other column not defined for that particular series" is misleading. We expect episode titles for all episodes and references are normally added to RTitle. If a reference was added to "RTitle" without an episode title in "Title" it would cause confusion. We don't normally add references to fields that aren't defined, simply because they're not defined. Why would anyone add a refernce to, say, WrittenBy if the field was empty? That makes no sense. I don't disagree with the code change in principle, but the documentation should still require episode titles, making the code change unnecessary. As you can see at Draft:Six Degrees (or 6 Degrees), if we format the list as we expect it to be formatted, the original problem has been solved. --AussieLegend (✉) 03:57, 4 April 2015 (UTC)
EpisodeNumberRowspan
A suggestion for an optional variable called EpisodeNumberRowspan (inserted after EpisodeNumber), to give a rowspan to the EpisodeNumber column (default 1). The use for it can be seen in my sandbox - this provides easier visual identification for more serialized television series where multi-part episodes are a major factor of the series (e.g. Doctor Who), instead of episode numbers like 164a and 164b. Update: I attempted this myself in Module:Episode list/sandbox (with no experience in Lua, but plenty in other languages, so I caught on rather quickly), tested it out in Module:Episode list/testcases, and it seems to work. The additional/modified lines in the sandbox are lines 121-123 and 131. Alex|The|Whovian 13:01, 1 July 2015 (UTC)
- looks like there is discussion at Template talk:Episode list#Doctor Who Series Tables. Frietjes (talk) 13:03, 1 July 2015 (UTC)
Anchor not working
Per the instructions for {{Episode list}}, when
ProdCode
is defined, this parameter also creates a link anchor, prefixed by "
. That doesn't seem to be working. For example, List of Jessie episodes#pc101 should link to this episode but it doesn't. I'm not really up on Lua so can somebody please fix this? --AussieLegend (✉) 08:29, 22 July 2015 (UTC)pc
"; for example, "[[List of episodes#pc01]]
"
Edit request
Please see edit request at Template_talk:Infobox_television_season#Request regarding the accessibility tracking category. EvergreenFir (talk) Please {{re}} 17:11, 1 September 2015 (UTC)
- It's been fixed in the sandbox. Alakzi (talk) 17:53, 1 September 2015 (UTC)
- @AlexTheWhovian: EvergreenFir (talk) Please {{re}} 04:33, 2 September 2015 (UTC)
- You need to page someone who can actually copy it over. Alakzi (talk) 07:46, 2 September 2015 (UTC)
- @Frietjes and Mr. Stradivarius: ? Alex|The|Whovian 03:06, 3 September 2015 (UTC)
- Done. Next time consider using {{edit template-protected}} - that will get the attention of more template editors than just Frietjes and I. — Mr. Stradivarius ♪ talk ♪ 04:42, 3 September 2015 (UTC)
- @Frietjes and Mr. Stradivarius: ? Alex|The|Whovian 03:06, 3 September 2015 (UTC)
- You need to page someone who can actually copy it over. Alakzi (talk) 07:46, 2 September 2015 (UTC)
- @AlexTheWhovian: EvergreenFir (talk) Please {{re}} 04:33, 2 September 2015 (UTC)
Edit Request
This edit request has been answered. Set the |answered= or |ans= parameter to no to reactivate your request. |
Many television series have "multi-part episodes", and editors have used |LineColor=none
. These come up frequently when clearing out Category:Episode lists with invalid line colors, so is it possible to edit the module so that when the LineColour is set to none, it doesn't necessarily list in the category? It's not invalid as such concerning the contrast issues for which the category was initially created. Also, it would be handy if something like |LineColor=black
is used, it actually gives a black line instead of reporting it as invalid. Alex|The|Whovian 04:43, 26 November 2015 (UTC)
- Could you make the required changes to Module:Episode list/sandbox please? — Martin (MSGJ · talk) 14:17, 27 November 2015 (UTC)
- @MSGJ: Done. Alex|The|Whovian 08:11, 29 November 2015 (UTC)
- Sorry, must have missed your ping. I am no Lua coder but can you just confirm you have tested these changes and they are working as intended? — Martin (MSGJ · talk) 09:31, 21 December 2015 (UTC)
- As per Module talk:Episode list/testcases#LineColor, they work. Make sure that only the edits up to the end of November are implemented; there's recent edits from something else that I'm working on. Thanks! Alex|The|Whovian 09:52, 21 December 2015 (UTC)
- Okay, deployed — Martin (MSGJ · talk) 11:22, 21 December 2015 (UTC)
- @MSGJ: Sorry, slight (honestly, tiny) error I found and adjusted in the sandbox. It now works perfectly. Alex|The|Whovian 12:09, 21 December 2015 (UTC)
- The sandbox still contains your changes from yesterday which you have not yet formally requested. — Martin (MSGJ · talk) 12:27, 21 December 2015 (UTC)
- @MSGJ: Sorry, slight (honestly, tiny) error I found and adjusted in the sandbox. It now works perfectly. Alex|The|Whovian 12:09, 21 December 2015 (UTC)
- Okay, deployed — Martin (MSGJ · talk) 11:22, 21 December 2015 (UTC)
- As per Module talk:Episode list/testcases#LineColor, they work. Make sure that only the edits up to the end of November are implemented; there's recent edits from something else that I'm working on. Thanks! Alex|The|Whovian 09:52, 21 December 2015 (UTC)
- Sorry, must have missed your ping. I am no Lua coder but can you just confirm you have tested these changes and they are working as intended? — Martin (MSGJ · talk) 09:31, 21 December 2015 (UTC)
- @MSGJ: Done. Alex|The|Whovian 08:11, 29 November 2015 (UTC)
Bugger. Sorry. All that needs changing is one line: "a-f0-9" needs to be changed to "a-fA-F0-9". Alex|The|Whovian 12:29, 21 December 2015 (UTC)
- @AlexTheWhovian: Hey, any chance these changes are the reason some articles are displaying the default line color in episode tables? Examples: Fargo (season 2)#Episodes, The X-Files (season 3)#Episodes, Agents of S.H.I.E.L.D. (season 2)#Episodes. Drovethrughosts (talk) 12:57, 21 December 2015 (UTC)
- Yeah, Drovethrughosts, exactly that. I forgot the usage of uppercase letters in line colours alongside lowercase letters. Once the minor fix is implemented, they'll revert to how they're meant to be. Alex|The|Whovian 13:03, 21 December 2015 (UTC)
- Thanks. Drovethrughosts (talk) 13:04, 21 December 2015 (UTC)
- I've implemented AlexTheWhovian's minor fix. That should solve the problem. --AussieLegend (✉) 14:24, 21 December 2015 (UTC)
- Thanks. Drovethrughosts (talk) 13:04, 21 December 2015 (UTC)
- Yeah, Drovethrughosts, exactly that. I forgot the usage of uppercase letters in line colours alongside lowercase letters. Once the minor fix is implemented, they'll revert to how they're meant to be. Alex|The|Whovian 13:03, 21 December 2015 (UTC)
Production Code ID
This edit request has been answered. Set the |answered= or |ans= parameter to no to reactivate your request. |
In the module, an ID attribute is included for production codes (e.g. #pc3J5914); however this produces a deformed ID attribute when the code is set to {{TableTBA}} (will result in id="pc.3Csmall.3E.3Cspan_style.3D.22color:_.232C2C2C.22_title.3D.22To_be_announced.22.3ETBA.3C.2Fspan.3E.3C.2Fsmall.3E"
). I've modified a line in the sandbox to prevent this. Alex|The|Whovian? 06:24, 12 March 2016 (UTC)
- For simplicity, it's replace
if td_tags.ProdCode and args.ProdCode then
with if td_tags.ProdCode and args.ProdCode and (string.find(args.ProdCode,'TBA') == nil) then
. Alex|The|Whovian? 06:29, 12 March 2016 (UTC)- Done — xaosflux Talk 00:47, 15 March 2016 (UTC)
Edit Request #2
This edit request has been answered. Set the |answered= or |ans= parameter to no to reactivate your request. |
Edit request to prevent malformed episode and production number IDs when {{episode list}} is used for double-episodes. These may be in the form of "ep87_----_88", "ep87.3Chr_.2F.3E88", or "ep87.3Cbr_.2F.3E88" when "----", "<hr />", or "<br />" are used respectively.. The sandbox has been modified to prevent this, per Special:Diff/712324068/714042176. Alex|The|Whovian? 07:44, 7 April 2016 (UTC)
- Done Izno (talk) 11:20, 7 April 2016 (UTC)
- Just a note that we've traditionally used "87", "87/88", "87/88/89" or smilar styling for episode numbers because other formats have been problematic when linking to episode entries using "#epx". --AussieLegend (✉) 13:01, 7 April 2016 (UTC)
- Noted. Now they're not. Alex|The|Whovian? 13:05, 7 April 2016 (UTC)
- Just a note that we've traditionally used "87", "87/88", "87/88/89" or smilar styling for episode numbers because other formats have been problematic when linking to episode entries using "#epx". --AussieLegend (✉) 13:01, 7 April 2016 (UTC)
88|3]]. I beg to differ. Remember, we have to code for the average editor, not those of us who can code. When would you use "----" anyway? I'm intrigued, because I've never seen anyone do it in the last 9 years. --AussieLegend (✉) 13:31, 7 April 2016 (UTC)
- Apologies, I should have been more clearer and made it simpler. When using line breaks or thematic breaks, the episode number cell would have a deformed ID (an ID nonetheless, though), and to link to that episode, we'd need to use [[Show name#ep87.3Cbr_.2F.3E88]], for example. The recent edits prevent this, by making the ID equal to only the number of the first episode - that is, if the double episode spans episode numbers 87 and 88, then [[Show name#ep87]] will link directly to the episode. Alex|The|Whovian? 13:58, 7 April 2016 (UTC)
- This will need to be documented at Template:Episode list/doc. --AussieLegend (✉) 14:31, 7 April 2016 (UTC)
- Apologies, I should have been more clearer and made it simpler. When using line breaks or thematic breaks, the episode number cell would have a deformed ID (an ID nonetheless, though), and to link to that episode, we'd need to use [[Show name#ep87.3Cbr_.2F.3E88]], for example. The recent edits prevent this, by making the ID equal to only the number of the first episode - that is, if the double episode spans episode numbers 87 and 88, then [[Show name#ep87]] will link directly to the episode. Alex|The|Whovian? 13:58, 7 April 2016 (UTC)
Edit Request
This edit request has been answered. Set the |answered= or |ans= parameter to no to reactivate your request. |
Implement the code in the sandbox that automatically fills in empty cells with {{TableTBA}}, per the discussions at Special:Permalink/696011961#Empty cells and Template talk:Episode list#Empty variables. Alex|The|Whovian? 13:51, 6 April 2016 (UTC)
- Could you please provide a diff from the current module showing the delta in changes? This can either be from a "resynced sandbox to current sandbox" link or via Special:ComparePages. Thanks! --Izno (talk) 15:19, 6 April 2016 (UTC)
- @Izno: See [6]. – nyuszika7h (talk) 15:23, 6 April 2016 (UTC)
- Done Izno (talk) 15:30, 6 April 2016 (UTC)
Revisiting this with a minor fix for allowing RTitle to be used with an empty Title. Diffs here. Alex|The|Whovian? 14:21, 28 May 2016 (UTC)
- Done Izno (talk) 15:39, 28 May 2016 (UTC)
hundreds of broken articles....
This edit request has been answered. Set the |answered= or |ans= parameter to no to reactivate your request. |
The category, Category:Pages with script errors, has just added hundreds of new entries, seemingly all caused by the recent change to this module by AlexTheWhovian. Please revert these changes to fix the errors, unless they can be rapidly fixed. As per the notice at the top of the module, changes on such a module should have been tested in the sandbox first.--JohnBlackburnewordsdeeds 11:27, 9 December 2016 (UTC)
- Reverted the edits. Alex|The|Whovian? 11:30, 9 December 2016 (UTC)
- I think it was just the last one, based on the timing of the problem - sorry, should have been clearer in my request. Thanks for the prompt response.--JohnBlackburnewordsdeeds 11:31, 9 December 2016 (UTC)
- Needed to revert it all anyways; wasn't getting the results I was expecting. Alex|The|Whovian? 01:51, 10 December 2016 (UTC)
- I think it was just the last one, based on the timing of the problem - sorry, should have been clearer in my request. Thanks for the prompt response.--JohnBlackburnewordsdeeds 11:31, 9 December 2016 (UTC)