Add wikilinks fixing to content scrubber

Bug #383189 reported by Chris Rossi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
KARL3
Fix Released
Low
Shane Hathaway

Bug Description

This is the Phase 2 described in #381694.

In Karl2 wiki links were able to differ wildly from the page title they were link to, which in Karl3 broke a lot of wiki links, since we expect the link to match exactly the linked to page title.

As a quick fix, we have relaxed the matching algorithm used in karl.content.models.wiki:cook to allow the differences found in Karl2 to just work. Ultimately, though, we don't want to support the loose matching rules in Karl2. To this end we are going to update the content scrubber to alter any wikilinks which currently match loosely to match exactly their referenced page titles, and then revert the wiki page cooking logic to use exact matching again.

Changed in karl3:
assignee: nobody → Chris Rossi (chris-archimedeanco)
importance: Undecided → Medium
milestone: none → m17
Revision history for this message
Paul Everitt (paul-agendaless) wrote :

Triage

Changed in karl3:
milestone: m17 → m18
Revision history for this message
Paul Everitt (paul-agendaless) wrote :

I'm going to mark this as Low, as it is somewhat optional. The purpose of this task is to allow us to remove the hack.

Changed in karl3:
importance: Medium → Low
Revision history for this message
Paul Everitt (paul-agendaless) wrote :

Shuffle off to next week.

Changed in karl3:
milestone: m18 → m19
Revision history for this message
Paul Everitt (paul-agendaless) wrote :

Move to next week.

Changed in karl3:
milestone: m19 → m20
Revision history for this message
Chris Rossi (chris-archimedeanco) wrote :

Another week, another milestone.

Changed in karl3:
milestone: m20 → m21
Revision history for this message
Chris Rossi (chris-archimedeanco) wrote :

Rescheduling for this week.

Changed in karl3:
milestone: m21 → m22
Revision history for this message
Paul Everitt (paul-agendaless) wrote :

Punt this to beyond this current week, into next week (M24).

Changed in karl3:
milestone: m22 → m24
Revision history for this message
Paul Everitt (paul-agendaless) wrote :

I chatted with Shane, and we agreed that we ought to get out of the idea of sticking all issues related to X to just one person. I explained that our goal to close this task is to clean up some content and remove a shim we had in place. He can get the details from Chris.

Note that Shane is also working in this neighborhood on https://bugs.launchpad.net/karl3/+bug/399353

Changed in karl3:
assignee: Chris Rossi (chris-archimedeanco) → Shane Hathaway (shane-hathawaymix)
milestone: m24 → m23
Revision history for this message
Paul Everitt (paul-agendaless) wrote :

Let's make way for some other work today.

Changed in karl3:
milestone: m23 → m24
Revision history for this message
Paul Everitt (paul-agendaless) wrote :

Move out of this week.

Changed in karl3:
milestone: m24 → m25
Revision history for this message
Paul Everitt (paul-agendaless) wrote :

Ping pong, back to ChrisR. We have successfully punted on this for 2 months. [wink]

Changed in karl3:
assignee: Shane Hathaway (shane-hathawaymix) → Chris Rossi (chris-archimedeanco)
milestone: m25 → m26
Revision history for this message
Paul Everitt (paul-agendaless) wrote :

Move to next week.

Changed in karl3:
milestone: m26 → m27
Revision history for this message
Paul Everitt (paul-agendaless) wrote :

Let's move this to next week.

Changed in karl3:
milestone: m27 → m28
Changed in karl3:
assignee: Chris Rossi (chris-archimedeanco) → Shane Hathaway (shane-hathawaymix)
status: New → In Progress
Revision history for this message
Shane Hathaway (shane-hathawaymix) wrote :

Done in revision 3891.

Changed in karl3:
status: In Progress → Fix Committed
Changed in karl3:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.