Last modified: 2014-04-15 18:22:16 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T65696, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 63696 - Flow: display, behavior, and API failures when displaying a post revision
Flow: display, behavior, and API failures when displaying a post revision
Status: NEW
Product: MediaWiki extensions
Classification: Unclassified
Flow (Other open bugs)
master
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-04-08 19:32 UTC by spage
Modified: 2014-04-15 18:22 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description spage 2014-04-08 19:32:28 UTC
You can display a particular revision of a post in Flow, e.g. in topic actions, choose History, then click the timestamp under a post activity.

https://www.mediawiki.org/w/index.php?title=Talk:Sandbox&topic_postId=rs9kbf8vtldmbjm3&topic_revId=rs9kbf8vtldmbjm3&workflow=rs9k21e73g4qsurn

There are several glitches with this.

* Three horizontal lines at top is overkill:
  <Topic title>
  --------------------------------------------
  This topic was started on <Flow board>
  --------------------------------------------
  This is a permanent link to the first version of this post. You can view later versions on the post history page.
  --------------------------------------------

* It's not clear you're only viewing one post from the topic.

* It shouldn't say "You can view later versions on the post history page" if the first version is the only version.

* If you choose Permalink from the revision's action menu, you get a link to latest revision, not this one (which you're already viewing)

* If you Hide are you hiding just this version?  I tried it and got a pink API error box, the 500 response contains
        PHP fatal error in /usr/local/apache/common-local/php-1.23wmf21/extensions/Flow/includes/WorkflowLoader.php line 107: <br/> 
        Argument 1 passed to Flow\WorkflowLoader::loadWorkflow() must be an instance of Title, boolean given, called in /usr/local/apache/common-local/php-1.23wmf21/extensions/Flow/includes/WorkflowLoader.php on line 85 and defined

* Suppressing the particular version you see might make sense, I don't know if it does that.

* It has  Reply • Edit • Thank.
  - Reply doesn't make sense, you don't want to reply to an old version that says "X is Y" when the latest version says "X is ABC".
  - Edit goes and edits the latest version, which is a little confusing.
  - I don't think Thanks generates a link to the version for which you're thanking.

If you are viewing the latest version of a post these could all be enabled, but it's still confusing, and how would the Thanks extension know when this is the case?

We should spec out what you should be able to do when viewing an older version of a post. Reducing or eliminating Action menu and flow-post-action menu items would reduce the scope for these errors.
Comment 1 spage 2014-04-08 20:20:00 UTC
The API failure I mention above (when you Hide/Unhide from the display of a post revision) occurs because the workflow parameter is missing from the API request. But before fixing this we should decide if it even makes sense to offer the operation.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links