User talk:SoledadKabocha/linkclassifier

Latest comment: 8 years ago by SoledadKabocha in topic Critical bugs

TODO for on-demand mode

edit

Make the portlet link remove itself when clicked. This might require moving the logic that adds the link into linkclassifier.js, as opposed to asking the user to manually add code to the skin. Compare the way that markblocked does it.

(This is just a note to myself; I will not ask Anomie about this until I have implemented it successfully myself.) --SoledadKabocha (talk) 22:27, 30 October 2012 (UTC) (marked first paragraph as done 18:28, 2 November 2012 (UTC))Reply

Categories TODO

edit

--SoledadKabocha (talk) 18:30, 2 November 2012 (UTC) (last edit 05:03, 28 May 2015 (UTC))Reply

General code maintenance

edit
  • Alphabetize the added categories
  • Consider creating User:SoledadKabocha/linkclassifier-lite.js which contains only the categories from the original linkclassifier but incorporates my other code changes
  • Consider creating User:SoledadKabocha/linkclassifier-portable.js which contains only categories that are meaningful on multiple wikis, perhaps because their names are coded as MediaWiki configuration parameters, e.g. Category:Noindexed pages, Category:Pages containing omitted template arguments, Category:Pages where expansion depth is exceeded, Category:Pages where node count is exceeded, Category:Pages where post-expand include size is exceeded
    • (why didn't it seem to work?)
  • Figure out how to prevent the script from catching section edit links on pages viewed through redirects, e.g. Hr
  • Figure out why the tooltip gets removed entirely when the link is a redirect broken due to its target beginning with #, e.g. Talk:Thillai Nataraja Temple as of this revision (TODO: Create a test case in my own sandbox)
  • To make function chaining work more reliably, add a redundant method of counting API requests remaining (much as I tried in markBlockedPlus itself, except it is more difficult to predict the number of API requests that will be needed here). The problem is that in some cases either (1) markBlockedPlus starts too early and allows linkclassifier to stomp on its tooltips, or less commonly (2) markBlockedPlus doesn't seem to run at all. Was the jQuery library changed in a recent MW version to behave less deterministically?
  • Figure out how to fix the failure on User talk:ELISA N susie (I know the script is failing because it never gets to calling markBlockedPlus, which should be visible on the "User page" tab, and the Uninterruptible handler only gets removed after the 20-second timeout. I tried this fix but it didn't work) (incomplete: probably not a valid test case now that a block template has been added to said page. I'd have to blank my own user talk to create a similar case, and that might get me into some trouble)
  • To protect against future changes to the query-continue mechanism: (removed old inaccurate content)

--SoledadKabocha (talk) 22:42, 3 March 2013 (UTC) (last edit 05:04, 28 May 2015 (UTC))Reply

Invalid Unicode issue

edit

This edit summary should also have mentioned accented letters. (TODO: Document which letters exactly.) --SoledadKabocha (talk) 01:08, 16 November 2014 (UTC)Reply

Critical bugs

edit

Investigate why, on the article Pyromania (permanent link), the script correctly classifies some links in the lede but sometimes fails before it manages to add the redirect class to the "child pyromaniacs" link in the "Epidemiology" section. (NB: The example link is piped as it appears in the article.) When it fails, a hard reload sometimes helps. --SoledadKabocha (talk) 19:05, 30 December 2014 (UTC) (+ 17:37, 2 January 2015 (UTC))Reply

A similar problem with the link to National Geographic Magazine occurs in Swarm intelligence#Further reading. All other redirect links in that article appear to be classified correctly. --SoledadKabocha (talk) 02:28, 29 June 2015 (UTC)Reply
The above issue probably resulted from the use of synchronous AJAX in my common.js and other scripts; it may not have been a legitimate bug in this script. New issue:
On very long articles such as List of Internet top-level domains, the linkclassifier-finished class is sometimes added before the script is really finished. --SoledadKabocha (talk) 18:27, 26 November 2015 (UTC)Reply