--- nfo/php/libs/com.newsblob.phphtmllib/widgets/data_list/SQLDataListSource.inc	2003/02/22 21:08:37	1.2
+++ nfo/php/libs/com.newsblob.phphtmllib/widgets/data_list/SQLDataListSource.inc	2004/05/06 12:59:55	1.3
@@ -335,7 +335,7 @@
      * @return string
      */
     function build_groupby_clause() {
-        $groupby = $this->_groupby_clause;
+        $groupby = $this->_db_options["groupby_clause"];
         if ($groupby) {
             $groupby = " GROUP BY $groupby ";
         }
@@ -421,12 +421,14 @@
      * Do the query to the DB and pull in all of
      * the records locally.
      *
+     * @return boolean - the query passed/failed.
      */
     function do_query() {
 		//execute the sql query.
 		//we don't actually get the results of
 		//the query here, when just execute it.
 		user_error("SQLDataListSource::do_query() - Child must override");
+        return false;
     }
 
 	/**
@@ -437,6 +439,7 @@
 	 */
 	function get_next_data_row() {
 	   user_error("SQLDataListSource::get_next_data_row() - Child must override");
+       return array();
 	}	
 
     /**