Forum OpenACS Development: system-wide static parameter

Collapse
1: system-wide static parameter
Posted by Michael Totschnig on 05/08/07 11:55 AM
I am adding a feature (displaying numbers of unread messages per forum to user) to the forums package which we want to commmit to CVS Head. It should be possible to activate the feature with a site-wide parameter, i.e. all packages should have the same value for the parameter. This is because otherwise it would be possible to have a reasonable behaviour for the forums-portlet when it aggregates information about different packages, and probably it makes more sense to switch this feature on and off for the whole system.
If I understand it right, the package paramters are stored per instance. Is it possible to set a paramter for all instances of a package somehow?
Collapse
2: Re: system-wide static parameter (response to 1)
Posted by Nima Mazloumi on 05/08/07 12:21 PM
You would need to extend the data model and APM to store a parameter type for each parameter like "local" and "global" and also the requivalent parameter::get method. I think you have to TIP to the OCT before you commit, because you want to extend oacs core.
Collapse
3: Re: system-wide static parameter (response to 2)
Posted by Michael Totschnig on 05/08/07 02:02 PM
Thank you that's what I feared with respect to the package parameters. I thought maybe there might be an alternative approach without extending oacs core.
Collapse
4: Re: system-wide static parameter (response to 1)
Posted by Nima Mazloumi on 05/08/07 02:34 PM
No need to be afraid of. I believe that this feature is important and really needed. Just make a proposal. And make sure that the default behavior is not changed. This way your TIP - if designed properly - will go through more easily.
Collapse
5: Re: system-wide static parameter (response to 1)
Posted by Dave Bauer on 05/08/07 09:07 PM
This TIP has already been approved but not implemented.

I think it would be great if this could get into the next openacs release.

I think the design in this TIP covers your needs and most other uses of parameters.

https://openacs.org/forums/message-view?message_id=271910

Collapse
6: Re: system-wide static parameter (response to 5)
Posted by Michael Totschnig on 05/09/07 05:23 PM
I have implemented the part concerning parameter scope for postgres and submitted the patch for it:
https://openacs.org/bugtracker/openacs/patch?patch%5fnumber=845
Collapse
7: Re: system-wide static parameter (response to 6)
Posted by Michael Totschnig on 05/09/07 05:25 PM
Forgot to say: There is no upgrade script so far, can write one, if this is accepted for commital to CVS head.
Collapse
9: Re: system-wide static parameter (response to 7)
Posted by Michael Totschnig on 05/09/07 06:16 PM
Another point: I submitted a second version of the patch, since in the first one I missed retrieving the value.
Even now, I did not yet find a way how to change the parameter::get function, since it does some fairly complex stuff already. For the moment, I only changed parameter::get_from_package_key.
Collapse
8: Re: system-wide static parameter (response to 1)
Posted by Nima Mazloumi on 05/09/07 05:35 PM
great