Changeset 746 for trunk/app/models

Timestamp:

09/01/08 20:58:25 (3 months ago)

Author:

aarkerio

Message:

Little Bugs fixed

Location:

trunk/app/models

Files:

• test.php (modified) (1 diff)
• vclassroom.php (modified) (7 diffs)

trunk/app/models/test.php

@@ -71 +71 @@
  {
   $points = (int) 0;
+  $conditions = array('Result.test_id'=>$test_id, 'Result.user_id'=>$user_id, 'Result.vclassroom_id'=>$vclassroom_id);
+  $this->Result->unbindAll();  // removeds unnecesary stuff
+  $answers    = $this->Result->findAll($conditions);  // get the answers gived by student
 
-  $conditions = array('Result.test_id'=>$test_id, 'Result.user_id'=>$user_id, 'Result.vclassroom_id'=>$vclassroom_id);
-  
-  $this->Result->unbindAll();  // removeds unnecesary stuff
+  if ( count($answers) < 1):    // test not answer yet
+    return null;
+  endif;
 
-  $answers    = $this->Result->findAll($conditions);  // get the answers gived by student
-  
   foreach($answers as $a):
-    $question = $this->Question->find(array('Question.id'=>$a['Result']['question_id']), array('worth')); // how much points question have?
+    $question = $this->Question->find(array('Question.id'=>$a['Result']['question_id']), array('worth')); //how much points question have?
     foreach($question['Answer'] as $qa):
         if ($qa['id'] == $a['Result']['answer_id'] && $qa['question_id'] == $a['Result']['question_id'] && $qa['correct'] == 1):

trunk/app/models/vclassroom.php

@@ -140 +140 @@
  {
    // 1) wee need check tree models: Test, Webquest and Treasures ans results associated to them:
-   // Result, ResultTreasure and ResultWebquest, so will get the stundents answer  
+   // Result, ResultTreasure and ResultWebquest, so will get the students answer  
   try{   
      $record                  = array();
@@ -151 +151 @@
      // Consult Test Model associated to this Vclassroom
      $this->TestsVclassroom->bindModel(array('belongsTo'=>array('Test')));
-     
      $conditions        = array('TestsVclassroom.vclassroom_id'=>$vclassroom_id);
      $fields            = array('TestsVclassroom.test_id', 'TestsVclassroom.id', 'Test.title'); 
      $record['tests']   = $this->TestsVclassroom->findAll($conditions, $fields);
-     // die(debug($record));
 
      // foreach test check in Result model
      foreach($record['tests'] as $k => $t):
-       $record['tests'][$k]['TestsVclassroom']['title'] = $t['Test']['title'];
-       $this->Test->Result->unbindAll();
-       $this->Test->Result->bindModel(array('belongsTo'=>array('Question','Test')));
-       
-       $conditions = array('Result.vclassroom_id'=>$vclassroom_id,'Result.user_id'=>$user_id,'Result.test_id'=>$t['TestsVclassroom']['test_id']);
-       $fields     = array('Result.correct', 'Result.answer_id','Question.worth');
-       $results    = $this->Test->Result->findAll($conditions, $fields);
-       //debug($results);
-       // Get points from answers
-       if ( count($results) > 0 ):
-            $points     = (int) 0;
-       else:
-            $points     = null;   //student has not answered this test yet
-       endif;
-       
-       foreach($results as $r):
-            if ($r['Result']['correct'] == 1):
-                  $points     += (int) $r['Question']['worth'];     
-        endif;
-       endforeach;
-       
-       $record['tests'][$k]['TestsVclassroom']['points'] = $points;
-     endforeach;
-     //die(debug($record['tests']));
+       $record['tests'][$k]['TestsVclassroom']['points'] = $this->Test->getPoints($t['TestsVclassroom']['test_id'], $user_id, $vclassroom_id);
+     endforeach;
+     // die(debug($record['tests']));
 
      // Now Treasure model
@@ -418 +395 @@
    endif; 
  }
-
+/*
+ *  studentPoints 
+ *   
+ */
  public function studentPoints($user_id, $vclassroom_id)
  {
@@ -424 +404 @@
      $points  = (int) 0; // student total points in eCourse 
    
-     // Consult Test Model 
-     /* $conditions = array('Result.vclassroom_id'=>$vclassroom_id, 'Result.user_id'=>$user_id);
-     $fields     = array('Result.points');
-     $tests      = $this->Test->Result->findAll($conditions, $fields);
+     // Consult TestsVclassroom  Model 
+     $conditions = array('TestsVclassroom.vclassroom_id'=>$vclassroom_id);
+     $fields     = array('TestsVclassroom.test_id');
+     $tests      = $this->TestsVclassroom->findAll($conditions, $fields);
+     
      foreach ($tests as $test):
-            $points += (int) $this->Test->getPoints($test['test']['id'], $user_id, $vclassroom_id);
-     endforeach;
-     die('pints  '. $points);*/
+            $points += (int) $this->Test->getPoints($test['TestsVclassroom']['test_id'], $user_id, $vclassroom_id);
+     endforeach;
      
      // Consult Treasure Model
@@ -456 +436 @@
             $points += $pa['Participation']['points'];
      endforeach;
-  
+
+     // Reports
+     $conditions     = array('Report.vclassroom_id'=>$vclassroom_id, 'Report.student_id'=>$user_id);
+     $fields         = array('Report.points'); //null; 
+     $reports        = $this->Report->findAll($conditions, $fields);
+     foreach ($reports as $rep):
+            $points += $rep['Report']['points'];
+     endforeach;
+     
      // Webquest
      $conditions     = array('ResultWebquest.vclassroom_id'=>$vclassroom_id, 'ResultWebquest.user_id'=>$user_id);
@@ -465 +453 @@
      endforeach;
          
-     //die(debug($record));
+     //die(debug($points));
      return $points;
   }
@@ -483 +471 @@
    $data        = $this->UsersVclassroom->findAll($conditions, $fields);
 
-   if ( $data == null )
-   {
+   if ( $data == null ):
      return null; 
-   }
-   else
-   {
+   else:
        $vclassrooms = array();
-
-       foreach ($data as $v)
-       {
-     foreach($v['UsersVclassroom'] as $m)
-     {
-         array_push($vclassrooms, $m);
-     }
-       }
-   }
+       foreach ($data as $v):
+      foreach($v['UsersVclassroom'] as $m):
+          array_push($vclassrooms, $m);
+      endforeach;
+       endforeach;
+   endif;
    return $vclassrooms;
  }
 
-  /**
-   * Adds a join record between two records of a hasAndBelongsToMany association
-   *
-   * @param mixed $assoc The name of the HABTM association
-   * @param mixed $assoc_ids The associated id or an array of associated ids
-   * @param integer $id The id of the record in this model
-   * @return boolean Success
-   */
-  public function addAssoc($assoc, $assoc_ids, $id = null)
-  {
-    if ($id != null) 
-    {
-      $this->id = $id;
-    }
-
-    $id = $this->id;
-
-    if (is_array($this->id)) 
-    {
-      $id = $this->id[0];
-    }
-        
-    if ($this->id !== null && $this->id !== false)
-    {
-      $db =& ConnectionManager::getDataSource($this->useDbConfig);
-            
-      $joinTable = $this->hasAndBelongsToMany[$assoc]['joinTable'];
-
-      $table = $db->name($db->fullTableName($joinTable));
-            
-      $keys[] = $this->hasAndBelongsToMany[$assoc]['foreignKey'];
-      $keys[] = $this->hasAndBelongsToMany[$assoc]['associationForeignKey'];
-      $fields = join(',', $keys);
-            
-      if ( !is_array($assoc_ids) ) 
-      {
-    $assoc_ids = array($assoc_ids);
-      }
-        
-      // to prevent duplicates
-      $this->deleteAssoc($assoc,$assoc_ids,$id);
-            
-      foreach ($assoc_ids as $assoc_id) 
-      {
-    $values[]  = $db->value($id, $this->getColumnType($this->primaryKey));
-    $values[]  = $db->value($assoc_id);
-    $values    = join(',', $values);
-        
-        $q = "INSERT INTO {$table} ({$fields}) VALUES ({$values})";
-
-        // echo $q;
-
-    $db->execute($q);
-
-    unset ($values);
-      }
-            
-      return true;
-    } 
-    else 
-    {
-      return false;
-    }
-  }
-
-  /**
-   * Deletes any join records between two records of a hasAndBelongsToMany association
-   *
-   * @param integer $id The id of the record in this model
-   * @param mixed $assoc The name of the HABTM association
-   * @param mixed $assoc_ids The associated id or an array of associated ids
-   * @return boolean Success
-   */
-  public function deleteAssoc($assoc, $assoc_ids, $id = null)
-  {
-    if ($id != null) 
-    {
-      $this->id = $id;
-    }
-
-    $id = $this->id;
-
-    if (is_array($this->id)) 
-    {
-      $id = $this->id[0];
-    }
-        
-    if ($this->id !== null && $this->id !== false) {
-      $db =& ConnectionManager::getDataSource($this->useDbConfig);
-            
-      $joinTable = $this->hasAndBelongsToMany[$assoc]['joinTable'];    
-      $table = $db->name($db->fullTableName($joinTable));
-            
-      $mainKey = $this->hasAndBelongsToMany[$assoc]['foreignKey'];
-      $assocKey = $this->hasAndBelongsToMany[$assoc]['associationForeignKey'];
-            
-      if (!is_array($assoc_ids)) 
-      {
-    $assoc_ids = array($assoc_ids);
-      }
-            
-      foreach ($assoc_ids as $assoc_id) 
-      {
-    $db->execute("DELETE FROM {$table} WHERE {$mainKey} = '{$id}' AND {$assocKey} = '{$assoc_id}'");
-      }
-            
-       return true;
-    } 
-    else 
-    {
-        return false;
-    }
-  }
 /*
  *  check if student belongs to class group