On Wed, Sep 3, 2014 at 2:29 AM, Sorawee Porncharoenwase nullzero.free@gmail.com 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/