Ticket #1107 (closed refactor: fixed)

Opened 3 years ago

Last modified 23 months ago

Move package autocomplete from package controller and move to API

Reported by: rgrp Owned by: tidy-up bitesize
Priority: minor Milestone:
Component: ckan Keywords:
Cc: Repository: ckan
Theme: none


Currently autocomplete method on package controller. This method should be in API (like other autocomplete methods).

Will need to update client code (just forms atm I think).

Change History

comment:1 Changed 3 years ago by thejimmyg

  • Milestone ckan-v1.5 deleted

Child ticket of #954

comment:2 Changed 2 years ago by dread

Notes to complete this:

  • ckan/logic/action/get.py:'def package_autocomplete' exists and is the one that should be used
  • need to ensure all places in the templates and public/scripts use this action directly, not dataset/autocomplete (or its alias package/autocomplete)
  • ckan/controllers/package.py:'def autocomplete' should be removed.
  • ckan/config/routes.py dataset/autocomplete should be removed.

comment:3 Changed 23 months ago by icmurray

  • Status changed from new to closed
  • Resolution set to fixed

autocomplete action has been deprecated in favour of /api/2/util/dataset/autocomplete : https://github.com/okfn/ckan/blob/master/ckan/controllers/package.py#L655

Note: See TracTickets for help on using tickets.