Ignore:
Timestamp:
May 4, 2010, 7:40:33 AM (11 years ago)
Author:
gilles.vincent@…
Message:

Commentaires phpDoc

Je sors également quelques affectations qui sont noyées dans des conditions PHP : en général on perd en lisibilité avec ce type de syntaxe.

Le seul avantage que j'y vois est quand on a des 'AND' en cascade dans les conditions (mais à ce niveau ça reste de la micro-optimisation, ça ne justifie pas beaucoup de rendre le code moins lisible..)

File:
1 edited

Legend:

Unmodified
Added
Removed

  • _plugins_/ajaxforms/formulaires/editer_logo.php

    r27686 r37969  
    11<?php
    2 
     2/**
     3 * Formulaire #EDITER_LOGO
     4 *
     5 * Ce formulaire ajoute, modifie ou supprime des logos sur les objets de SPIP.
     6 * - En dehors d'une boucle, ce formulaire modifie le logo du site.
     7 * - Dans une boucle, il modifie le logo de la table selectionnee.
     8 * Pensez juste que l'appel de #LOGO_{TYPE} s'appuie sur le nom de la cle primaire et non sur le
     9 * nom de l'objet reel. Par exemple on ecrira #LOGO_GROUPE (et non #LOGO_GROUPEMOTS) pour afficher
     10 * un logo issu du formulaire mis dans une boucle GROUPES_MOTS
     11 * - il est possible de lui passer les parametres objet et id : #FORMULAIRE_EDITER_LOGO{article,1}
     12 * - il est possible de spécifier une url de redirection apres traitement :
     13 * ex. #FORMULAIRE_EDITER_LOGO{article,1,#URL_ARTICLE}
     14 */
    315if (!defined("_ECRIRE_INC_VERSION")) return;
    416
     17/**
     18 * Chargement du formulaire
     19 *
     20 * @param string $objet        Objet SPIP auquel sera lie le document (ex. article)
     21 * @param integer $id_objet    Identifiant de l'objet
     22 * @param string $retour       Url de redirection apres traitement
     23 * @return Array               Variables d'environnement pour le fond
     24 */
    525function formulaires_editer_logo_charger_dist($objet, $id_objet, $retour=''){
    626        $res = array(
    … …  
    2545        $etats = $res['logo_survol'] ? array('on','off') : array('on');
    2646        foreach($etats as $etat) {
    27                 if ($logo = $chercher_logo($id_objet, $_id_objet, $etat)){
     47                $logo = $chercher_logo($id_objet, $_id_objet, $etat);
     48                if ($logo){
    2849                        $res['logo_'.$etat] = $logo[0];
    2950                }
    … …  
    3657}
    3758
     59/**
     60 * Verification avant traitement
     61 *
     62 * On verifie que l'upload s'est bien passe et
     63 * que le document recu est une image (d'apres son extension)
     64 *
     65 * @param string $objet
     66 * @param integer $id_objet
     67 * @param string $retour
     68 * @return Array Tableau des erreurs
     69 */
    3870function formulaires_editer_logo_verifier_dist($objet, $id_objet, $retour=''){
    3971        $erreurs = array();
    4072        // verifier les extensions
    41         foreach(formulaire_editer_logo_get_sources() as $etat=>$file) {
     73        $sources = formulaire_editer_logo_get_sources();
     74        foreach($sources as $etat=>$file) {
    4275                // seulement si une reception correcte a eu lieu
    4376                if ($file AND $file['error'] == 0) {
    … …  
    4982}
    5083
     84/**
     85 * Traitement de l'upload d'un logo
     86 *
     87 * Il est affecte au site si la balise n'est pas dans une boucle,
     88 * sinon a l'objet concerne par la boucle ou indiquee par les parametres d'appel
     89 *
     90 * @param string $objet
     91 * @param integer $id_objet
     92 * @param string $retour
     93 * @return Array
     94 */
    5195function formulaires_editer_logo_traiter_dist($objet, $id_objet, $retour=''){
    5296        $res = array('editable'=>' ');
    … …  
    66110       
    67111        // effectuer la suppression si demandee d'un logo
    68         if (($on = _request('supprimer_logo_on')) OR (_request('supprimer_logo_off'))){
    69                 if ($logo = $chercher_logo($id_objet, $_id_objet, $on ? 'on' : 'off'))
     112        $on = _request('supprimer_logo_on');
     113        if ($on OR _request('supprimer_logo_off')){
     114                $logo = $chercher_logo($id_objet, $_id_objet, $on ? 'on' : 'off');
     115                if ($logo)
    70116                        spip_unlink($logo[0]);
    71117                $res['message_ok'] = _T('ajaxform:confirmer_suppression');
    … …  
    76122                include_spip('action/iconifier');
    77123                $ajouter_image = charger_fonction('spip_image_ajouter','action');
    78                 foreach(formulaire_editer_logo_get_sources() as $etat=>$file) {
    79                         if ($file and $file['error']==0)        {
    80                                 if ($logo = $chercher_logo($id_objet, $_id_objet, $etat))
     124                $sources = formulaire_editer_logo_get_sources();
     125                foreach($sources as $etat=>$file) {
     126                        if ($file and $file['error']==0) {
     127                                $logo = $chercher_logo($id_objet, $_id_objet, $etat);
     128                                if ($logo)
    81129                                        spip_unlink($logo[0]);
    82130                                $ajouter_image($type.$etat.$id_objet," ",$file);
    … …  
    95143
    96144
     145/**
     146 * Extraction des sources des fichiers uploades correspondant aux 2 logos (normal + survol)
     147 * si leur upload s'est bien passé
     148 *
     149 * @return Array
     150 */
    97151function formulaire_editer_logo_get_sources(){
    98152        if (!$_FILES) $_FILES = $GLOBALS['HTTP_POST_FILES'];
Note: See TracChangeset for help on using the changeset viewer.