• last updated 12 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Ported essential post-release fixes from oacs-5-10 branch

    • -8
    • +14
    /openacs-4/packages/acs-tcl/tcl/install-procs.tcl
    • -1
    • +1
    /openacs-4/packages/dotlrn/install.xml
improved comments

backport from oacs-head, otherwise the cache viewer is unusuable

fixed page contract for cache viewer

Bumped version number of acs-tcl to 5.10.2d2

    • -2
    • +2
    /openacs-4/packages/dotlrn/dotlrn.info
Fixed bug in install-from-repository (issue #3472)

Install from repository (e.g., of /dotlrn) was horribly broken, see,

e.g., issue #3472. Furthermore, this bug was hard to debug, since it

showed up just during a complex installation attempts involving

install.xml, and it requires fetching from the repository. Therefore,

the debug rounds were quite time-consuming.

There were several problems involved:

1) dotlrn/install.xml was trying to set the theme to the

dotlrn-bootstrap3-theme, which was not loaded from the repository

2) The loading order was not correct, since the theme requires an

existing dotlrn instance to register the theme templates there.

3) The tests in install::xml::action::mount checking, if the package

was already mounted were incorrect, probably since many years. It

looks to me as some refactorings of the "get_node_id" semantics

(many years ago) were not reflected in this code. This bug let

the code assume, that the package was already mounted. Therefore,

the package was never mounted, causing a long tail of subsequent

issues.

4) the site_node::instantiate_and_mount call in

install::xml::action::mount was called with incorrect parameters,

causing overwriting of the site root node, leaving the site

completely unusable.

Bumped version number of acs-tcl to 5.10.2d1

    • -2
    • +2
    /openacs-4/packages/acs-tcl/acs-tcl.info
    • -8
    • +14
    /openacs-4/packages/acs-tcl/tcl/install-procs.tcl
fixed overquoting in "ad_progress_base_end"

dirty editor buffer

    • -1
    • +1
    /openacs-4/packages/dotlrn/install.xml
switch responsibilities for theme setting

    • -1
    • +1
    /openacs-4/packages/dotlrn/install.xml
One more attempt to install bootstrap3 theme automatically

    • -1
    • +1
    /openacs-4/packages/dotlrn/install.xml
fixed cut&paste problem

    • -1210
    • +0
    /openacs-4/packages/acs-tcl/tcl/install-procs.tcl
reset to released version to keep differences little

    • -1
    • +1201
    /openacs-4/packages/acs-tcl/tcl/install-procs.tcl
deactivate automated theme setting for now

    • -1
    • +1
    /openacs-4/packages/dotlrn/install.xml
keep theme-stetting for dotlrn themes in dotlrn, since it requires a dotlrn instance

but when the dotlrn is installed, it requires the dotlrn-theme package to be installed

    • -1
    • +1
    /openacs-4/packages/dotlrn/install.xml
Continue working on the install.xml issue.

added more debugging info

undo one more recent change

undo recent change. This was not the problem

added debug statement

try to get a non-empty context id

added debugging line

Fixed incorrect result handling from ::acs::site_node get_node_id

When

::acs::site_node get_node_id -url /$mount_point

is executed with a non-existing mount point, the node info of the

parent_id is returned. The code assumed that for such cases, the

resulting "object_id" member is empty. Since this affects only the

two action procs

install::xml::action::mount

install::xml::action::mount-existing

i have adjusted the behavior there.

Furthermore, some tests are now less mind-twisted.

    • -13
    • +11
    /openacs-4/packages/acs-tcl/tcl/install-procs.tcl
strangely, parameter CommDefaultSiteTemplate_p was set on wrong package_id

Before this change, the install from repo on dotlrn led to error below.

this change makes sure, that the used "package_id" is the package_id from dotlrn.

: ERROR: -20000: The specified package 702 AND/OR parameter CommDefaultSiteTemplate_p do not exist in the system

: CONTEXT: PL/pgSQL function apm__id_for_name(integer,character varying) line 13 at RAISE

: PL/pgSQL function apm__set_value(integer,character varying,character varying) line 6 at assignment

oacs-5-10=# select * from apm_packages where package_key like '%dotlrn%';

package_id | package_key | instance_name | default_locale

------------+-------------------------+----------------------------+----------------

1843 | dotlrn-bm | dotLRN Bulk Mail Applet |

1850 | dotlrn-faq | dotLRN FAQ Applet |

1868 | dotlrn-forums | dotLRN Forums Applet |

1886 | dotlrn-fs | dotLRN File Storage Applet |

1933 | dotlrn-news | dotLRN News Applet |

1981 | dotlrn-portlet | dotLRN Portlet |

1998 | dotlrn-assessment | dotLRN Assessment Applet |

2040 | dotlrn-bootstrap3-theme | dotLRN Bootstrap 3 Theme |

oacs-5-10=# select * from apm_packages where package_id = 702;

package_id | package_key | instance_name | default_locale

------------+-------------+------------------------+----------------

702 | acs-subsite | #acs-kernel.Main_Site# |

moved theme setting to theme package (dotlrn-bootstrap3-theme)

This make sure that in the loading sequence, the theme is available when one

tries to switch to it.

    • -1
    • +1
    /openacs-4/packages/dotlrn/install.xml
added "dotlrn-bootstrap3-theme" to dependencies to make it installable via "install.xml" for install-from-repository

    • -2
    • +3
    /openacs-4/packages/dotlrn/dotlrn.info
fixed dependencies

improved log messages

bump maturity from 1 to 2

fixed typo in filename of image

Cleanup stale sse connections