User talk:Subpagination Bot/Task 1: Difference between revisions

From Citizendium
Jump to navigation Jump to search
imported>Chris Day
No edit summary
imported>Jitse Niesen
(reply to Larry)
Line 6: Line 6:


Well, I've looked it over as exhaustively as I'm going to, and it looks excellent.  One reservation is that the script doesn't simply scoop up the checklist data, but actually processes and reorganizes it.  This is probably a good idea, but I am concerned about parsing in that case.  Will it handle different styles of indentation?  What about capitalization?  Maybe some people have messed with that--I don't know if template attributes are case-sensitive, but if not... --[[User:Larry Sanger|Larry Sanger]] 11:03, 15 September 2007 (CDT)
Well, I've looked it over as exhaustively as I'm going to, and it looks excellent.  One reservation is that the script doesn't simply scoop up the checklist data, but actually processes and reorganizes it.  This is probably a good idea, but I am concerned about parsing in that case.  Will it handle different styles of indentation?  What about capitalization?  Maybe some people have messed with that--I don't know if template attributes are case-sensitive, but if not... --[[User:Larry Sanger|Larry Sanger]] 11:03, 15 September 2007 (CDT)
:It should handle all indentation styles correctly. Everything is case sensitive, except the first letter of the name of the template, so you can write "<nowiki>{{</nowiki>checklist &hellip;" or "<nowiki>{{</nowiki>Checklist &hellip;". Both are recognized. But you are right that this parsing should be done with a great deal of care. I added a test that all template attribute should be recognized and that none should be repeated, to make sure that any pages where the checklist cannot be parsed is skipped. -- [[User:Jitse Niesen|Jitse Niesen]] 07:58, 16 September 2007 (CDT)


Hi Jitse, I just checked through your test run samples and everything looks OK.  It is good that you reordered the content of the checklist and put the cat1-3 fields lower.  That will help the constables a lot, in the future, since it will allow them to cleanly cut and paste  the "cat1 to date" fields directly into the {{tl|approval}} template that lives on the approval page (part of the current approval process developed by [[User talk:D. Matt Innis|Matt]] to improve and simplify record keeping). Nice job. [[User:Chris Day|Chris Day]] [[User talk:Chris Day|(talk)]] 15:50, 15 September 2007 (CDT)
Hi Jitse, I just checked through your test run samples and everything looks OK.  It is good that you reordered the content of the checklist and put the cat1-3 fields lower.  That will help the constables a lot, in the future, since it will allow them to cleanly cut and paste  the "cat1 to date" fields directly into the {{tl|approval}} template that lives on the approval page (part of the current approval process developed by [[User talk:D. Matt Innis|Matt]] to improve and simplify record keeping). Nice job. [[User:Chris Day|Chris Day]] [[User talk:Chris Day|(talk)]] 15:50, 15 September 2007 (CDT)

Revision as of 06:58, 16 September 2007

Comments:

  • Note that the old subpage templates are of the form subpagesN, not subpagesNN.

--Larry Sanger 10:45, 15 September 2007 (CDT)

Oops, I was confused after all. "If none of these conditions are met" you say. --Larry Sanger 10:55, 15 September 2007 (CDT)

Well, I've looked it over as exhaustively as I'm going to, and it looks excellent. One reservation is that the script doesn't simply scoop up the checklist data, but actually processes and reorganizes it. This is probably a good idea, but I am concerned about parsing in that case. Will it handle different styles of indentation? What about capitalization? Maybe some people have messed with that--I don't know if template attributes are case-sensitive, but if not... --Larry Sanger 11:03, 15 September 2007 (CDT)

It should handle all indentation styles correctly. Everything is case sensitive, except the first letter of the name of the template, so you can write "{{checklist …" or "{{Checklist …". Both are recognized. But you are right that this parsing should be done with a great deal of care. I added a test that all template attribute should be recognized and that none should be repeated, to make sure that any pages where the checklist cannot be parsed is skipped. -- Jitse Niesen 07:58, 16 September 2007 (CDT)

Hi Jitse, I just checked through your test run samples and everything looks OK. It is good that you reordered the content of the checklist and put the cat1-3 fields lower. That will help the constables a lot, in the future, since it will allow them to cleanly cut and paste the "cat1 to date" fields directly into the {{approval}} template that lives on the approval page (part of the current approval process developed by Matt to improve and simplify record keeping). Nice job. Chris Day (talk) 15:50, 15 September 2007 (CDT)