xqt(a)svn.wikimedia.org ha scritto:
Revision: 8292
[...]
Log Message:
-----------
bugfix for r8051 (bug #3015645)
[...]
def skip_section(text):
- l = list()
- for s in sections_to_skip.itervalues():
- l.extend(s)
- sect_titles = '|'.join(l)
-
+ sect_titles = '|'.join(sections_to_skip[wikipedia.getSite().lang])
sectC = re.compile('(?mi)^==\s*(' + sect_titles + ')\s*==')
- newtext = ''
-
Preliminary remarks: I don't work anymore in this project because
lacking of coordination and my advises for cooperation got ignored and
even bad interpreted as impolite requests. Of course it isn't a serious
loss for the team.
I don't know what you wanted fix here (maybe the problem was recently
added 'ar' row in dictionary only), but this commit is wrong.
This function should skip all the sections listed in the dictionary,
without language based distintion.
Yes, I know, this source is badly written and need a rewrite, as well
as a bad example for commit messages quality and so on, but sometime
something are made in a way for a reason.
Thanks for taking care of my script though.
Regards,
--
Francesco Cosoleto
La maggior parte dei lussi, e molte delle cosiddette comodità, non sono
indispensabili, ma sono anzi veri e propri ostacoli all'elevazione
morale dell'Uomo. (Thoreau, "Walden")