Page MenuHomeMiraheze
Create Task
Maniphest T4682

When viewing recreated Cargo table, an MWException is thrown.
Closed, ResolvedPublic
Actions

Description

Wiki of Concern: https://pokemonmasters.miraheze.org

Description of Issue

I am trying to recreate the Cargo table for Template:Trainer and Template:Sync Pair Pokemon. The recreate seems to complete without issue UI-wise, but when trying to view the table, an MWException is thrown for both the main table and the replacement table. Attempting to recreate again will fail with the error stating that the replacement table already exists.

The table is also not visible from Special:CargoTables, so I can't delete and try recreating again.

I actually hit a maybe similar error before with Template:Move, but changing the table name (Moves -> SPMoves -> MovesTable) and creating a brand new table managed to get it working again. Unfortunately, my hacky workaround didn't work this time hah.

I'm not sure how to further debug this issue from here.

I imagine there's also probably a handful of stray tables now that probably should be cleaned up that I can't delete.

Thank you in advance for any assistance.

Event Timeline

Elaeagnifolia created this task.Aug 30 2019, 16:34
Herald added a subscriber: RhinosF1. · View Herald TranscriptAug 30 2019, 16:34
RhinosF1 added projects: Extensions, Site Reliability Engineering.Aug 30 2019, 18:26
Herald added subscribers: CnocBride, Void, Reception123, NDKilla. · View Herald TranscriptAug 30 2019, 18:26
RhinosF1 moved this task from Backlog to Deployed Extension Bugs on the Extensions board.Aug 30 2019, 18:26
RhinosF1 moved this task from Radar to Bugs on the Site Reliability Engineering board.

[5194592a30340c715f4a0458] 2019-08-30 18:27:01: Fatal exception of type "MWException" on https://pokemonmasters.miraheze.org/wiki/Special:CargoTables/Trainers - can Site Reliability Engineering check the log please?

RhinosF1 added a comment.EditedAug 30 2019, 18:32
In T4682#89126, @RhinosF1 wrote:

[5194592a30340c715f4a0458] 2019-08-30 18:27:01: Fatal exception of type "MWException" on https://pokemonmasters.miraheze.org/wiki/Special:CargoTables/Trainers - can Site Reliability Engineering check the log please?

/wiki/Special:CargoTables/Trainers MWException from line 238 of /srv/mediawiki/w/extensions/Cargo/includes/CargoUtils.php: Error: Table Trainers not found.

RhinosF1 assigned this task to Paladox.Aug 30 2019, 18:55
Paladox added a comment.EditedAug 30 2019, 19:05

I've gone ahead and deleted the cargo__Trainer* tables. Can you delete the table and create this through https://pokemonmasters.miraheze.org/wiki/Special:CreateTemplate please? I'm not sure why you had issues with recreating the table.

(Table was empty and also there was no entry for this in cargo_tables either).

Elaeagnifolia added a comment.Aug 30 2019, 20:36

I recreated the table but still encountered the error.

However, I did manage to identify what the issue was after I tried testing a hypothesis I had.

The error was coming from the String (size=1500) portion of the Cargo declare. As soon as I changed that piece, I recreated the table and it worked fine. I tested this on another Wiki not hosted on Miraheze with the most recent version of Cargo installed, and it errored out in a similar fashion. From Cargo documentation, this is valid syntax, so will probably have to take a look at Cargo's recent updates to see what changed and file an issue for the extension.

What is problematic is that, if a buggy replacement table exists, even if you fix the String (size=###) portion, the only way to recreate the table is to either change the table name being declared or escalate to someone with access to directly remove the erroneous table.

Having said that, I'm pretty sure I have some other orphaned erroneous tables from other templates as well.

If it's not too much trouble, could I just get all the Cargo tables on the Pokémon Masters Wiki to be deleted so that I can recreate all of them properly?

RhinosF1 added a comment.Oct 20 2019, 08:53

Status?

• Examknow triaged this task as Normal priority.Oct 24 2019, 02:06
Herald added a subscriber: • Examknow. · View Herald TranscriptOct 24 2019, 02:06
Elaeagnifolia closed this task as Resolved.Nov 6 2019, 23:25

Resolving this since it's no longer needed. I'll be closing the aforementioned Wiki due to community lack of interest (other existing sources with the information already).