Forum OpenACS Development: Error on CR update

Collapse
Posted by Malte Sussdorff on
Quick question. CR is at 5.2.0, trying an update, getting:

Failed to install Content Repository, version 5.4.0d7. The following error was generated:

psql:upgrade-5.2.0d16-5.2.0d17.sql:101: ERROR: relation "imagesi" does not exist CONTEXT: SQL statement "create rule images_r as on insert to imagesi do instead ( update cr_dummy set val = ( select content_revision__new( new.title, new.description, now(), new.mime_type, new.nls_language, case when new.text is null then new.data else new.text end, content_symlink__resolve(new.item_id), new.revision_id, now(), new.creation_user, new.creation_ip, new.object_package_id )); insert into images ( image_id, width, height ) values (cr_dummy.val, new.width, new.height); );" PL/pgSQL function "content_type__refresh_trigger" line 73 at execute statement SQL statement "select content_type__refresh_trigger ( $1 )" PL/pgSQL function "inline_0" line 9 at SQL statement

I don't care about this bug, I want to remove 5.2.0d16-5.2.0d17 in HEAD as we don't need it. Any objections ?

Collapse
2: Re: Error on CR update (response to 1)
Posted by Malte Sussdorff on
This error will haunt any person trying to upgrade from an old install (e.g. 5.1.5), which does include a couple thousand ]project-open[ installations.

As the script is not needed as it was duplicated in later installs, would it be possible to either remove or fix it in a way which does not break upgrades? That would be awesome, as it is hard to tell people "download the source code, remove the following files and then do the upgrade" 😊.

Obviously ]project-open[ could just provide the whole OpenACS core internally as it used to do so far, but my goal is to make sure, you can install ]project-open[ with the latest release OpenACS and the OpenACS released files from the OpenACS repository :).

Collapse
3: Re: Error on CR update (response to 2)
Posted by Dave Bauer on
Yes, I'll look into this.

I think you are correct we can remove the upgrade since it only really applied for 5.3->5.4 installs and upgrade to newer version than 5.4 don't require it.

Collapse
4: Re: Error on CR update (response to 2)
Posted by Iuri Sampaio on
Not only because it is hard to tell people (yes indeed), but also because it makes them wonder what the error is and if the script is there is because there is a reason to it.

In my case, I was trying to fix the script, debugging it to understand what "imagesi" was and what was its context.
Plus, i did create a new view called imagesi in the system just to make the script work. :)