[ckan-dev] Recent upgrade (?) and issues

[Rufus Pollock]

On 12 June 2011 20:32, David Raznick <kindly at gmail.com> wrote:
> On Sun, Jun 12, 2011 at 3:42 PM, Rufus Pollock <rufus.pollock at okfn.org>
> wrote:
[..]
>> When trying to purge packages in ckan-admin/trash I get error messages of
>> form:
>>
>> Problem purging revision 0abb62c9-f1b0-4bdd-8010-1c30fff6a784: can
>> only use resources on packages if there is only one resource_group.
>
>
> I imagine that this is a deceptive error (made by me:)).  I think it is
> because the resource_group is being purged so the resources property on the
> package raises an assertion as it did not take into account a time when
> there were 0 resource_groups.
> Anyway, I have added a fix to for this logical error (now on ckan.net) which
> may or may not fix this issue as I have not thought of solid test case yet.
> Could you tell me if it does...

It does! Great work.

>> I note that purge seemed to work ok for packages with only one
>> revision but failed with those with more than one revision. The purge
>> method in vdm works by deleting revisions 'backwards' from most recent
>> and updating the continuity. It then finally deletes the continuity
>> object if you delete the last revision of that object. I'm wondering
>> if the vdm purge method needs updating in some way to reflect latest
>> changes in core code ...?
>
> There are many issues flying around due to purging at the moment and I was
> going to look at them in bulk.   If you purge a package with a group it
> fails too.  I am not sure this has ever worked.
>
> Also, PackageGroup revisioning has *never* worked.  If you remove a
> relationship between a package and a group it gets deleted from the
> package_group table physically (instead of staying there with a delete
> state).  Also a package_group revision only gets made if you attach a
> package to a group and *not* if you attach a group to a package.  I have not
> reported these errors as they are in the process of being sorted out, with
> the work on moderated edits.

Great stuff.

Rufus