initialized objects and improved coding style
authorPetr Skoda <skodak@moodle.org>
Sat, 18 Sep 2010 12:46:11 +0000 (12:46 +0000)
committerPetr Skoda <skodak@moodle.org>
Sat, 18 Sep 2010 12:46:11 +0000 (12:46 +0000)
mod/data/field.php
mod/data/lib.php
mod/data/templates.php
mod/data/view.php

index 2403024..b04770b 100755 (executable)
@@ -187,7 +187,7 @@ switch ($mode) {
 
                     // Update the default sort field
                     if ($fid == $data->defaultsort) {
-                        unset($rec);
+                        $rec = new object();
                         $rec->id = $data->id;
                         $rec->defaultsort = 0;
                         $rec->defaultsortdir = 0;
@@ -220,6 +220,7 @@ switch ($mode) {
 
     case 'sort':    // Set the default sort parameters
         if (confirm_sesskey()) {
+            $rec = new object();
             $rec->id = $data->id;
             $rec->defaultsort = $defaultsort;
             $rec->defaultsortdir = $defaultsortdir;
index 9b59013..8183a44 100755 (executable)
@@ -2524,12 +2524,12 @@ function data_export_csv($export, $delimiter_name, $dataname, $count, $return=fa
     global $CFG;
     require_once($CFG->libdir . '/csvlib.class.php');
     $delimiter = csv_import_reader::get_delimiter($delimiter_name);
-    $filename = clean_filename("${dataname}-${count}_record");
+    $filename = clean_filename("{$dataname}-{$count}_record");
     if ($count > 1) {
         $filename .= 's';
     }
     $filename .= clean_filename('-' . gmdate("Ymd_Hi"));
-    $filename .= clean_filename("-${delimiter_name}_separated");
+    $filename .= clean_filename("-{$delimiter_name}_separated");
     $filename .= '.csv';
     if (empty($return)) {
         header("Content-Type: application/download\n");
@@ -2563,7 +2563,7 @@ function data_export_csv($export, $delimiter_name, $dataname, $count, $return=fa
 function data_export_xls($export, $dataname, $count) {
     global $CFG;
     require_once("$CFG->libdir/excellib.class.php");
-    $filename = clean_filename("${dataname}-${count}_record");
+    $filename = clean_filename("{$dataname}-{$count}_record");
     if ($count > 1) {
         $filename .= 's';
     }
@@ -2598,7 +2598,7 @@ function data_export_xls($export, $dataname, $count) {
 function data_export_ods($export, $dataname, $count) {
     global $CFG;
     require_once("$CFG->libdir/odslib.class.php");
-    $filename = clean_filename("${dataname}-${count}_record");
+    $filename = clean_filename("{$dataname}-{$count}_record");
     if ($count > 1) {
         $filename .= 's';
     }
@@ -2800,7 +2800,7 @@ function data_extend_settings_navigation(settings_navigation $settings, navigati
     $groupmode = groups_get_activity_groupmode($PAGE->cm);
 
     if (data_user_can_add_entry($data, $currentgroup, $groupmode)) { // took out participation list here!
-        if (empty($editentry)) {
+        if (empty($editentry)) { //TODO: undefined
             $addstring = get_string('add', 'data');
         } else {
             $addstring = get_string('editentry', 'data');
index 1c89b6d..beb8ecf 100755 (executable)
@@ -112,6 +112,7 @@ include('tabs.php');
 $resettemplate = false;
 
 if (($mytemplate = data_submitted()) && confirm_sesskey()) {
+    $newtemplate = new object();
     $newtemplate->id = $data->id;
     $newtemplate->{$mode} = $mytemplate->template;
 
index 45ddf0e..4deaf4e 100755 (executable)
@@ -405,6 +405,7 @@ if ($showactivity) {
         if ($approve && confirm_sesskey() && $approvecap) {
             if ($approverecord = $DB->get_record('data_records', array('id'=>$approve))) {   // Need to check this is valid
                 if ($approverecord->dataid == $data->id) {                       // Must be from this database
+                    $newrecord = new object();
                     $newrecord->id = $approverecord->id;
                     $newrecord->approved = 1;
                     $DB->update_record('data_records', $newrecord);