Page MenuHomePhabricator

NicoV (Nicolas Vervelle)
User

Projects

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Thursday

  • Clear sailing ahead.

User Details

User Since
Oct 10 2014, 12:19 PM (300 w, 3 d)
Availability
Available
LDAP User
Unknown
MediaWiki User
NicoV [ Global Accounts ]

Recent Activity

Yesterday

NicoV added a comment to T257495: WPCleaner not updating due to lack of HTTP redirect handling in getdown library.

@bd808
I'm indeed not subscribed to cloud-announce, I will register for it then...

Mon, Jul 13, 7:18 AM · WPCleaner
NicoV added a comment to T257221: Issue with preview.

@Bakertheacre
No, the Cobra library is integrated in WPCleaner, there's no need to get it.
So, you mean that even "Expand templates" doesn't work? I have to add logs to understand at what point it goes wrong.

Mon, Jul 13, 7:15 AM · WPCleaner

Sat, Jul 11

NicoV updated subscribers of T257495: WPCleaner not updating due to lack of HTTP redirect handling in getdown library.

@bd808
The temporary setup would be for a few weeks (ideally in my mind a bit after the holidays), so that most users will use WPCleaner at least once. When they use it once, it will retrieve the new configuration and will be fixed.

Sat, Jul 11, 7:43 AM · WPCleaner

Fri, Jul 10

NicoV closed T257182: CW#8 Ignore titles with a comment after as Resolved.
Fri, Jul 10, 10:11 AM · WPCleaner
NicoV closed T257585: Restrict automatic bot fix for #19 (Headlines start with one "=") as Resolved.
Fri, Jul 10, 10:11 AM · WPCleaner
NicoV added a comment to T257495: WPCleaner not updating due to lack of HTTP redirect handling in getdown library.

@bd808
I've got no answer from getdown developers, so I went browsing their code: I highly doubt that getdown is handling any kind of HTTP redirect, so using a 301 instead of a 308 won't achieve anything. They don't seem to use any HTTP aware library, just basic java.net classes.

Fri, Jul 10, 7:22 AM · WPCleaner
NicoV moved T257585: Restrict automatic bot fix for #19 (Headlines start with one "=") from CheckWiki to Done on the WPCleaner board.

I've modified the code to do so. It will be available in the next release.

Fri, Jul 10, 6:46 AM · WPCleaner

Thu, Jul 9

NicoV created T257618: Version check at startup.
Thu, Jul 9, 7:31 PM · WPCleaner
NicoV created T257585: Restrict automatic bot fix for #19 (Headlines start with one "=").
Thu, Jul 9, 3:00 PM · WPCleaner
NicoV moved T257182: CW#8 Ignore titles with a comment after from CheckWiki to Done on the WPCleaner board.

It should be fixed. Available in the next release.

Thu, Jul 9, 11:51 AM · WPCleaner
NicoV added a comment to T257221: Issue with preview.

@Bakertheacre
You have 3 different options in the menu: Preview, Expand templates and Expand templates and Preview. Can you try each of them and tell me the result?

Thu, Jul 9, 10:33 AM · WPCleaner
NicoV added a comment to T257495: WPCleaner not updating due to lack of HTTP redirect handling in getdown library.

@bd808
Ok, for the explanation. Is there a possibility to change at least temporarily the way the redirect is done (not a 308)?

Thu, Jul 9, 6:17 AM · WPCleaner

Wed, Jul 8

NicoV added a comment to T257513: Can ISBN with incorrect syntax (WCW 69) suggest the template.

@Jerodlycett
I can't test now, but you can to modify the configuration, like the [[ [[Utilisateur:NicoV/WikiCleanerConfiguration#ISBN]] | French configuration ]], parameter general_isbn_templates, by adding a line ISBN|1,2,3,4,5,6,7,8,9,10|true.

Wed, Jul 8, 8:23 PM · WPCleaner
NicoV moved T257513: Can ISBN with incorrect syntax (WCW 69) suggest the template from Backlog to CheckWiki on the WPCleaner board.
Wed, Jul 8, 8:20 PM · WPCleaner
NicoV closed T257384: Redirect for digest.txt incorrect as Resolved.
Wed, Jul 8, 6:23 PM · cloud-services-team (Kanban), WPCleaner
NicoV created T257495: WPCleaner not updating due to lack of HTTP redirect handling in getdown library.
Wed, Jul 8, 6:23 PM · WPCleaner
NicoV updated the task description for T257384: Redirect for digest.txt incorrect.
Wed, Jul 8, 6:21 PM · cloud-services-team (Kanban), WPCleaner
NicoV added a comment to T234617: Toolforge. introduce new domain toolforge.org.

Problem with 308: Permanent Redirect.

Wed, Jul 8, 6:09 PM · Patch-For-Review, Goal, Toolforge, cloud-services-team (Kanban), Kubernetes
NicoV added a comment to T257384: Redirect for digest.txt incorrect.

But there's one thing that worries me a lot: existing installations don't seem to pickup the redirection properly and don't update themselves

Wed, Jul 8, 5:58 PM · cloud-services-team (Kanban), WPCleaner
NicoV moved T257384: Redirect for digest.txt incorrect from Backlog to Done on the WPCleaner board.
Wed, Jul 8, 5:54 PM · cloud-services-team (Kanban), WPCleaner
NicoV added a comment to T257384: Redirect for digest.txt incorrect.

Everything seems fixed now, but if you did modifications as suggested in my first comment, you need to update getdown.txt, where you should have

Wed, Jul 8, 5:54 PM · cloud-services-team (Kanban), WPCleaner
NicoV added a comment to T257384: Redirect for digest.txt incorrect.

@Jerodlycett
Yes, my suggestion only worked during the time with the problematic URL. Now that they are fixed, I need to change them also in the installer, in getdown configuration... In the process of doing so

Wed, Jul 8, 5:37 PM · cloud-services-team (Kanban), WPCleaner
NicoV created T257485: Upgrade HttpComponents.
Wed, Jul 8, 5:30 PM · WPCleaner
NicoV added a comment to T257384: Redirect for digest.txt incorrect.

Until I manage to understand how to properly deal with the domain migration of Toolforge, you can bypass the problem by editing the getdown.txt file in the folder where WPCleaner is installed.

Wed, Jul 8, 2:16 PM · cloud-services-team (Kanban), WPCleaner
NicoV added a comment to T254640: Default lighttpd config created by `webservice` breaks serving files starting with the same string as the tool's name under `--canonical`.

Hello. Is it the same problem for my tool WPCleaner, T257384 ?
The redirect removes the top directory in the path, breaking all the tools for each user as the tool can't update itself.

Wed, Jul 8, 10:17 AM · cloud-services-team (Kanban), Tools, Toolforge
NicoV added a comment to T257417: HTTP error 404 if a script has the same name as the project.

Hello. Is it the same problem for my tool WPCleaner, T257384 ?
The redirect removes the top directory in the path, breaking all the tools for each user as the tool can't update itself.

Wed, Jul 8, 10:17 AM · Tools
NicoV added a comment to T257384: Redirect for digest.txt incorrect.

This is due to Toolforge migration T234617, which introduces problems with filenames starting with the same name as the tool T257417, T254640, which is the case for WPCleaner. I will try to understand how to fix it later, very busy day...

Wed, Jul 8, 10:14 AM · cloud-services-team (Kanban), WPCleaner

Tue, Jul 7

NicoV closed T257285: Error 56: "Arrows in ASCII art" in it.wiki. How to fix it? as Resolved.
Tue, Jul 7, 8:44 PM · WPCleaner
NicoV updated the task description for T257236: Dump analysis peformance.
Tue, Jul 7, 7:24 PM · WPCleaner
NicoV updated the task description for T257236: Dump analysis peformance.
Tue, Jul 7, 1:54 PM · WPCleaner
NicoV updated the task description for T257236: Dump analysis peformance.
Tue, Jul 7, 1:49 PM · WPCleaner
NicoV added a comment to T255101: Add project WPCleaner to translatewiki.net.

@abi_ Ok, I've disabled the generation of the .po files in my build process.

Tue, Jul 7, 10:52 AM · WPCleaner, Language-Team (Language-2020-Focus-Sprint), translatewiki.net
NicoV added a comment to T257221: Issue with preview.

It looks like normal logs (accessing CheckWiki API or MediaWiki API). I will have to add logs for the Preview.

Tue, Jul 7, 10:32 AM · WPCleaner
NicoV updated the task description for T257236: Dump analysis peformance.
Tue, Jul 7, 10:30 AM · WPCleaner
NicoV updated the task description for T257236: Dump analysis peformance.
Tue, Jul 7, 10:28 AM · WPCleaner
NicoV updated the task description for T257236: Dump analysis peformance.
Tue, Jul 7, 10:20 AM · WPCleaner
NicoV moved T257285: Error 56: "Arrows in ASCII art" in it.wiki. How to fix it? from CheckWiki to Done on the WPCleaner board.

I've added the restriction for chem tags. It will be available in the next release

Tue, Jul 7, 10:15 AM · WPCleaner
NicoV moved T257285: Error 56: "Arrows in ASCII art" in it.wiki. How to fix it? from Backlog to CheckWiki on the WPCleaner board.
Tue, Jul 7, 9:56 AM · WPCleaner
NicoV added a comment to T257285: Error 56: "Arrows in ASCII art" in it.wiki. How to fix it?.

@Marcodpat Will it be ok if WPCleaner stops reporting #56 in <chem> tags?

Tue, Jul 7, 9:56 AM · WPCleaner
NicoV updated the task description for T257236: Dump analysis peformance.
Tue, Jul 7, 9:53 AM · WPCleaner
NicoV updated the task description for T257236: Dump analysis peformance.
Tue, Jul 7, 7:21 AM · WPCleaner
NicoV updated the task description for T257236: Dump analysis peformance.
Tue, Jul 7, 7:09 AM · WPCleaner
NicoV added a comment to T257236: Dump analysis peformance.

Detailed results for first part of enwiki (2020/07/06):

Tue, Jul 7, 7:06 AM · WPCleaner
NicoV updated the task description for T257236: Dump analysis peformance.
Tue, Jul 7, 7:06 AM · WPCleaner

Mon, Jul 6

NicoV updated the task description for T257236: Dump analysis peformance.
Mon, Jul 6, 7:44 PM · WPCleaner
NicoV added a comment to T257221: Issue with preview.

Then I'll have to add logs to understand what's going on.

Mon, Jul 6, 7:40 PM · WPCleaner
NicoV updated the task description for T257236: Dump analysis peformance.
Mon, Jul 6, 6:29 PM · WPCleaner
NicoV added a comment to T257221: Issue with preview.

@Bakertheacre Do you see anything else in the logs ?
This line in the log only is only the retrieval of article 9/11 in Movies Theory.

Mon, Jul 6, 6:19 PM · WPCleaner
NicoV updated the task description for T257236: Dump analysis peformance.
Mon, Jul 6, 6:16 PM · WPCleaner
NicoV created T257236: Dump analysis peformance.
Mon, Jul 6, 6:11 PM · WPCleaner
NicoV created T257182: CW#8 Ignore titles with a comment after.
Mon, Jul 6, 7:16 AM · WPCleaner

Sun, Jul 5

NicoV closed T256807: Option to change heading level individually as Resolved.

Released

Sun, Jul 5, 7:34 PM · WPCleaner
NicoV created T257141: Create builders.
Sun, Jul 5, 2:49 PM · WPCleaner
NicoV created T257140: Refactor contents/analysis.
Sun, Jul 5, 2:46 PM · WPCleaner
NicoV renamed T256456: Java 8 from Code: move from Java 7 to Java 8 to Java 8.
Sun, Jul 5, 2:43 PM · WPCleaner
NicoV moved T256456: Java 8 from Backlog to Refactoring on the WPCleaner board.
Sun, Jul 5, 2:43 PM · WPCleaner
NicoV moved T256807: Option to change heading level individually from Backlog to Done on the WPCleaner board.

@Bakertheacre I've done the improvements I thought about. If you think of other things to do, tell me.

Sun, Jul 5, 2:42 PM · WPCleaner
NicoV added a comment to T256807: Option to change heading level individually.

I've also improved #19 (Headlines start with one "=") : if there's only one error at the last title, it will be fixed automatically

Sun, Jul 5, 2:41 PM · WPCleaner
NicoV added a comment to T256807: Option to change heading level individually.

I've also improved #7 (Headings all start with three "=") : if there's only one title, it will be fixed automatically.

Sun, Jul 5, 2:08 PM · WPCleaner
NicoV added a comment to T256807: Option to change heading level individually.

@Bakertheacre
I've also improved #83 (Heading start with three "=" and later with level two) : if the second title is with level two, there's a suggestion to change the first title to the same level (this suggestion is also applied automatically). It will be in the next release.

Sun, Jul 5, 1:02 PM · WPCleaner

Sat, Jul 4

NicoV added a comment to T256807: Option to change heading level individually.

I noticed the Tool --> Preview does not do anything. Is that supposed to open a new web browser with my changes in it so I can verify before sending?

Sat, Jul 4, 7:49 PM · WPCleaner
NicoV added a comment to T256807: Option to change heading level individually.

@Bakertheacre I've modified WPCleaner for #25, is it better?
Do you see other situations where this option could be beneficial?
I'm worried of adding it to places where it will result in incorrect modifications by the user, so I prefer avoiding to do it. For example, I won't suggest for Title 2 (because Title 3 is at the same level than Title 2), but I will suggest it for Title 5 (because Title 6 is at a correct level compared to Title 4). I've tested it on frwiki, and I have the feeling that it accounts for about half the errors for #25.

Sat, Jul 4, 7:40 PM · WPCleaner
NicoV added a comment to T256807: Option to change heading level individually.

I'm adding the right clicking for #25 (heading hierarchy) when there's only one subtitle that has the problem (to avoid people changing only one title when there are several to change)

Sat, Jul 4, 6:53 PM · WPCleaner
NicoV added a comment to T256807: Option to change heading level individually.

@Bakertheacre When a heading is in error, you usually have a "Edit TOC" function when you right-click on it : it allows to change the level of the titles, several at a time (because often to fix an error with the level you have to change several titles)

Sat, Jul 4, 6:37 PM · WPCleaner
Bakertheacre awarded T256548: Ending comments at the end of the line a Y So Serious token.
Sat, Jul 4, 6:28 PM · WPCleaner
NicoV closed T256319: Parser bug with km² to <sup> tags as Resolved.

Released

Sat, Jul 4, 6:07 PM · WPCleaner
NicoV closed T256322: WCW Error #8 caused by ref after header undetected as Resolved.

Released

Sat, Jul 4, 6:07 PM · WPCleaner
NicoV closed T256548: Ending comments at the end of the line as Resolved.

Released

Sat, Jul 4, 6:07 PM · WPCleaner
NicoV created T257122: Automatic fixing on dump.
Sat, Jul 4, 5:42 PM · WPCleaner
NicoV moved T257119: Enable CHECKWIKI error #59 for automated fixing in bot mode from Backlog to Done on the WPCleaner board.
Sat, Jul 4, 4:55 PM · WPCleaner
NicoV created T257115: Reporting of edit failures due to AbuseFilter and SpamBlacklist.
Sat, Jul 4, 2:45 PM · WPCleaner
NicoV created T257114: Integration with Citation tool.
Sat, Jul 4, 2:38 PM · WPCleaner
NicoV added a comment to T256548: Ending comments at the end of the line.

@Jerodlycett : The modification I just did, only add, as the first option, the suggestion to end the comment at the end of the line if there's another comment beginning further down. Is it enough ?

Sat, Jul 4, 2:31 PM · WPCleaner
NicoV moved T256548: Ending comments at the end of the line from Backlog to Done on the WPCleaner board.

This should be fixed. It will be available in the next release.

Sat, Jul 4, 2:29 PM · WPCleaner
NicoV renamed T256548: Ending comments at the end of the line from Ending Comments to Ending comments at the end of the line.
Sat, Jul 4, 2:08 PM · WPCleaner
NicoV claimed T256548: Ending comments at the end of the line.

The current suggestion is not to close the comment at end of the next line, but to close it just before the next one begins... In the example, it results in ending it on the next line, but that's because the next comment begins there.

Sat, Jul 4, 2:05 PM · WPCleaner
NicoV updated the task description for T256548: Ending comments at the end of the line.
Sat, Jul 4, 2:02 PM · WPCleaner
NicoV moved T256322: WCW Error #8 caused by ref after header undetected from CheckWiki to Done on the WPCleaner board.

It should be fixed in the next release.
Such cases should be detected with 2 suggestions :

Sat, Jul 4, 1:50 PM · WPCleaner
NicoV claimed T256322: WCW Error #8 caused by ref after header undetected.

Currently, it's detected at least by #504 (reference in a title), I will see for it to be detected also by #8

Sat, Jul 4, 12:42 PM · WPCleaner
NicoV added a comment to T256322: WCW Error #8 caused by ref after header undetected.

Example on enwiki :

Sat, Jul 4, 12:40 PM · WPCleaner
NicoV claimed T256319: Parser bug with km² to <sup> tags.
Sat, Jul 4, 12:31 PM · WPCleaner
NicoV added a comment to T256548: Ending comments at the end of the line.

Hi @Bakertheacre , yes it's the intended behavior for the replacement : it doesn't change the final result, just properly close the first comment.

Sat, Jul 4, 12:31 PM · WPCleaner
NicoV moved T256319: Parser bug with km² to <sup> tags from Backlog to Done on the WPCleaner board.

Should be fixed, will be available in the next release, after I've performed some extra testing (modifying the parser can have strange side effects sometimes...).

Sat, Jul 4, 12:25 PM · WPCleaner
NicoV closed T257098: #534: Bug, replacing || by | in a table as Resolved.

Fixed.

Sat, Jul 4, 8:44 AM · WPCleaner
NicoV created T257098: #534: Bug, replacing || by | in a table.
Sat, Jul 4, 7:16 AM · WPCleaner

Fri, Jul 3

NicoV closed T256667: #513 (internal link in external link): bug when there's no space character as Resolved.

Fixed.

Fri, Jul 3, 8:50 PM · WPCleaner

Thu, Jul 2

NicoV added a comment to T255101: Add project WPCleaner to translatewiki.net.

Yes, it seems fixed.

Thu, Jul 2, 7:09 PM · WPCleaner, Language-Team (Language-2020-Focus-Sprint), translatewiki.net
NicoV moved T256584: Error n degree 2 from Backlog to Done on the WPCleaner board.
Thu, Jul 2, 7:02 PM · WPCleaner

Wed, Jul 1

NicoV added a comment to T59753: VisualEditor: All metadata moved from bottom of the page into a reference.

@Pols12 I haven't seen it recently

Wed, Jul 1, 2:33 PM · VisualEditor, VisualEditor-DataModel
NicoV renamed T256807: Option to change heading level individually from An Idea to Option to change heading level individually.
Wed, Jul 1, 7:18 AM · WPCleaner

Mon, Jun 29

NicoV created T256667: #513 (internal link in external link): bug when there's no space character.
Mon, Jun 29, 5:50 PM · WPCleaner
NicoV closed T256584: Error n degree 2 as Resolved.

The problem is coming from Check Wiki project configuration, I fixed it.

Mon, Jun 29, 5:35 PM · WPCleaner

Sun, Jun 28

NicoV created T256581: Check all templates in an article.
Sun, Jun 28, 6:55 PM · WPCleaner
NicoV created T256580: Automatically load next page for edit.
Sun, Jun 28, 6:45 PM · WPCleaner
NicoV moved T256578: Restrict #501 (spelling and typography) in alt attribute of images from Backlog to CheckWiki on the WPCleaner board.
Sun, Jun 28, 6:41 PM · WPCleaner
NicoV moved T256577: False positives for #534 (Special:LintError/bogus-image-options) from Backlog to CheckWiki on the WPCleaner board.
Sun, Jun 28, 6:41 PM · WPCleaner
NicoV created T256578: Restrict #501 (spelling and typography) in alt attribute of images.
Sun, Jun 28, 6:40 PM · WPCleaner
NicoV created T256577: False positives for #534 (Special:LintError/bogus-image-options).
Sun, Jun 28, 6:36 PM · WPCleaner

Fri, Jun 26

NicoV created T256456: Java 8.
Fri, Jun 26, 9:02 AM · WPCleaner

Wed, Jun 24

NicoV moved T256321: {{largethumb}} and WCW Error #534 from Backlog to CheckWiki on the WPCleaner board.
Wed, Jun 24, 9:08 PM · WPCleaner