• last updated 3 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
add location to ad_returnredirect

bump version and update todo

watch out for case when source title is nil

always set item_blog_url, save_url and unsave_url (checks as to whether to display or not are done in the adp); pass -- to string_truncate to be safe; pass user_id to permission_p

fix bug in subscriptions move/copy feature that appears when user can write to exactly 2 aggregators; fix link on index page; bump version to 1.0a

coupla simple changes to index page if conditions

bump version number in .info

file news-aggregator.css was initially added on branch oacs-5-2.

    • -0
    • +0
    ./www/resources/news-aggregator.css
straightened out grouping issue in display (ran into the old issue detailed here - http://openacs.org/bugtracker/openacs/bug?bug%5fnumber=428); backing out maltes upgrade script change as it doesn't break anything and will create inconsistent data model - portlet should really be updated to fix the issue; tagging for rel v0.9.9

    • -0
    • +66
    ./www/resources/news-aggregator.css
Removed deletion of package_id otherwise any upgrade is screwed

file multi-source-chunk.adp was initially added on branch oacs-5-2.

    • -0
    • +0
    ./www/multi-source-chunk.adp
file single-source-chunk.adp was initially added on branch oacs-5-2.

    • -0
    • +0
    ./www/single-source-chunk.adp
fixed display issue with group tag by separating the two cases (one source/multiple sources to display) into to includible chunks

updated todo

remove debugging for multiple group by; drop title column for na_sources (not used by UI plus part of acs_objects with 5.2) and made user_id optional for source::new; still need to deal with the group by issue on index page - now, the first grouping (pub date) is not applied when there is only one source (source_id being the second grouping

some basic tests for aggregator and source apis

    • -0
    • +61
    ./tcl/test/news-aggregator-procs.tcl
file news-aggregator-procs.tcl was initially added on branch oacs-5-2.

    • -0
    • +0
    ./tcl/test/news-aggregator-procs.tcl
file upgrade-0.9.7-0.9.8.sql was initially added on branch oacs-5-2.

    • -0
    • +0
    ./sql/postgresql/upgrade/upgrade-0.9.7-0.9.8.sql
file toggle-show-description.tcl was initially added on branch oacs-5-2.

    • -0
    • +0
    ./www/toggle-show-description.tcl
file aggregator-edit.tcl was initially added on branch oacs-5-2.

    • -0
    • +0
    ./www/aggregator-edit.tcl
file aggregator-edit.adp was initially added on branch oacs-5-2.

    • -0
    • +0
    ./www/aggregator-edit.adp
lots of little tweaks for data cleanup on uninstantiate, some more UI work, bug fixes, etc.; added daveb's sweeper proc; not tagging v0.9.8 just yet though

    • -0
    • +75
    ./sql/postgresql/upgrade/upgrade-0.9.7-0.9.8.sql
    • -2
    • +9
    ./tcl/news-aggregator-procs-postgresql.xql
  1. … 11 more files in changeset.
file todo was initially added on branch oacs-5-2.

thought about puting off adding back the constraints a few version until we're pretty sure anyone out there with older versions has had a chance to get through earlier upgrades but then thought better of it

    • -0
    • +3
    ./sql/postgresql/upgrade/upgrade-0.9.4d-0.9.5b.sql
thought about puting off adding back the constraints a few version until we're pretty sure anyone out there with older versions has had a chance to get through earlier upgrades but then thought better of it

    • -3
    • +0
    ./sql/postgresql/upgrade/upgrade-0.9.4d-0.9.5b.sql
duh... bad comment in upgrade script

bump version to 0.9.7 and tag for release

fix up subscription move/copy

add a few indexes

file upgrade-0.9.6b-0.9.7.sql was initially added on branch oacs-5-2.

    • -0
    • +0
    ./sql/postgresql/upgrade/upgrade-0.9.6b-0.9.7.sql