Changeset 47701 in spip-zone


Ignore:
Timestamp:
May 12, 2011, 8:34:30 AM (8 years ago)
Author:
cedric@…
Message:

2 bugs sur le formulaire d'edition de document :

  • supporter de multiples occurences du formulaire d'upload (le lien pour switcher local/distant/ftp ne marchait que sur la derniere occurence)
  • is_numeric au lieu de is_int au retour de l'upload d'une vignette
Location:
_core_/plugins/medias/formulaires
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • _core_/plugins/medias/formulaires/illustrer_document.php

    r46791 r47701  
    6262                $files = joindre_trouver_fichier_envoye();
    6363
    64                 $ajoute = action_ajouter_documents_dist($id_vignette,$files,'',0,'vignette');
     64                $ajoute = $ajouter_documents($id_vignette,$files,'',0,'vignette');
    6565
    66 
    67                 if (is_int(reset($ajoute))){
    68                         $id_vignette = reset($ajoute);
     66                if (is_numeric(reset($ajoute))
     67                  AND $id_vignette = reset($ajoute)){
    6968                        include_spip('action/editer_document');
    7069                        document_modifier($id_document,array("id_vignette" => $id_vignette,'mode'=>'document'));
  • _core_/plugins/medias/formulaires/inc-upload_document.html

    r47328 r47701  
    2222                <:medias:bouton_download_local:>
    2323                [(#ENV{mediatheque,''}|oui)
    24                 | <a href='#' onclick="change_methode('mediatheque');return false;"><:medias:bouton_download_par_mediatheque:></a>]
     24                | <a href='#' onclick="change_methode('#GET{domid}','mediatheque');return false;"><:medias:bouton_download_par_mediatheque:></a>]
    2525                [(#ENV{proposer_ftp,''}|oui)
    26                 | <a href='#' onclick="change_methode('ftp');return false;"><:medias:bouton_download_par_ftp:></a>]
    27                 | <a href='#' onclick="change_methode('distant');return false;"><:medias:bouton_download_sur_le_web:></a>
     26                | <a href='#' onclick="change_methode('#GET{domid}','ftp');return false;"><:medias:bouton_download_par_ftp:></a>]
     27                | <a href='#' onclick="change_methode('#GET{domid}','distant');return false;"><:medias:bouton_download_sur_le_web:></a>
    2828        </div>
    2929        <p class='boutons'><input class='submit' type="submit" name="joindre_upload" value="<:medias:bouton_upload:>"/></p>
     
    5151        <div class='sourceup'>
    5252                <:medias:bouton_download_depuis:>
    53                         <a href='#' onclick="change_methode('upload');return false;"><:medias:bouton_download_local:></a>
     53                        <a href='#' onclick="change_methode('#GET{domid}','upload');return false;"><:medias:bouton_download_local:></a>
    5454                | <:medias:bouton_download_par_mediatheque:>
    5555                [(#ENV{proposer_ftp,''}|oui)
    56                 | <a href='#' onclick="change_methode('ftp');return false;"><:medias:bouton_download_par_ftp:></a>]
    57                 | <a href='#' onclick="change_methode('distant');return false;"><:medias:bouton_download_sur_le_web:></a>
     56                | <a href='#' onclick="change_methode('#GET{domid}','ftp');return false;"><:medias:bouton_download_par_ftp:></a>]
     57                | <a href='#' onclick="change_methode('#GET{domid}','distant');return false;"><:medias:bouton_download_sur_le_web:></a>
    5858        </div>
    5959        <p class='boutons'><input class='submit' type="submit" name="joindre_mediatheque" value="<:medias:bouton_attacher_document:>"/></p>
     
    8484        <div class='sourceup'>
    8585                <:medias:bouton_download_depuis:>
    86                 <a href='#' onclick="change_methode('upload');return false;"><:medias:bouton_download_local:></a>
     86                <a href='#' onclick="change_methode('#GET{domid}','upload');return false;"><:medias:bouton_download_local:></a>
    8787                [(#ENV{mediatheque,''}|oui)
    88                 | <a href='#' onclick="change_methode('mediatheque');return false;"><:medias:bouton_download_par_mediatheque:></a>]
     88                | <a href='#' onclick="change_methode('#GET{domid}','mediatheque');return false;"><:medias:bouton_download_par_mediatheque:></a>]
    8989                | <:medias:bouton_download_par_ftp:>
    90                 | <a href='#' onclick="change_methode('distant');return false;"><:medias:bouton_download_sur_le_web:></a>
     90                | <a href='#' onclick="change_methode('#GET{domid}','distant');return false;"><:medias:bouton_download_sur_le_web:></a>
    9191        </div>
    9292        <p class='boutons'>
     
    112112        <div class='sourceup'>
    113113                <:medias:bouton_download_depuis:>
    114                         <a href='#' onclick="change_methode('upload');return false;"><:medias:bouton_download_local:></a>
     114                        <a href='#' onclick="change_methode('#GET{domid}','upload');return false;"><:medias:bouton_download_local:></a>
    115115                [(#ENV{mediatheque,''}|oui)
    116                 | <a href='#' onclick="change_methode('mediatheque');return false;"><:medias:bouton_download_par_mediatheque:></a>]
     116                | <a href='#' onclick="change_methode('#GET{domid}','mediatheque');return false;"><:medias:bouton_download_par_mediatheque:></a>]
    117117                [(#ENV{proposer_ftp,''}|oui)
    118                 | <a href='#' onclick="change_methode('ftp');return false;"><:medias:bouton_download_par_ftp:></a>]
     118                | <a href='#' onclick="change_methode('#GET{domid}','ftp');return false;"><:medias:bouton_download_par_ftp:></a>]
    119119                | <:medias:bouton_download_sur_le_web:>
    120120        </div>
     
    130130<script type='text/javascript'>/*<!#EVAL{chr(91)}CDATA#EVAL{chr(91)}*/
    131131if (window.jQuery){
    132 function change_methode(methode){
    133         var id = "#joindre_"+methode+"#GET{domid}";
     132function change_methode(domid,methode){
     133        var id = "#joindre_"+methode+domid;
    134134        if (jQuery(id).is(':hidden')) {
    135                 jQuery('div.joindre_mode#GET{domid}:visible').slideUp('fast');
     135                jQuery('div.joindre_mode'+domid+':visible').slideUp('fast');
    136136                jQuery(id).slideDown('fast');
    137137        }
    138138        // placer en haut du formulaire les boutons submit par defaut correspondant a la methode active
    139         jQuery("#defaultsubmit#GET{domid}").html('').append(jQuery(id).find('.boutons').eq(-1).find('input').clone(true));
     139        jQuery("#defaultsubmit"+domid).html('').append(jQuery(id).find('.boutons').eq(-1).find('input').clone(true));
    140140}
    141 jQuery(function(){change_methode('#GET{methode}');});
     141jQuery(function(){change_methode('#GET{domid}','#GET{methode}');});
    142142}
    143143/*#EVAL{chr(93)}[(#EVAL{chr(93)})]>*/</script>
Note: See TracChangeset for help on using the changeset viewer.