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 <[email protected]>:
> 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 // [email protected] >>
>
>
> _______________________________________________
> Pywikipedia-l mailing list
> [email protected]
> https://lists.wikimedia.org/mailman/listinfo/pywikipedia-l
>

_______________________________________________
Pywikipedia-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/pywikipedia-l

Reply via email to