HomePhabricator

Change ensureFlowRevision to use Status and not check summary

Description

Change ensureFlowRevision to use Status and not check summary

In the course of debugging the import script we decided to add
some more detailed logging, including of the ensureFlowRevision call.

Erik suggested I change it to return a status.

In the course of looking at that, I decided to remove the edit summary
check, since it is not reliable. Users can set this summary on normal
wikitext edits.

Imports must be to non-existent pages. Before, allowCreation was
failing, but the test passed anyway because it never verified
the commit succeeded (and the fail-early is new).

Bug: T94953
Change-Id: If54e76f4a894fc068f9ba9389b5f99e4d0d2bfb8

Details

Committed
Mattflaschen-WMFApr 30 2015, 3:37 PM
Parents
rEFLW72f52dcd65fd: Merge "Remove ext.flow.parsoid.js"
Branches
Unknown
Tags
Unknown
References
refs/changes/26/207726/5
ChangeId
If54e76f4a894fc068f9ba9389b5f99e4d0d2bfb8