Changeset 37969 in spip-zone


Ignore:
Timestamp:
May 4, 2010, 7:40:33 AM (10 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..)

Location:
_plugins_/ajaxforms
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • _plugins_/ajaxforms/ajaxform_fonctions.php

    r23757 r37969  
    11<?php
    22
    3 // copie joyeusement sur le plugin etiquette (balise_TYPE_BOUCLE)
    4 // bete de copier plusieurs fois cette fonction du coup.
    5 // a mettre dans le core ?
     3/**
     4 * Affiche la boucle éventuelle dans laquelle est placée la balise et si
     5 * c'est le cas, le type de boucle
     6 *
     7 * copie de balise_TYPE_BOUCLE du plugin etiquette
     8 *
     9 * @param Object $p
     10 * @return Object
     11 */
    612function balise_AJAXFORM_TYPE_BOUCLE_dist($p) {
    713        $type = $p->boucles[$p->id_boucle]->id_table;
    814        $p->code = $type ? $type : "balise_hors_boucle";
    9         return $p;   
     15        return $p;
    1016}
    1117
  • _plugins_/ajaxforms/formulaires/ajouter_un_document.php

    r24826 r37969  
    11<?php
    22
     3/**
     4 * Formulaire #AJOUTER_UN_DOCUMENT
     5 *
     6 * Ce formulaire permet de telecharger UN document (un zip ne sera pas decompresse).
     7 * - En dehors d'une boucle, le document est simplement ajoute a la table spip_document
     8 * - Dans une boucle, il sera ajoute et lie a l'objet de la boucle.
     9 * - il est possible de passer les parametres objet et id : #FORMULAIRE_AJOUTER_UN_DOCUMENT{article,1}
     10 *
     11 * Note 1) document ou image ?
     12 *      Les documents sont ajoutes en mode 'choix', c'est a dire que c'est SPIP qui
     13 *      place le document en type 'image' ou 'document' selon le profil du fichier envoye.
     14 *      Par defaut, une image de plus de 400px sera consideree comme un document.
     15 *
     16 * Note 2) afficher les documents
     17 *      Pour afficher une liste de documents qui s'actualise quand on ajoute un element,
     18 *      il faut soit ne pas utiliser l'ajax, soit inserer la liste des documents
     19 *      a la suite du formulaire en utilisant le pipeline charger des
     20 *      formulaires CVT pour inserer ce contenu.
     21 *
     22 */
    323if (!defined("_ECRIRE_INC_VERSION")) return;
    424
     25/**
     26 * Chargement du formulaire
     27 *
     28 *
     29 * @param string $objet        Objet SPIP auquel sera lie le document (ex. article)
     30 * @param integer $id_objet    Identifiant de l'objet
     31 * @param string $mode         Voir la note 1) ci-dessus
     32 * @return Array
     33 */
    534function formulaires_ajouter_un_document_charger_dist($objet, $id_objet, $mode='choix'){
    635        $res = array(
     
    1443        );
    1544
    16         // si l'on vien juste de poster le formlaire et qu'il a ete valide
     45        // si l'on vient juste de poster le formulaire et qu'il a ete valide
    1746        // on veut pouvoir recommencer a poster
    1847        // on ne prend du coup pas les anciennes valeurs dans l'environnement
     
    2453}
    2554
     55/**
     56 * Verification de l'upload
     57 *
     58 * Il s'agit uniquement de signaler si une erreur d'upload a eu lieu
     59 * Pas de controle sur le type et les autorisations qui sont geres par ajouter_document
     60 *
     61 * @param string $objet
     62 * @param integer $id_objet
     63 * @param string $mode
     64 * @return Array
     65 */
    2666function formulaires_ajouter_un_document_verifier_dist($objet, $id_objet, $mode='choix'){
    2767        $erreurs = array();     
     
    3979}
    4080
    41 /*
     81/**
     82 * Traitement de l'upload
     83 *
    4284 * La fonction de traitement pour l'instant ne prend pas en compte :
    4385 * - les documents distants
    4486 * - les autres modes de documents ('choix' par defaut)
     87 *
     88 * @param string $objet
     89 * @param integer $id_objet
     90 * @param string $mode
     91 * @return Array
    4592 */
    4693function formulaires_ajouter_un_document_traiter_dist($objet, $id_objet, $mode='choix'){
  • _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.