Deja solo los /n para el fin de linea. Corrige la observacion de luca. Utiliza la...

diff --git a/src/data/preguntas.txt b/src/data/preguntas.txt
index c7026ddee34d4b87af761d69783f760acecebb40..31048eaa5b9315372b8c34363a53649c58607415 100644 (file)
--- a/src/data/preguntas.txt
+++ b/src/data/preguntas.txt
@@ -1,3 +1,2 @@
-0|1|1115001612|Esta es la pregunta 1?\r
-1|2|1115001629|Esta es la pregunta 2, se lee?\r
-2|4|1115001925|Otra Pregunta mas?\r
+0|1|1115255646|Anda??
+1|2|1115255655|Que es?

diff --git a/src/data/respuestas.txt b/src/data/respuestas.txt
index 6a378fee0fa4e0d2b904dc739ba81d029c7e02f7..4d51dbdac31aae1d47c4c0944fe52e0405c09f32 100644 (file)
--- a/src/data/respuestas.txt
+++ b/src/data/respuestas.txt
@@ -1,7 +1 @@
-0|1|1115003171|SI, es\r
-0|1|1115003208|Claro!!!!!!!\r
-2|1|1115003306|puede ser\r
-1|1|1115003435|hola rta 1 a pregunta 2\r
-0|1|1115003464|no anda\r
-2|1|1115004853|otra respuesta\r
-2|1|1115004963|Otra mas\r
+1|1|1115255717|Esta es la respuesta a la pregunta 2

diff --git a/src/faq.desactivarPregunta.php b/src/faq.desactivarPregunta.php
index cdc950523097eedac9e613b397a92b5f0b03888c..e7d9b77ad1b69cbd98b95bb0ddb6b26cfe594699 100644 (file)
--- a/src/faq.desactivarPregunta.php
+++ b/src/faq.desactivarPregunta.php
@@ -8,13 +8,13 @@
 //
 // $Id$
 
+require_once 'lib/pagina.php';
 require_once 'lib/Pregunta.php';
 require_once 'lib/faq.functions.php';
-require_once 'lib/pagina.php';
 require_once 'lib/validacion.php';
 require_once 'lib/faq.forms.php';
 
-marco_cabecera('Desactivar una Pregunta');
+pagina_cabecera('Desactivar una Pregunta');
 
 // Si me llegó el form completo
 if (isset($_POST['PreguntaElegida']))
@@ -28,6 +28,6 @@ else
        faq_form_desactivar_pregunta();
 }
 
-marco_pie('Jonathan Schein', 'jonathanschein@fibertel.com.ar');
+pagina_pie('Jonathan Schein', 'jonathanschein@fibertel.com.ar');
 
-?>
\ No newline at end of file
+?>

diff --git a/src/faq.ingresarPregunta.php b/src/faq.ingresarPregunta.php
index ccfe630bb7eb3d396193d3c9c49643c425ab9d1d..410e6605763dcb4b419435b28ae8fbbc8e617a8e 100644 (file)
--- a/src/faq.ingresarPregunta.php
+++ b/src/faq.ingresarPregunta.php
@@ -8,13 +8,13 @@
 //
 // $Id$
 
+require_once 'lib/pagina.php';
 require_once 'lib/Pregunta.php';
 require_once 'lib/faq.functions.php';
-require_once 'lib/pagina.php';
 require_once 'lib/validacion.php';
 require_once 'lib/faq.forms.php';
 
-marco_cabecera('Ingreso de Preguntas');
+pagina_cabecera('Ingreso de Preguntas');
 
 // Si me llegó el form completo
 if (isset($_POST['UserId']) and isset($_POST['Pregunta']))
@@ -29,6 +29,6 @@ else
     faq_form_ingresar_pregunta();
 }
 
-marco_pie('Jonathan Schein', 'jonathanschein@fibertel.com.ar');
+pagina_pie('Jonathan Schein', 'jonathanschein@fibertel.com.ar');
 
 ?>
\ No newline at end of file

diff --git a/src/faq.listadoPreguntas.php b/src/faq.listadoPreguntas.php
index a1cede1c1077df2558b90a1e75f3f040b1735ec6..573be74e04be50adb5c82e0dae66220ea135c3dd 100644 (file)
--- a/src/faq.listadoPreguntas.php
+++ b/src/faq.listadoPreguntas.php
@@ -8,13 +8,16 @@
 //
 // $Id$
 
-require_once 'lib/faq.functions.php';
 require_once 'lib/pagina.php';
+require_once 'lib/faq.functions.php';
+
+// Chequea que se este logueado
+pagina_check_login();
 
-marco_cabecera('Listado de Preguntas');
+pagina_cabecera('Listado de Preguntas');
 
 Faq_ListarPreguntas($_SESSION['user']);
 
-marco_pie('Jonathan Schein', 'jonathanschein@fibertel.com.ar');
+pagina_pie('Jonathan Schein', 'jonathanschein@fibertel.com.ar');
 
 ?>
\ No newline at end of file

diff --git a/src/faq.responderPregunta.php b/src/faq.responderPregunta.php
index aeef924e7f1a1b26b92bb1346b609bc743c2ba07..55962fadb249dd897bf0760663082642a03ff8b8 100644 (file)
--- a/src/faq.responderPregunta.php
+++ b/src/faq.responderPregunta.php
@@ -8,13 +8,13 @@
 //
 // $Id$
 
+require_once 'lib/pagina.php';
 require_once 'lib/Pregunta.php';
 require_once 'lib/faq.functions.php';
-require_once 'lib/pagina.php';
 require_once 'lib/validacion.php';
 require_once 'lib/faq.forms.php';
 
-marco_cabecera('Responder una Pregunta');
+pagina_cabecera('Responder una Pregunta');
 
 // Si me llegó el form completo
 if (isset($_POST['Respuesta']))
@@ -29,6 +29,6 @@ else
                faq_form_responder_pregunta($_POST['PreguntaId']);
 }
 
-marco_pie('Jonathan Schein', 'jonathanschein@fibertel.com.ar');
+pagina_pie('Jonathan Schein', 'jonathanschein@fibertel.com.ar');
 
 ?>
\ No newline at end of file

diff --git a/src/lib/faq.forms.php b/src/lib/faq.forms.php
index 0784dcb22da40c66f2a566c1ba90867d8ca0fc62..c0dd137d23ac34e12820cb2be13b39c8f3e5e13e 100644 (file)
--- a/src/lib/faq.forms.php
+++ b/src/lib/faq.forms.php
@@ -11,6 +11,7 @@
 require_once('Pregunta.php');
 require_once('faq.functions.php');
 
+// Este formulario da de alta una pregunta en el sistema
 function faq_form_ingresar_pregunta()
 {
 ?>
@@ -31,6 +32,8 @@ function faq_form_ingresar_pregunta()
       </form>
 <?php
 }
+
+// Este formulario permite responder una pregunta en el sistema
 function faq_form_responder_pregunta($PreguntaId)
 {
 ?>
@@ -51,6 +54,9 @@ function faq_form_responder_pregunta($PreguntaId)
       </form>
 <?php
 }
+
+// Este formulario muestra un combo box desde el que un administrador puede desactivar preguntas
+// (equivalente a pasarlas a un log historico)
 function faq_form_desactivar_pregunta()
 {
 ?>

diff --git a/src/lib/faq.functions.php b/src/lib/faq.functions.php
index d33b3fe673137db6783b904188318aec9f5deae2..5cf92333bc912474de7e3bae3c68eb4581fb66e1 100644 (file)
--- a/src/lib/faq.functions.php
+++ b/src/lib/faq.functions.php
@@ -11,6 +11,8 @@
 require_once 'Pregunta.php';
 require_once 'Respuesta.php';  
 
+// Levanta las preguntas y respuestas del archivo
+// Devuelve un array con cada una de las preguntas creadas
 function Faq_ObtenerPreguntas()
 {
  // Verifica que el file que guarda las preguntas exista
@@ -102,10 +104,13 @@ function Faq_ObtenerPreguntas()
  return;
 }
 
+// Ingresa una pregunta en el sistema. 
+// Tiene que haber estado validado que el usuario pueda hacerlo
 function Faq_IngresarPregunta($UsrId, $Pregunta)
 {
  $Fecha        = time();
 
+ // Obtiene la lista de preguntas ya existentes
  $Preguntas = Faq_ObtenerPreguntas();
  $CantidadPreguntas = count($Preguntas);
 
@@ -119,7 +124,7 @@ function Faq_IngresarPregunta($UsrId, $Pregunta)
 
  //Formato:
  // PreguntaId | UserId | Fecha | Pregunta
- $Linea = $NuevoId."|".$UsrId."|".$Fecha."|".$Pregunta."\r\n";
+ $Linea = $NuevoId."|".$UsrId."|".$Fecha."|".$Pregunta."\n";
  $fp = fopen("data/preguntas.txt","a");
  if(!$fp) 
  {
@@ -130,13 +135,15 @@ function Faq_IngresarPregunta($UsrId, $Pregunta)
  return "La pregunta fue dada de alta correctamente";
 }
 
+// Ingresa una respuesta para una pregunta seleccionada
+// Tienen que estar chequeadas las precondiciones
 function Faq_IngresarRespuesta($PreguntaId, $UsrId, $Respuesta)
 {
  $Fecha        = time();
 
  //Formato:
  // PreguntaRespondidaId| UserId | Fecha | Respuesta
- $Linea = $PreguntaId."|".$UsrId."|".$Fecha."|".$Respuesta."\r\n";
+ $Linea = $PreguntaId."|".$UsrId."|".$Fecha."|".$Respuesta."\n";
  $fp = fopen("data/respuestas.txt","a");
  if(!$fp) 
  {
@@ -147,9 +154,12 @@ function Faq_IngresarRespuesta($PreguntaId, $UsrId, $Respuesta)
  return "La respuesta fue dada de alta correctamente";
 }
 
+// Lista las preguntas existentes en el sistema
+// Recibe el usuario que esta logueado, para determinar 
+// que opciones de menu le tiene que brindar ( dar de baja pregunta,
+// contestar pregunta, ranquear pregunta ).
 function Faq_ListarPreguntas($Usuario)
 {
- echo $Usuario->getId();
  $Preguntas = Faq_ObtenerPreguntas();
  
  $CantidadPreguntas = count($Preguntas);
@@ -180,11 +190,11 @@ function Faq_ListarPreguntas($Usuario)
  }
  if ( $Usuario->esAdmin() )
  {
-               //echo "<a href='faq.desactivarPregunta.php'>Pasar preguntas a un log historico</a><br/>";
-               include 'faq.desactivarPregunta.php';
+               echo "<a href='faq.desactivarPregunta.php'>Pasar preguntas a un log historico</a><br/>";
  }
 }
 
+// La funcion agrega el id de la pregunta a la lista de preguntas desactivadas
 function Faq_DesactivarPregunta( $PreguntaId )
 {
  $fp = fopen("data/desactivadas.txt","a");
@@ -192,7 +202,7 @@ function Faq_DesactivarPregunta( $PreguntaId )
  {
    return "Error en el archivo";
  }
- fwrite($fp, $PreguntaId."|".time()."\r\n");
+ fwrite($fp, $PreguntaId."|".time()."\n");
  fclose($fp);
  return "La pregunta fue dada de baja, no sera listada mas, pero conservada en un historico";   
 }

diff --git a/src/lib/file.admins.php b/src/lib/file.admins.php
index 74498c96275a3c8de67b0efc4ff122a7d0fa2cf6..9e1ac911ec227996928343a18cacb72a5b8ffbe5 100644 (file)
--- a/src/lib/file.admins.php
+++ b/src/lib/file.admins.php
@@ -15,7 +15,7 @@
  */
 function file_admins_check()
 {
-    return count(@file('data/admins.txt')) > 1;
+               return count(@file('data/admins.txt')) > 1;
 }
 
 /**

diff --git a/src/lib/file.php b/src/lib/file.php
index ab50b24b4160581e1e504262b179d1687471d4e0..2e3b403cbcff2aaab6e9f006397bddf15433a0d2 100644 (file)
--- a/src/lib/file.php
+++ b/src/lib/file.php
@@ -65,7 +65,7 @@ function fgetallcsv($filename, $delim = ',')
  */
 function fsearchcsv($filename, $what, $id = 0, $delim = ',')
 {
-    if (($f = fopen($filename, 'r')) === false) return false;
+               if (($f = fopen($filename, 'r')) === false) return false;
     while (!feof($f))
     {
         $d = fgetcsv($f, 4096);