syntaxe POO (visibilite) et maj indentation

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

diff --git a/app/controllers/sources/morceau.php b/app/controllers/sources/morceau.php
index 3d68bed..ffd1018 100644 (file)
--- a/app/controllers/sources/morceau.php
+++ b/app/controllers/sources/morceau.php
@@ -1,13 +1,13 @@
 <?php
 
-  class mw_sources_morceau extends mw_controller
-  {
-    var $content_sources_mod;
+  class mw_sources_morceau 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,26 +15,26 @@
       return "impossible de valider le module";
     }
 
-    function index(){
+    public function index(){
       $env = $this->env();
       $data = $env->data();
-      if($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($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;
           $album = 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("album")]) && $_GET[$env->param("album")])
-            { if(($album = $data->source($_GET[$env->param("album")], true)) !== false)
-              { $env->set_out("album", $album);
+          if($groupe !== false){
+            if(isset($_GET[$env->param("album")]) && $_GET[$env->param("album")]){
+              if(($album = $data->source($_GET[$env->param("album")], true)) !== false){
+                $env->set_out("album", $album);
 /*
                 if(!isset($groupe)){
                   $groupe = $data->get_admin_groupe($album["groupes"]);
@@ -44,13 +44,13 @@
               }
             }
           }
-          if($groupe !== false)
-          { $env->set_out("groupe", $groupe);
-            if($album !== false)
-            { $select = array("status" => $this->content_sources_mod->album_status_id);
+          if($groupe !== false){
+            $env->set_out("groupe", $groupe);
+            if($album !== false){
+              $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);
                 $select = array();
                 $select["status"] = $this->content_sources_mod->morceau_status_id;
                 $select["order_by"] = "ordre";
@@ -71,26 +71,26 @@
       else $env->erreur("Impossible de lire la liste des licences");
     }
 
-    function view(){
+    public function view(){
       $env = $this->env();
       $data = $env->data();
-      if($this->content_sources_mod->licences !== false)
-      { $env->set_out("licences", $this->content_sources_mod->licences);
-        if(isset($_GET[$env->param("morceau")]) && $_GET[$env->param("morceau")])
-        { if($morceau = $data->source($_GET[$env->param("morceau")], true))
-          { $env->set_out("morceau", $morceau);
+      if($this->content_sources_mod->licences !== false){
+        $env->set_out("licences", $this->content_sources_mod->licences);
+        if(isset($_GET[$env->param("morceau")]) && $_GET[$env->param("morceau")]){
+          if($morceau = $data->source($_GET[$env->param("morceau")], true)){
+            $env->set_out("morceau", $morceau);
             if($groupe = $data->get_admin_groupe($morceau["groupes"])){
 //              $env->set_out("groupe", $groupe);
               $env->set_out("groupes", $morceau["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($morceau["id"])) !== false)
-                { $ariane = array_reverse($ariane);
+              if(($albums = $data->sources($select)) !== false){
+                $env->set_out("albums", $albums);
+                if(($ariane = $data->source_ariane($morceau["id"])) !== false){
+                  $ariane = array_reverse($ariane);
                   $album = null;
-                  foreach($ariane as $id_ariane => $source_ariane)
-                  { if(!isset($album)) $album = $source_ariane;
+                  foreach($ariane as $id_ariane => $source_ariane){
+                    if(!isset($album)) $album = $source_ariane;
                   }
                   if($album) $env->set_out("album", $album);
                 }
@@ -108,5 +108,3 @@
     }
 
   }
-
-?>
\ No newline at end of file