Bugfix, faltaba arrastrar otras variables de GET.

[z.facultad/75.43/tp1.git] / src / lib / file.creditos.php

diff --git a/src/lib/file.creditos.php b/src/lib/file.creditos.php
index c07370bf86cf5c2a3f163cb6485bbd46b1e6d8f9..4a9e9235679a1dfa17e0457cf9775806a7dd33c0 100644 (file)
--- a/src/lib/file.creditos.php
+++ b/src/lib/file.creditos.php
@@ -14,7 +14,12 @@ require_once 'lib/file.php';
  * Actualiza el archivo de créditos al loguearse.
  * Calcula si hay que restar créditos por días sin preguntar.
  *
- * @return false si hay error
+ * TODO ver si anda por el tema de las preguntas, que se pueden actualizar los
+ *      creditos sin que pregunte. Probablemente haya que agregar una fecha de
+ *      último chequeo para diferenciar entragas por login de entradas por
+ *      evaluar o preguntas (me parece que con 'dias' no alcanza).
+ *
+ * @return array con los campos escritos.
  */
 function file_creditos_login($user)
 {
@@ -25,7 +30,9 @@ function file_creditos_login($user)
     $dias2 %= 30;
     if ($dias2 != $dias or $creditos2 != $creditos)
     {
-        return (bool)file_creditos_write($user, $creditos2, $bonus, $dias2);
+        $ret = file_creditos_write($user, $creditos2, $bonus, $dias2);
+        if ($ret === false) return false;
+        else return array_slice($ret, 1);
     }
     return true;
 }
@@ -39,8 +46,7 @@ function file_creditos_login($user)
  */
 function file_creditos_preguntar($user)
 {
-    $row = file_creditos_get($user);
-    if (!$row) return false;
+    if (!($row = file_creditos_get($user))) return false;
     list($fecha, $creditos, $bonus, $dias) = $row;
     return file_creditos_write($user, $creditos - 1, $bonus, 0);
 }
@@ -53,16 +59,21 @@ function file_creditos_preguntar($user)
  */
 function file_creditos_evaluar($user, $es_bonus = false, $bonus_cant = 5)
 {
-    $row = file_creditos_get($user);
-    if (!$row) return false;
+    if (!($row = file_creditos_get($user))) return false;
     list($fecha, $creditos, $bonus, $dias) = $row;
-    if ($es_bonus) $bonus++;
-    if ($bonus >= $bonus_cant)
+    $bonus2 = $bonus;
+    $creditos2 = $creditos;
+    if ($es_bonus) $bonus2++;
+    if ($bonus2 >= $bonus_cant)
     {
-        $creditos += intval($bonus / $bonus_cant) * $bonus_cant;
-        $bonus %= $bonus_cant;
+        $creditos2 += intval($bonus2 / $bonus_cant) * $bonus_cant;
+        $bonus2 %= $bonus_cant;
     }
-    return file_creditos_write($user, $creditos, $bonus, $dias);
+    if ($bonus2 != $bonus or $creditos2 != $creditos)
+    {
+        return (bool)file_creditos_write($user, $creditos2, $bonus2, $dias);
+    }
+    return true;
 }
 
 /**
@@ -78,10 +89,21 @@ function file_creditos_get($user)
     return $row;
 }
 
+/**
+ * Agrega $n créditos al usuario $user (false si hay error).
+ */
+function file_creditos_add($user, $n)
+{
+    if (!($row = file_creditos_get($user))) return false;
+    list($fecha, $creditos, $bonus, $dias) = $row;
+    $creditos += $n;
+    return (bool)file_creditos_write($user, $creditos, $bonus, $dias);
+}
+
 /**
  * Escribe una nueva entrada en el archivo de créditos (actualiza el 'índice').
  *
- * @returns array con los campos escritos.
+ * @return array con los campos escritos.
  */
 function file_creditos_write($user, $creditos, $bonus, $dias)
 {