Ticket #1533 (closed defect: wontfix)

Opened 2 years ago

Last modified 2 years ago

Make sure ckanext-wordpresser works on CKAN 1.5.1

Reported by: amercader Owned by: rgrp
Priority: major Milestone: ckan-backlog
Component: ckan Keywords:
Cc: [email protected]…, rgrp, [email protected] Repository: ckan
Theme: none

Description (last modified by ross) (diff)

Superceded by #2234

Super ticket: #1508

Etherpad: http://ckan.okfnpad.org/wordpresser

The wordpresser extension does not work with ckan 1.5.1. The wordpress page is not loaded.

Looks like this changeset may be involved https://github.com/okfn/ckanext-wordpresser/commit/0efa0649896a7339865f178a302be639ddc561e1

I think that even before that changeset, the page was shown without any margins.

In any case, the extension seems to be too sensible to changes in the ckan layout so perhaps it should take into account the CKAN version and have a fixed set of xpaths and templates that work for each version

Change History

comment:1 Changed 2 years ago by amercader

  • Owner set to ross
  • Status changed from new to assigned

comment:2 Changed 2 years ago by ross

Investigating:

  1. The XPATH that the middleware is currently using will definitely not match the navigation on 1.5.1, we'll need to change the xpath and also extract the anchors from the LI tags that are retrieved from the wordpress site.
  1. Getting the main wordpress content just doesn't seem to work at all. Lots of class/id confusion.

Will fix for 1.5.1 in branch

comment:3 Changed 2 years ago by ross

  • Milestone changed from current-ckan-sprint-2011-12-19 to ckan-backlog

comment:4 Changed 2 years ago by rgrp

  • Description modified (diff)

comment:5 Changed 2 years ago by rgrp

  • Description modified (diff)
  • Milestone changed from ckan-backlog to current-ckan-sprint-2012-01-09

comment:6 Changed 2 years ago by rgrp

  • Milestone changed from current-ckan-sprint-2012-01-09 to ckan-v1.6

comment:7 Changed 2 years ago by rgrp

Moving to v1.6 backlog. Sort of working but ross suggests we should be using atom feeds.

comment:8 Changed 2 years ago by rgrp

  • Owner changed from ross to rgrp
  • Milestone changed from ckan-v1.6 to current-ckan-sprint-2012-01-09

Reassigning to me as per chat with Ross.

comment:9 Changed 2 years ago by rgrp

  • Milestone changed from current-ckan-sprint-2012-01-09 to ckan-v1.6

Not really urgent in my view atm. Propose to defer at least out of this sprint. Needs further discussion of purpose and implementation.

comment:10 Changed 2 years ago by rgrp

  • Milestone changed from ckan-v1.6 to ckan-v1.7

Moving out of v1.6 (see previous comment).

comment:11 Changed 2 years ago by ross

  • Status changed from assigned to closed
  • Resolution set to wontfix

This has been superceded by #2234

comment:12 Changed 2 years ago by ross

  • Description modified (diff)
  • Milestone changed from ckan-v1.7 to ckan-backlog
Note: See TracTickets for help on using tickets.