syntaxe POO (visibilite) et maj indentation

[mw_sourceml] / app / controllers / users / morceaux.php

diff --git a/app/controllers/users/morceaux.php b/app/controllers/users/morceaux.php
index fce394f..9bf2fdf 100644 (file)
--- a/app/controllers/users/morceaux.php
+++ b/app/controllers/users/morceaux.php
@@ -2,13 +2,12 @@
 
   class mw_users_morceaux extends mw_controller{
 
-    var $user;
+    public $user;
+    public $status;
+    public $album_status_id;
+    public $morceau_status_id;
 
-    var $status;
-    var $album_status_id;
-    var $morceau_status_id;
-
-    function validate(){
+    public function validate(){
       $env = $this->env();
       $data = $env->data();
       if(($this->status = $data->source_status()) === false){
@@ -29,7 +28,7 @@
       return true;
     }
 
-    function index(){
+    public function index(){
       $env = $this->env();
       $data = $env->data();
       if(($groupes = $data->groupes($this->user["id"])) === false){
@@ -105,7 +104,7 @@
       $env->set_out("morceaux", $morceaux);
     }
 
-    function add(){
+    public function add(){
       $env = $this->env();
       $data = $env->data();
       $morceau = $data->empty_source();
@@ -155,7 +154,7 @@
             $env,
             $origin_source_xml_url,
             $origin_source_xml_content,
-            true
+            false
           );
         }
       }
@@ -218,6 +217,8 @@
           );
           $source_infos["date_creation"] = $_POST["annee_date_creation"]."-".$_POST["mois_date_creation"]."-".$_POST["jour_date_creation"];
           $source_infos["description"] = $_POST["description"];
+          $morceau["description"] = $source_infos["description"];
+          $morceau["date_creation"] = $source_infos["date_creation"];
           foreach($_POST as $post_key => $post_value){
             if(substr($post_key, 0, 13) == "document_nom_"){
               if($id_document = substr($post_key, 13)){
@@ -241,7 +242,7 @@
                 $morceau["groupes"],
                 $morceau["titre"],
                 $this->morceau_status_id,
-                $morceau["licence"]["id"],
+                isset($morceau["licence"]["id"]) ? $morceau["licence"]["id"] : null,
                 $morceau["documents"],
                 $morceau["reference"],
                 $morceau["derivations"],
@@ -259,7 +260,7 @@
             }
           }
           $env->redirect(
-            $env->url("users/morceaux"),
+            $env->url("users/morceaux/edit", array("id" => $id_source)),
             "le morceau a été ajouté"
           );
         }
@@ -273,7 +274,7 @@
       $env->set_out("album", $album);
     }
 
-    function edit(){
+    public function edit(){
       $env = $this->env();
       $data = $env->data();
       if(($groupes = $data->groupes($this->user["id"])) === false){
@@ -322,12 +323,19 @@
         $env->erreur("Impossible de lire la liste des compositions");
         return;
       }
-      if($compositions) foreach($compositions[$morceau["id"]] as $id_album) { $morceau["album"] = $id_album; break; }
+      if($compositions) foreach($compositions[$morceau["id"]] as $id_album){
+        $morceau["album"] = $id_album;
+        break;
+      }
       if($_POST){
         $source_infos = array(
           "date_inscription" => $_POST["date_inscription"],
           "ordre" => isset($morceau["ordre"]) ? $morceau["ordre"] : 0
         );
+        $permissions = $morceau["permissions"];
+        $image = isset($morceau["image"]) ? $morceau["image"] : null;
+        $image_uri = isset($morceau["image_uri"]) ? $morceau["image_uri"] : null;
+        $image_uri = $morceau["image_uri"];
         $morceau = $data->empty_source(
           array(
             "id" => $morceau["id"],
@@ -335,6 +343,10 @@
             "image" => isset($morceau["image"]) ? $morceau["image"] : ""
           )
         );
+        $morceau["permissions"] = $permissions;
+        $morceau["date_creation"] = $_POST["annee_date_creation"]."-".$_POST["mois_date_creation"]."-".$_POST["jour_date_creation"];
+        $morceau["image"] = $image;
+        $morceau["image_uri"] = $image_uri;
         $users_sources_mod = $env->get_controller("users/sources");
         if($_POST["album"]){
           if($album = $data->source($_POST["album"], true)){
@@ -380,6 +392,7 @@
             ),
             isset($_POST["use_edit_reference_content"]) ? true : false
           );
+          $morceau["licence"]["id"] = 0;
         }
         else{
           if(!($morceau["titre"] = trim($_POST["titre"]))){
@@ -390,6 +403,7 @@
           );
           $source_infos["date_creation"] = $_POST["annee_date_creation"]."-".$_POST["mois_date_creation"]."-".$_POST["jour_date_creation"];
           $source_infos["description"] = $_POST["description"];
+          $morceau["description"] = $source_infos["description"];
           foreach($_POST as $post_key => $post_value){
             if(substr($post_key, 0, 13) == "document_nom_"){
               if($id_document = substr($post_key, 13)){
@@ -460,7 +474,8 @@
       $env->set_out("morceau", $morceau);
     }
 
-    function maj_xml(){
+    public function maj_xml(){
+      $env = $this->env();
       $data = $env->data();
       if(!($morceau = $data->source($_GET[$env->param("id")], true))){
         $env->erreur("Impossible de lire les informations du morceau");
@@ -472,9 +487,9 @@
         return;
       }
       if
-      (    isset($_GET[$env->param("id")])
-        && isset($_GET[$env->param("xml")])
-        && ($_GET[$env->param("xml")] == "derviation" ? isset($_GET[$env->param("derivation")]) : true)
+      (    !isset($_GET[$env->param("id")])
+        || !isset($_GET[$env->param("xml")])
+        || ($_GET[$env->param("xml")] == "derviation" && !isset($_GET[$env->param("derivation")]))
       ){
         $env->erreur("parametre de fichier xml manquant");
         return;
@@ -522,7 +537,7 @@
       $env->run("users/morceaux/edit", array("get" => array("id" => $morceau["id"])));
     }
 
-    function del(){
+    public function del(){
       $env = $this->env();
       $data = $env->data();
       if(!($morceau = $data->source($_GET[$env->param("id")], true))){
@@ -544,7 +559,7 @@
       );
     }
 
-    function groupes(){
+    public function groupes(){
       $env = $this->env();
       if(
             !($sources_invitations_controller = $env->get_controller("users/sources_invitations"))
@@ -557,5 +572,3 @@
     }
 
   }
-
-?>
\ No newline at end of file