syntaxe POO (visibilite) et maj indentation

[mw_sourceml] / app / controllers / sources / piste.php

diff --git a/app/controllers/sources/piste.php b/app/controllers/sources/piste.php
index c6a26e4..fa853c3 100644 (file)
--- a/app/controllers/sources/piste.php
+++ b/app/controllers/sources/piste.php
@@ -1,13 +1,13 @@
 <?php
 
-  class mw_sources_piste extends mw_controller
-  {
-    var $content_sources_mod;
+  class mw_sources_piste extends mw_controller{
 
-    function validate(){
+    public $content_sources_mod;
+
+    public function validate(){
       $env = $this->env();
-      if($this->content_sources_mod = $env->get_controller("content/sources"))
-      { $this->content_sources_mod->validation_result = true;
+      if($this->content_sources_mod = $env->get_controller("content/sources")){
+        $this->content_sources_mod->validation_result = true;
         $this->content_sources_mod->validate_status($env);
         $this->content_sources_mod->validate_licences($env);
         return $this->content_sources_mod->validation_result;
@@ -15,42 +15,42 @@
       return "impossible de valider le module";
     }
 
-    function index(){
+    public function index(){
       $env = $this->env();
       $data = $env->data();
-      if(isset($this->content_sources_mod->licences) && $this->content_sources_mod->licences !== false)
-      { $env->set_out("licences", $this->content_sources_mod->licences);
-        if(($groupes = $data->groupes()) !== false)
-        { $env->set_out("groupes", $groupes);
+      if(isset($this->content_sources_mod->licences) && $this->content_sources_mod->licences !== false){
+        $env->set_out("licences", $this->content_sources_mod->licences);
+        if(($groupes = $data->groupes()) !== false){
+          $env->set_out("groupes", $groupes);
           $groupe = null;
           $morceau = null;
-          if(isset($_GET[$env->param("groupe")]) && $_GET[$env->param("groupe")])
-          { if(isset($groupes["list"][$_GET[$env->param("groupe")]]))
-            { $groupe = $groupes["list"][$_GET[$env->param("groupe")]];
+          if(isset($_GET[$env->param("groupe")]) && $_GET[$env->param("groupe")]){
+            if(isset($groupes["list"][$_GET[$env->param("groupe")]])){
+              $groupe = $groupes["list"][$_GET[$env->param("groupe")]];
               $env->set_out("groupe", $groupe);
             }
             else $groupe = false;
           }
-          if($groupe !== false)
-          { if(isset($_GET[$env->param("morceau")]) && $_GET[$env->param("morceau")])
-            { if(($morceau = $data->source($_GET[$env->param("morceau")], true)) !== false)
-              { $env->set_out("morceau", $morceau);
+          if($groupe !== false){
+            if(isset($_GET[$env->param("morceau")]) && $_GET[$env->param("morceau")]){
+              if(($morceau = $data->source($_GET[$env->param("morceau")], true)) !== false){
+                $env->set_out("morceau", $morceau);
 /*
-                if(!isset($groupe))
-                { $groupe = $data->get_admin_groupe($morceau["groupes"]);
+                if(!isset($groupe)){
+                  $groupe = $data->get_admin_groupe($morceau["groupes"]);
                   if(!$groupe) $groupe = false;
                 }
 */
               }
             }
           }
-          if($groupe !== false)
-          { $env->set_out("groupe", $groupe);
-            if($morceau !== false)
-            { $select = array("status" => $this->content_sources_mod->morceau_status_id);
+          if($groupe !== false){
+            $env->set_out("groupe", $groupe);
+            if($morceau !== false){
+              $select = array("status" => $this->content_sources_mod->morceau_status_id);
               if(isset($groupe)) $select["id_groupe"] = $groupe["id"];
-              if(($morceaux = $data->sources($select)) !== false)
-              { $env->set_out("morceaux", $morceaux);
+              if(($morceaux = $data->sources($select)) !== false){
+                $env->set_out("morceaux", $morceaux);
                 $select = array();
                 $select["status"] = $this->content_sources_mod->piste_status_id;
                 $select["order_by"] = "ordre";
@@ -63,8 +63,8 @@
               else $env->erreur("impossible de lire la liste des morceaux");
               $select = array("status" => $this->content_sources_mod->album_status_id);
               if(isset($groupe)) $select["id_groupe"] = $groupe["id"];
-              if(($albums = $data->sources($select)) !== false)
-              { $env->set_out("albums", $albums);
+              if(($albums = $data->sources($select)) !== false){
+                $env->set_out("albums", $albums);
               }
               else $env->erreur("impossible de lire la liste des albums");
             }
@@ -77,29 +77,29 @@
       else $env->erreur("impossible de lire la liste des licences");
     }
 
-    function view(){
+    public function view(){
       $env = $this->env();
       $data = $env->data();
-      if(isset($this->content_sources_mod->licences) && $this->content_sources_mod->licences !== false)
-      { $env->set_out("licences", $this->content_sources_mod->licences);
-        if(isset($_GET[$env->param("piste")]) && $_GET[$env->param("piste")])
-        { if($piste = $data->source($_GET[$env->param("piste")], true))
-          { $env->set_out("piste", $piste);
+      if(isset($this->content_sources_mod->licences) && $this->content_sources_mod->licences !== false){
+        $env->set_out("licences", $this->content_sources_mod->licences);
+        if(isset($_GET[$env->param("piste")]) && $_GET[$env->param("piste")]){
+          if($piste = $data->source($_GET[$env->param("piste")], true)){
+            $env->set_out("piste", $piste);
             if($groupe = $data->get_admin_groupe($piste["groupes"])){
 //              $env->set_out("groupe", $groupe);
               $env->set_out("groupes", $piste["groupes"]);
               $select = array("status" => $this->content_sources_mod->album_status_id);
               $select["id_groupe"] = $groupe["id"];
-              if(($albums = $data->sources($select)) !== false)
-              { $env->set_out("albums", $albums);
-                if(($ariane = $data->source_ariane($piste["id"])) !== false)
-                { $ariane = array_reverse($ariane);
+              if(($albums = $data->sources($select)) !== false){
+                $env->set_out("albums", $albums);
+                if(($ariane = $data->source_ariane($piste["id"])) !== false){
+                  $ariane = array_reverse($ariane);
                   $morceau = null;
                   $album = null;
-                  foreach($ariane as $id_ariane => $source_ariane)
-                  { if(!isset($morceau)) $morceau = $source_ariane;
-                    else
-                    { if(!isset($album)) $album = $source_ariane;
+                  foreach($ariane as $id_ariane => $source_ariane){
+                    if(!isset($morceau)) $morceau = $source_ariane;
+                    else{
+                      if(!isset($album)) $album = $source_ariane;
                     }
                   }
                   if($morceau) $env->set_out("morceau", $morceau);
@@ -119,5 +119,3 @@
     }
 
   }
-
-?>
\ No newline at end of file