Last modified: 2012-08-04 21:10:37 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 T34838, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 32838 - Do not show AFTv4 and AFTv5 on the same page
Do not show AFTv4 and AFTv5 on the same page
Status: RESOLVED DUPLICATE of bug 32857
Product: MediaWiki extensions
Classification: Unclassified
ArticleFeedbackv5 (Other open bugs)
unspecified
All All
: Highest blocker (vote)
: ---
Assigned To: Yoni Shostak
http://prototype.wikimedia.org/releas...
:
: 32837 (view as bug list)
Depends on: 32857
Blocks: 39042
  Show dependency treegraph
 
Reported: 2011-12-07 01:38 UTC by Fabrice Florin
Modified: 2012-08-04 21:10 UTC (History)
2 users (show)

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


Attachments

Description Fabrice Florin 2011-12-07 01:38:46 UTC
Howie found a page on prototype that shows both AFTv4 and AFTv5, one on top of one other (see link).

This should not be happening. AFT v4 should be suppressed when v5 is available on a page.
Comment 1 John Du Hart 2011-12-07 01:39:42 UTC
*** Bug 32837 has been marked as a duplicate of this bug. ***
Comment 2 Fabrice Florin 2011-12-07 23:52:42 UTC
We believe that this problem can be fixed by keeping two separate lists of articles, as described below:

White List:
Article_Feedback_5

Black List:
Article_Feedback_Blacklist

Once you support these category names in the beta code on prototype, Dario and I plan try to add a few articles on prototype in these categories, so we can test if the system works as intended.

_______________________

From Dario:

Here's a summary of the AFT activation logic to be used for the 12/12 launch.

1) AFT checks whether an article is in the whitelist:
http://en.wikipedia.org/wiki/Category:Article_Feedback_5
(I just made up the name, we can change it to whatever we want as this category will be used for testing purposes only and will disappear once AFT goes in production at 100%)

This category will include articles that are "safe" for the test phase, i.e. a random sample of articles taken from enwiki with the exclusion of:
• redirect pages
• geotagged articles

I am generating this list (A) right now and waiting for Mark to confirm whether my method to detect geotagged articles is accurate.
I have a second list (B) ready that only excludes redirect pages, should we be able to drop the no-geotag limitation.

We will need to run a bot to populate the category with articles from this list, whether it's (A) or (B) depends on OmniTI's progress with the geotag issue
Howie can you check whether Kaldari or Asher can help with the bot?

2) AFT checks whether an article is in the blacklist:
http://en.wikipedia.org/wiki/Category:Article_Feedback_Blacklist

We should honor this blacklist after checking for the whitelist as it includes manually flagged articles on which AFT (whether it's 4 or 5) should be disabled. This list includes in particular disambiguation pages that cannot be detected using page titles and is constantly updated by the community.

So to recap, we only display AFT if article X is in Category:Article_Feedback_5 and is not in Category:Article_Feedback_Blacklist

Dario
Comment 3 Fabrice Florin 2011-12-07 23:55:09 UTC
We believe that this problem can be fixed by keeping two separate lists of articles, as described below:

White List:
Article_Feedback_5

Black List:
Article_Feedback_Blacklist

Once you support these category names in the beta code on prototype, Dario and I plan try to add a few articles on prototype in these categories, so we can test if the system works as intended.

_______________________

From Dario:

Here's a summary of the AFT activation logic to be used for the 12/12 launch.

1) AFT checks whether an article is in the whitelist:
http://en.wikipedia.org/wiki/Category:Article_Feedback_5
(I just made up the name, we can change it to whatever we want as this category will be used for testing purposes only and will disappear once AFT goes in production at 100%)

This category will include articles that are "safe" for the test phase, i.e. a random sample of articles taken from enwiki with the exclusion of:
• redirect pages
• geotagged articles

I am generating this list (A) right now and waiting for Mark to confirm whether my method to detect geotagged articles is accurate.
I have a second list (B) ready that only excludes redirect pages, should we be able to drop the no-geotag limitation.

We will need to run a bot to populate the category with articles from this list, whether it's (A) or (B) depends on OmniTI's progress with the geotag issue
Howie can you check whether Kaldari or Asher can help with the bot?

2) AFT checks whether an article is in the blacklist:
http://en.wikipedia.org/wiki/Category:Article_Feedback_Blacklist

We should honor this blacklist after checking for the whitelist as it includes manually flagged articles on which AFT (whether it's 4 or 5) should be disabled. This list includes in particular disambiguation pages that cannot be detected using page titles and is constantly updated by the community.

So to recap, we only display AFT if article X is in Category:Article_Feedback_5 and is not in Category:Article_Feedback_Blacklist

Dario
Comment 4 Yoni Shostak 2011-12-08 16:08:56 UTC
Resolved by #32857, separate categories as outlined in the comments above.

*** This bug has been marked as a duplicate of bug 32857 ***
Comment 5 Fabrice Florin 2011-12-10 20:14:24 UTC
This problem has not been solved to my satisfaction, based on what is on
prototype now.

For example, this unlisted article should not be displaying AFTv5, because it
is not in the 'Article Feedback 5' category:
http://prototype.wikimedia.org/release-en/AFT5_unlisted_article

Please go through our testing page to verify that all conditions on that page
have been met:
http://prototype.wikimedia.org/release-en/AFT5_test_hub

See the same comments on bug 32857.

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


Navigation
Links