On Wed, Sep 3, 2014 at 2:29 AM, Sorawee Porncharoenwase
<[email protected]> wrote:
> I propose this change:
>
> - set `page.text` to `None` to discard changes

Doesn't the page.text setter already do this?

> - delete `page.text` to reload content -- equivalent to get(force=True)

del page.text already removes the cached / modified page text.

so a forced reload of text is currently two lines:

del page.text
new = page.text

I dont see why 'del page.text' should initiate network activity to
reload the page text.

> - in Page.save(), if `page.text` is equal to unchanged text, no API call
> - to touch a page, use page.touch(), which will call action="edit" &
> appendtext="", instead. The advantage is that there is no need to preload
> text.

We have a touch() changeset pending.

https://gerrit.wikimedia.org/r/#/c/144717/

-- 
John Vandenberg

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

Reply via email to