• last updated 17 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Handle here as well the case, that we are just showcasing the portlet for "Portal Templates"

Handle here as well the case when this portlet is rendered in "Portal Templates"

Handle the case of "Portal Templates", when the portlet is only showcased and should not be interacted with

make check more robust

harden page_contract

Format code in a way to avoid false positives where a column name is taken as a deprecated proc

    • -9
    • +6
    /openacs-4/packages/forums/tcl/forums-procs.tcl
Cleanup deprecated api from test

Go ahead and reimplement also lc_time_utc_to_local

Reimplement lc_time_local_to_utc using lc_time_tz_convert

Improve test for lc_time_local_to_utc, showing that its behavior is inconsitent with respect to invalid input

harden page contract

perm-include checks already that privs are not empty

Use UTF-8 emoji HTML entities instead of static images to convert smileys in the forums package

This has the advantage of not requiring a request to serve the images. It also avoids the need to hardcode the image size, as they will be basically treated as fonts.

Whitespace cleanup

Increase proc coverage, deprecate proc in the process

Test cornercase behavior

Whitespace changes

Make Service Contract implementations private

Hardcode the word "Page" in the message keys as the German localization does

Many thanks to Markus Moser for reporting the inconsistency

Deprecate forum::new_questions_allow and forum::new_questions_deny, one should use forum::edit, which will also track the change, or just inline the query instead

    • -3
    • +11
    /openacs-4/packages/forums/tcl/forums-procs.tcl
Deprecate forum::new_questions_allowed_p and replace occurrences

    • -2
    • +8
    /openacs-4/packages/forums/tcl/forums-procs.tcl
Increase proc coverage

Make faq::notification_delivery::do_notification private

    • -1
    • +1
    /openacs-4/packages/faq/tcl/q-and-a-procs.tcl
Use better idioms to cleanup the user

Provide an api to create a new FAQ, so that Create+Read of an FAQ can be tested

    • -0
    • +23
    /openacs-4/packages/faq/tcl/faq-procs-oracle.xql
    • -0
    • +33
    /openacs-4/packages/faq/tcl/faq-procs.tcl
    • -19
    • +4
    /openacs-4/packages/faq/www/admin/faq-new-2.tcl
file faq-procs-oracle.xql was initially added on branch oacs-5-10.

    • -0
    • +0
    /openacs-4/packages/faq/tcl/faq-procs-oracle.xql
file faq-procs-postgresql.xql was initially added on branch oacs-5-10.

    • -0
    • +0
    /openacs-4/packages/faq/tcl/faq-procs-postgresql.xql
Go straight to the database skipping the many trivial stored procedures, this reduces divergency between Oracle and Postgres

feed_parser::test::parse_feed is also used by the news-aggregator package, make it public again, document this and claim its coverage

Make test api private

Deprecate feed_parser::sort_result, trivial wrapper over lreverse used only internally