Forum OpenACS Development: get_value__parameter_id error

Collapse
1: get_value__parameter_id error
Posted by Cesareo Garci­a Rodicio on 11/05/13 10:05 PM
Hi

I'm checking this error when adding a comment:

ERROR: column "get_value__parameter_id" does not exist at character 128
QUERY: select attr_value from apm_parameter_values v
where v.package_id = get_value__package_id
and parameter_id = get_value__parameter_id

As Gustaf suggests it might be a problem upgrading (I think it's relating with some other problems when upgrading from 5.5 to 5.8).

I'll check this out and keep the solution here.

Regards,
Cesáreo

Collapse
2: Re: get_value__parameter_id error (response to 1)
Posted by Cesareo Garci­a Rodicio on 11/05/13 10:21 PM
I've checked if this function exists in my database (and it doesn't)

 \df get_value*
\df get_value*
                       List of functions
 Schema | Name | Result data type | Argument data types | Type
--------+------+------------------+---------------------+------
(0 rows)

Seeking with these problem, I see it has already reported but ...

I check apm-create.file and it seems to be fine;

> grep "parameter_id =" acs-kernel/sql/postgresql/apm-create.sql
    and parameter_id = v_parameter_id;
    and parameter_id = v_parameter_id;
     where parameter_id = v_parameter_id
     where parameter_id = v_parameter_id
Collapse
3: Re: get_value__parameter_id error (response to 2)
Posted by Gustaf Neumann on 11/06/13 08:09 AM
try the following when calling psql with you database name:
\x
\df+ apm__get_value
you should see two functions with different argument types.
-g
Collapse
5: Re: get_value__parameter_id error (response to 3)
Posted by Cesareo Garci­a Rodicio on 11/06/13 11:45 AM
Yes, two functions:

openacs=# \df apm__get_value*
\df apm__get_value*
List of functions
-[ RECORD 1 ]-------+-------------------------------------
Schema              | public
Name                | apm__get_value
Result data type    | character varying
Argument data types | character varying, character varying
Type                | normal
-[ RECORD 2 ]-------+-------------------------------------
Schema              | public
Name                | apm__get_value
Result data type    | character varying
Argument data types | integer, character varying
Type                | normal
Collapse
4: Re: get_value__parameter_id error (response to 2)
Posted by Gustaf Neumann on 11/06/13 09:21 AM
The bug was introduced between OpenACS 5.6.0 and OpenACS 5.7.0 and effects only upgrades. i wonder, why we don't see more complaints about this. Anyhow, i've added an upgrade script to the oacs-5-8 branch. Please either upgrade from CVS, or wait until tomorrow and do an "upgrade from repository" when you are using OpenACS 5.8.0.

-gustaf

Collapse
6: Re: get_value__parameter_id error (response to 4)
Posted by Cesareo Garci­a Rodicio on 11/06/13 11:47 AM
As you wish. It sounds "more general" to use "upgrade from repository" but I can test whether option you want.
Collapse
7: Re: get_value__parameter_id error (response to 6)
Posted by Gustaf Neumann on 11/06/13 06:24 PM
please try tomorrow "upgrade from repo" .. -g
Collapse
8: Re: get_value__parameter_id error (response to 7)
Posted by Cesareo Garci­a Rodicio on 11/07/13 01:38 PM
Update Kernel:
acs-admin ... Install Software ... Upgrade from Repository

Kernel from 5.8.1d3 to 5.8.1d4

Restart and ... voilá 😊 Thanks

Collapse
9: Re: get_value__parameter_id error (response to 1)
Posted by Gustaf Neumann on 11/10/13 01:12 PM
As a summary: The error from apm__get_value is not related to broken upgrades, the same error existed as well at openacs.org. The reason, why we do not see more complaints about this basic functionality (retrieving a package parameter) is that this SQL function is only seldomly called (e.g. parameter::get uses its own sql statements). In the future, we should either drop the function or use and extend it (currently there is no support for global package parameters from the sql functions).