you should change

Page(self,site(),

to

Page(self.site(),

try and see if it works. But I agree that it can be very annoying for users if something like that breaks the whole workflow.

Hannes

2009/11/30 Marcin Cieslak saper@saper.info:

Hello!

This on wikipedia.py:

@@ -2591,6 +2600,7 @@

sysop = self._getActionUser(action = 'move', restriction = self.moveRestriction, sysop = False)         if deleteAndMove:             sysop = self._getActionUser(action = 'delete', restriction = '', sysop = True)

• Page(self,site(), newtitle).delete(self.site().mediawiki_message('delete_and_move_reason'), False, False)

# Check blocks         self.site().checkBlocks(sysop = sysop)

Causes this:

Traceback (most recent call last):  File "kalendariumwp.py", line 26, in <module>    p.move("Wikipedysta:Joymaster/%s" % t, reason=action, sysop=True, deleteAndMove=True)  File "/home/admini/saper/wikipedia/pywikipedia/wikipedia.py", line 2646, in move    Page(self,site(), newtitle).delete(self.site().mediawiki_message('delete_and_move_reason'), False, False) NameError: global name 'site' is not defined

Could you please *stop* committing things *COMPLETELY* untested?

Who can revoke committer bit?

--              << Marcin Cieslak // saper@saper.info >>

---

Pywikipedia-l mailing list Pywikipedia-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/pywikipedia-l