http://localhost/ticket/1284 <p> Revisions aren't assigned to packages. Where are they coming from? This seems to have been occurring since the upgrade to 1.4.3b a couple of weeks ago? </p> en-us http://assets.okfn.org/p/ckan/img/ckan_logo_shortname.png http://localhost/ticket/1284 Trac 0.12.3 dread Mon, 05 Sep 2011 08:44:18 GMT http://localhost/ticket/1284#comment:1 http://localhost/ticket/1284#comment:1 <p> Fixed by rgrp in cset:1952445d2802 on release-v1.4.3 and default. Comment: </p> <p> do not create revision for changes to users as they are not revisioned. </p> <ul><li>This had already been removed in cset:8d6fde0e2196 (then in user controller) but Adria seems to have reinstated this when refactoring things into the logic layer. This 'bug' explains the huge number of empty revisions we have been seeing on ckan.net. </li></ul> Ticket dread Mon, 05 Sep 2011 08:50:03 GMT http://localhost/ticket/1284#comment:2 http://localhost/ticket/1284#comment:2 <ul> <li><strong>status</strong> changed from <em>new</em> to <em>closed</em> </li> <li><strong>resolution</strong> set to <em>fixed</em> </li> </ul> <p> I've added tests for this (to prevent yet another regression!) in cset:1b77ce8ce560. Closing. </p> Ticket rgrp Mon, 05 Sep 2011 09:19:18 GMT http://localhost/ticket/1284#comment:3 http://localhost/ticket/1284#comment:3 <p> I'm not sure this is the sort of thing we need tests for (we can have too many tests!). Something worse discussing ... </p> Ticket dread Mon, 05 Sep 2011 09:40:26 GMT http://localhost/ticket/1284#comment:4 http://localhost/ticket/1284#comment:4 <p> Really? How many times do you want to fix this bug again? For the sake of three lines of test... </p> Ticket