Hi, i was just looking over the dev snapshot (2009-Mar-11) diff and I saw something that caught my attention. Here is a part of the diff that i was looking at:

        $old = db_query("SELECT * FROM {content_".$name."}");
                while ($field = db_fetch_array($old)) {
-                               $start = office_hours_tf_to_mil($field->$name."_starthours");
-                               $end = office_hours_tf_to_mil($field->$name."_endhours");
-                       $ret[] = update_sql("UPDATE {content_".$name."} SET ".$name."_starthours = ".$start.",".$name."_endhours = ".$end." WHERE nid = %d", $field->nid);
+                       list($hr, $min) = explode(":",$field->$name."_starthours");

The problem is that you make a $field = db_fetch_array instead of $field = db_fetch_object
Hope this helps.

Comments

Ozeuss’s picture

Ozeuss commented
Status: Active » Fixed

Thanks! somehow i missed that. fixed committed

Jeffdo’s picture

Jeffdo commented

Hi Ozeuss. The snapshot is suddenly dated yesterday (10th). You're working on it right now? I was just about to install this puppy ...

Jeff

Ozeuss’s picture

Ozeuss commented

The snapshot is dated today (11th)..
I'm not working on it, so you can download it and just make sure the above fix in the .install file is there (i made the commit just about half an hour ago and the tgz dev version gets updated by the drupal.org every 12 hours). a cvs checkout should be updated right away.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.