Index: openacs-4/packages/news-aggregator/sql/postgresql/upgrade/upgrade-0.9.7-0.9.8.sql
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/news-aggregator/sql/postgresql/upgrade/Attic/upgrade-0.9.7-0.9.8.sql,v
diff -u -N -r1.1.2.1 -r1.1.2.2
--- openacs-4/packages/news-aggregator/sql/postgresql/upgrade/upgrade-0.9.7-0.9.8.sql	14 Jan 2006 23:27:59 -0000	1.1.2.1
+++ openacs-4/packages/news-aggregator/sql/postgresql/upgrade/upgrade-0.9.7-0.9.8.sql	17 Jan 2006 23:07:28 -0000	1.1.2.2
@@ -55,13 +55,15 @@
 
 end;' language 'plpgsql';
 
--- MS: remove owner_id and package_id from na_sources as they are not used
--- title should be removed next if na_source remains an object (1/13/06)
+-- no real feed owner at this point, with feeds being shared among packages
 alter table na_sources 
     drop column owner_id;
-
+-- this is available in acs_objects beginnning with 5.2
 alter table na_sources 
     drop column package_id;
+-- customizing this should be a per user option
+alter table na_sources 
+    drop column title;
 
 -- add on delete cascade so packages can uninstall cleanly
 -- doing this for now, may eventually move this data to the user preferences package
Index: openacs-4/packages/news-aggregator/tcl/aggregator-procs.tcl
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/news-aggregator/tcl/aggregator-procs.tcl,v
diff -u -N -r1.1.1.1.2.4 -r1.1.1.1.2.5
--- openacs-4/packages/news-aggregator/tcl/aggregator-procs.tcl	14 Jan 2006 23:27:59 -0000	1.1.1.1.2.4
+++ openacs-4/packages/news-aggregator/tcl/aggregator-procs.tcl	17 Jan 2006 23:07:28 -0000	1.1.1.1.2.5
@@ -381,6 +381,7 @@
     @creation-date 2003-06-29
 } {
 
+    # MS: should really check for connection here (for 0.9.9)
     if { [string equal $package_id ""] } {
 	set package_id [ad_conn package_id]
     }
@@ -394,7 +395,9 @@
     set aggregator_id [db_exec_plsql new_aggregator {}]
 
     # find app group and assign correct privileges 
-    # (public = viewable by subsite members)
+    # (public = viewable by subsite members); this will be better in 5.2 with
+    # group::get_rel_segment (acording to daveb) as we'll be able to assign privs
+    # using the subsite member rel segment (rather than the entire app group)
     set app_group [application_group::group_id_from_package_id \
 		       -package_id [ad_conn subsite_id]]
     if { !$public_p } {
Index: openacs-4/packages/news-aggregator/tcl/source-procs.tcl
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/news-aggregator/tcl/source-procs.tcl,v
diff -u -N -r1.6.2.5 -r1.6.2.6
--- openacs-4/packages/news-aggregator/tcl/source-procs.tcl	14 Jan 2006 23:27:59 -0000	1.6.2.5
+++ openacs-4/packages/news-aggregator/tcl/source-procs.tcl	17 Jan 2006 23:07:28 -0000	1.6.2.6
@@ -12,8 +12,8 @@
 
 ad_proc -public news_aggregator::source::new {
     -feed_url:required
-    -user_id:required
     {-aggregator_id ""}
+    {-user_id ""}
     -array:boolean
 } {
     @author Simon Carstensen
Index: openacs-4/packages/news-aggregator/www/index.adp
===================================================================
RCS file: /usr/local/cvsroot/openacs-4/packages/news-aggregator/www/index.adp,v
diff -u -N -r1.12.2.7 -r1.12.2.8
--- openacs-4/packages/news-aggregator/www/index.adp	14 Jan 2006 23:27:59 -0000	1.12.2.7
+++ openacs-4/packages/news-aggregator/www/index.adp	17 Jan 2006 23:07:28 -0000	1.12.2.8
@@ -55,15 +55,15 @@
  <multiple name="items">
    <group column="pub_date"> 
       <div style="background-color: #eeeeee; padding-top: 6px; padding-bottom: 6px; padding-left: 4px; margin-top: 10px; margin-bottom: 10px;">
-       <span style="font-size: 125%; font-weight: bold;"><a href="@items.link@" title="@items.description@">@items.title;noquote@</a></span>, from @items.chunk_updated@ @items.pub_date@
+       <span style="font-size: 125%; font-weight: bold;"><a href="@items.link@" title="@items.description@">@items.title;noquote@</a></span>, from @items.chunk_updated@
       </div>
        <group column="source_id">
         <if @items.show_description_p@ true>
           <div style="margin-left: 10px; margin-top: 15px; margin-bottom: 15px;">
            <a name="@items.item_id@">
            <div style="font-size: 115%; font-weight: bold; margin-bottom: 5px;">
             <if @items.item_title@ not nil>
-             <a href="@items.item_link@">@items.item_title;noquote@</a> @items.pub_date@
+             <a href="@items.item_link@">@items.item_title;noquote@</a>
             </if>
            </div>
            <div class="item_pub_time" style="margin-bottom: 5px;">Posted: @items.pub_time@