From 1c15c381a75a9eec18d22bab5c80feacb47b40ad Mon Sep 17 00:00:00 2001 From: Fil <fil@rezo.net> Date: Sat, 1 Sep 2007 01:25:08 +0000 Subject: [PATCH] desole, en [10191] je n voulais pas envoyer tout ca d'un coup, de maniere a pouvoir segmenter les choses... ce commit revient donc partiellement sur [10039] qui avait introduit des onglets fort peu consensuels (?) mais aussi des concepts interessants comme le bloc infos en squelette, une ecriture (un peu) plus rationnelle du code de l'espace prive, une reflexion sur le role de divers boutons etc. J'ai ici retabli tout ce qui avait diverge de maniere un peu trop violente pour moi, et corrige pas mal de petits bugs (les lang_select dans les squelettes, qui faisaient que le bloc statut d'un article s'affichait dans la langue de l'article, par exemple)... je laisse les onglets dans le code, avec uneconfiguration qu'il faut aller chercher soi-meme ; ca reste un peu trop couteux a mon gout en termes de complexite (double css, doubles definitions a certains endroits dans le code) -- a voir comment on se sort de tout ca. --- ecrire/configuration/interfaceur.php | 4 +-- ecrire/exec/articles.php | 28 +++------------ ecrire/exec/auteur_infos.php | 3 +- ecrire/exec/breves_voir.php | 50 +++++++++++++------------- ecrire/exec/mots_edit.php | 2 +- ecrire/exec/naviguer.php | 52 +++++++++++----------------- ecrire/exec/sites.php | 14 ++++---- ecrire/inc/config.php | 2 +- ecrire/inc/dater.php | 12 ++++--- ecrire/inc/presentation.php | 27 ++++++++------- 10 files changed, 88 insertions(+), 106 deletions(-) diff --git a/ecrire/configuration/interfaceur.php b/ecrire/configuration/interfaceur.php index 5ecb69b6ed..d9a296b8ac 100644 --- a/ecrire/configuration/interfaceur.php +++ b/ecrire/configuration/interfaceur.php @@ -28,8 +28,8 @@ function configuration_interfaceur_dist() . "</div>" . "<div class='verdana2'>" . afficher_choix('interface_mode',$GLOBALS['meta']["interface_mode"] , - array('192' => _L('Utiliser la présentation traditionelle de SPIP'), - '193' => _L('Utiliser les onglets pour présenter les articles') + array('' => _L('Utiliser la présentation traditionnelle de SPIP'), + 'onglets' => _L('Utiliser les onglets pour présenter les articles') )) . "</div>"; diff --git a/ecrire/exec/articles.php b/ecrire/exec/articles.php index b8cc2eef7c..b4dcf59013 100644 --- a/ecrire/exec/articles.php +++ b/ecrire/exec/articles.php @@ -102,8 +102,6 @@ function articles_affiche($id_article, $row, $cherche_auteur, $ids, $cherche_mot $icone = $iconifier('id_article', $id_article,'articles', false); } else $icone = ''; - $instituer_article = charger_fonction('instituer_article', 'inc'); - $boite = pipeline ('boite_infos', array('data' => '', 'args' => array( 'type'=>'article', @@ -128,22 +126,13 @@ function articles_affiche($id_article, $row, $cherche_auteur, $ids, $cherche_mot changer_typo($row['lang']); $actions = - voir_en_ligne('article', $id_article, $statut_article, 'racine-24.gif', false) - . ($flag_editable ? bouton_modifier_articles($id_article, $id_rubrique, $modif, _T('avis_article_modifie', $modif), "article-24.gif", "edit.gif",$spip_lang_right) : ""); - - // revisions d'articles - if (($GLOBALS['meta']["articles_versions"]=='oui') - AND $row['id_version']>1 - AND autoriser('voirrevisions', 'article', $id_article)) - $actions .= icone_inline(_T('info_historique_lien'), generer_url_ecrire("articles_versions","id_article=$id_article"), "historique-24.gif", "rien.gif", $spip_lang_left); - - $actions .= "<div class='nettoyeur'></div>"; - + ($flag_editable ? bouton_modifier_articles($id_article, $id_rubrique, $modif, _T('avis_article_modifie', $modif), "article-24.gif", "edit.gif",$spip_lang_right) : ""); + $haut = + "<div class='bandeau_actions'>$actions</div>". (_INTERFACE_ONGLETS?"":"<span $dir_lang class='arial1 spip_medium'><b>" . typo($surtitre) . "</b></span>\n") . gros_titre($titre, '' , false) - . (_INTERFACE_ONGLETS?"":"<span $dir_lang class='arial1 spip_medium'><b>" . typo($soustitre) . "</b></span>\n") - . "<div class='bandeau_actions'>$actions</div>"; + . (_INTERFACE_ONGLETS?"":"<span $dir_lang class='arial1 spip_medium'><b>" . typo($soustitre) . "</b></span>\n"); $onglet_contenu = afficher_corps_articles($id_article,$virtuel,$row); @@ -162,14 +151,7 @@ function articles_affiche($id_article, $row, $cherche_auteur, $ids, $cherche_mot ; $onglet_interactivite = - // statistiques - (($row['statut'] == 'publie' - AND $row['visites'] > 0 - AND $GLOBALS['meta']["activer_statistiques"] != "non" - AND autoriser('voirstats', $type, $id)) ? - icone_horizontale(_T('icone_evolution_visites', array('visites' => $row['visites'])), generer_url_ecrire("statistiques_visites","id_article=$id"), "statistiques-24.gif","rien.gif", false) - : "") - . (_INTERFACE_ONGLETS?boites_de_config_articles($id_article):"") + (_INTERFACE_ONGLETS?boites_de_config_articles($id_article):"") ; $onglet_discuter = diff --git a/ecrire/exec/auteur_infos.php b/ecrire/exec/auteur_infos.php index c9f7e6d8c8..47867cb4bc 100644 --- a/ecrire/exec/auteur_infos.php +++ b/ecrire/exec/auteur_infos.php @@ -132,7 +132,8 @@ function cadre_auteur_infos($id_auteur, $auteur) ) )); - return debut_boite_info(true) . $boite . fin_boite_info(true); + if ($boite) + return debut_boite_info(true) . $boite . fin_boite_info(true); } diff --git a/ecrire/exec/breves_voir.php b/ecrire/exec/breves_voir.php index 47f5d299b1..fe44d02ef2 100644 --- a/ecrire/exec/breves_voir.php +++ b/ecrire/exec/breves_voir.php @@ -68,8 +68,9 @@ function afficher_breves_voir($id_breve, $cherche_mot, $select_groupe) ); if (($spip_display != 4) AND $id_breve>0 AND autoriser('publierdans','rubrique',$id_rubrique)) $iconifier = charger_fonction('iconifier', 'inc'); - if ($flag_editable AND ($statut == 'publie')) - $dater = charger_fonction('dater', 'inc'); + + $dater = charger_fonction('dater', 'inc'); + $editer_mot = charger_fonction('editer_mot', 'inc'); if ($champs_extra AND $extra) include_spip('inc/extra'); @@ -98,6 +99,7 @@ function afficher_breves_voir($id_breve, $cherche_mot, $select_groupe) ) ); echo ($iconifier ? $iconifier('id_breve', $id_breve, 'breves_voir', false) : ""); + echo creer_colonne_droite('', true); echo pipeline('affiche_droite', array( @@ -107,11 +109,17 @@ function afficher_breves_voir($id_breve, $cherche_mot, $select_groupe) ); echo meme_rubrique($id_rubrique, $id_breve, 'breve', 'date_heure'); + /* raccourcis ont disparu */ + echo bloc_des_raccourcis(icone_horizontale(_T('icone_nouvelle_breve'), generer_url_ecrire("breves_edit","new=oui&id_rubrique=$id_rubrique"), "breve-24.gif","creer.gif", 0)); + + + + + $afficher_contenu_objet = charger_fonction('afficher_contenu_objet', 'inc'); - $actions = - voir_en_ligne('breve', $id_breve, $statut, 'racine-24.gif', false) - . ($flag_editable ? icone_inline( + $actions = $flag_editable + ? icone_inline( // TODO -- _L("Fil a travaille sur cette breve il y a x minutes") !$modif ? _T('icone_modifier_breve') : _T('texte_travail_article', $modif), @@ -119,34 +127,26 @@ function afficher_breves_voir($id_breve, $cherche_mot, $select_groupe) !$modif ? "breve-24.gif" : "warning-24.gif", !$modif ? "edit.gif" : '', $GLOBALS['spip_lang_right'] - ) : "") - . icone_inline(_T('icone_nouvelle_breve'), generer_url_ecrire("breves_edit","new=oui&id_rubrique=$id_rubrique"), "breve-24.gif","creer.gif", $spip_lang_left) - ; - /* - if (autoriser('publierdans','rubrique',$id_rubrique) AND ($statut=="prop" OR $statut=="prepa")){ - $actions .= icone_inline(_T('icone_refuser_breve'), - redirige_action_auteur('editer_breve', "$id_breve-statut-refuse", "breves_voir","id_breve=$id_breve"), "breve-24.gif", "supprimer.gif", $spip_lang_right); - $actions .= icone_inline(_T('icone_publier_breve'), - redirige_action_auteur('editer_breve',"$id_breve-statut-publie","breves_voir","id_breve=$id_breve"), "breve-24.gif", "racine-24.gif", $spip_lang_right); - echo "</div>"; - } */ - - $actions .= "<div class='nettoyeur'></div>"; + ) + : ""; + $haut = - gros_titre($titre,'', false) - . "<div class='bandeau_actions'>$actions</div>"; + "<div class='bandeau_actions'>$actions</div>" + . gros_titre($titre,'', false); - $onglet_contenu = - (($flag_editable AND ($statut !== 'publie')) ? "<p class='breve_prop'>".affdate($date_heure)."</p>" : "") - . $afficher_contenu_objet('breve', $id_breve,$row) - ; + + $onglet_contenu = $afficher_contenu_objet('breve', $id_breve,$row); $onglet_proprietes = afficher_breve_rubrique($id_breve, $id_rubrique, $statut) - . ($dater ? $dater($id_breve, $flag_editable, $statut, 'breve', 'breves_voir', $date_heure) : "") + . ($dater + ? $dater($id_breve, $flag_editable, $statut, 'breve', 'breves_voir', $date_heure) + : '' + ) + . $editer_mot('breve', $id_breve, $cherche_mot, $select_groupe, $flag_editable, true) . ((($GLOBALS['meta']['multi_articles'] == 'oui') AND ($flag_editable)) ? langue_breve($id_breve,$row):"") . pipeline('affiche_milieu',array( diff --git a/ecrire/exec/mots_edit.php b/ecrire/exec/mots_edit.php index 81e0b68c09..44a21237ea 100644 --- a/ecrire/exec/mots_edit.php +++ b/ecrire/exec/mots_edit.php @@ -90,7 +90,7 @@ global $champs_extra, $connect_statut, $spip_display, $les_notes; . "<br /><span class='spip_xx-large'>" . $id_mot . '</span></div>'; - $out .= voir_en_ligne ('mot', $id_mot, false, 'racine-24.gif', false); + $out .= voir_en_ligne ('mot', $id_mot, false, 'racine-24.gif', false, false); $out .= fin_boite_info(true); // Logos du mot-clef diff --git a/ecrire/exec/naviguer.php b/ecrire/exec/naviguer.php index ada30feb45..62e405539b 100644 --- a/ecrire/exec/naviguer.php +++ b/ecrire/exec/naviguer.php @@ -85,6 +85,11 @@ function exec_naviguer_dist() echo infos_naviguer($id_rubrique, $statut, $ze_logo, $n_forums); echo ($iconifier('id_rubrique', $id_rubrique, 'naviguer', false)); + + + echo bloc_des_raccourcis(icone_horizontale(_T('icone_tous_articles'), generer_url_ecrire("articles_page"), "article-24.gif", '', false, false)); + + echo pipeline('affiche_gauche',array('args'=>array('exec'=>'naviguer','id_rubrique'=>$id_rubrique),'data'=>'')); // @@ -93,48 +98,33 @@ function exec_naviguer_dist() /*if ($spip_display != 4) { raccourcis_naviguer($id_rubrique, $id_parent); }*/ - echo creer_colonne_droite('', true); echo pipeline('affiche_droite',array('args'=>array('exec'=>'naviguer','id_rubrique'=>$id_rubrique),'data'=>'')); echo debut_droite('', true); // echo debut_cadre_relief($ze_logo, true); - $actions = - voir_en_ligne ('rubrique', $id_rubrique, $statut, 'racine-24.gif', false) - . icone_inline(_T('icone_tous_articles'), generer_url_ecrire("articles_page"), "article-24.gif", '', $spip_lang_left) - . (($id_rubrique > 0 AND $flag_editable)?icone_inline(_T('icone_modifier_rubrique'), generer_url_ecrire("rubriques_edit","id_rubrique=$id_rubrique&retour=nav"), $ze_logo, "edit.gif", $spip_lang_right):"") - ////// Supprimer cette rubrique (si vide) - . ((($id_rubrique>0) AND tester_rubrique_vide($id_rubrique) AND $flag_editable)? - icone_inline(_T('icone_supprimer_rubrique'), redirige_action_auteur('supprimer', "rubrique-$id_rubrique", "naviguer","id_rubrique=$id_parent"), $ze_logo, "supprimer.gif", $spip_lang_right) - :"") - . (autoriser('creerrubriquedans','rubrique',$id_rubrique)? - (!$id_rubrique - ? icone_inline(_T('icone_creer_rubrique'), generer_url_ecrire("rubriques_edit","new=oui&retour=nav"), "secteur-24.gif", "creer.gif",$spip_lang_left) - : icone_inline(_T('icone_creer_sous_rubrique'), generer_url_ecrire("rubriques_edit","new=oui&retour=nav&id_parent=$id_rubrique"), "rubrique-24.gif", "creer.gif",$spip_lang_left)) - :""); - - $n = sql_countsel('spip_rubriques'); - if ($n) { - if (autoriser('creerarticledans','rubrique',$id_rubrique)) - $actions .= icone_inline(_T('icone_ecrire_article'), generer_url_ecrire("articles_edit","id_rubrique=$id_rubrique&new=oui"), "article-24.gif","creer.gif", $spip_lang_left); - - $activer_breves = $GLOBALS['meta']["activer_breves"]; - if (autoriser('creerbrevedans','rubrique',$id_rubrique,NULL,array('id_parent'=>$id_parent))) - $actions .= icone_inline(_T('icone_nouvelle_breve'), generer_url_ecrire("breves_edit","id_rubrique=$id_rubrique&new=oui"), "breve-24.gif","creer.gif", $spip_lang_left); - - if (autoriser('creersitedans','rubrique',$id_rubrique)) - $actions .= icone_inline(_T('info_sites_referencer'), generer_url_ecrire('sites_edit', "id_rubrique=$id_rubrique"), "site-24.gif", "creer.gif", $spip_lang_left); + if ($flag_editable + AND $id_rubrique > 0) { + $actions = icone_inline(_T('icone_modifier_rubrique'), + generer_url_ecrire("rubriques_edit", + "id_rubrique=$id_rubrique&retour=nav"), $ze_logo, "edit.gif", $spip_lang_right); + + // Supprimer cette rubrique (si vide) + if (tester_rubrique_vide($id_rubrique)) + $actions .= icone_inline(_T('icone_supprimer_rubrique'), + redirige_action_auteur('supprimer', "rubrique-$id_rubrique", "naviguer","id_rubrique=$id_parent"), $ze_logo, "supprimer.gif", $spip_lang_right); } - - $actions .= "<div class='nettoyeur'></div>"; - - $haut = + else + $actions = ''; // rubrique non editable + + $haut = "<div class='bandeau_actions'>$actions</div>" + . gros_titre((!acces_restreint_rubrique($id_rubrique) ? '' : http_img_pack("admin-12.gif",'', "width='12' height='12'", _T('info_administrer_rubrique'))) . $titre,'', false) - . "<div class='bandeau_actions'>$actions</div>"; + . "<div class='nettoyeur'></div>\n"; if ($extra) include_spip('inc/extra'); diff --git a/ecrire/exec/sites.php b/ecrire/exec/sites.php index 67859d9135..13a7d96a26 100644 --- a/ecrire/exec/sites.php +++ b/ecrire/exec/sites.php @@ -103,6 +103,11 @@ function afficher_site($id_syndic, $id_rubrique, $nom_site, $row){ echo meme_rubrique($id_rubrique, $id_syndic, 'syndic'); echo pipeline('affiche_droite',array('args'=>array('exec'=>'sites','id_syndic'=>$id_syndic),'data'=>'')); + echo bloc_des_raccourcis( + icone_horizontale(_T('icone_voir_sites_references'), generer_url_ecrire("sites_tous",""), "site-24.gif","rien.gif", false) + ); + + echo debut_droite('', true); if ($syndication == 'off' OR $syndication == 'sus') @@ -113,15 +118,12 @@ function afficher_site($id_syndic, $id_rubrique, $nom_site, $row){ if (strlen($url_affichee) > 40) $url_affichee = substr($url_affichee, 0, 30)."..."; $actions = - voir_en_ligne('site', $id_syndic, $statut, 'racine-24.gif', false) - . ($flag_editable ? icone_inline(_T('icone_modifier_site'), generer_url_ecrire('sites_edit',"id_syndic=$id_syndic"), "site-24.gif", "edit.gif",$spip_lang_right) : "") - . icone_inline(_T('icone_voir_sites_references'), generer_url_ecrire("sites_tous",""), "site-24.gif","rien.gif", $spip_lang_left) - . "<div class='nettoyeur'></div>"; + ($flag_editable ? icone_inline(_T('icone_modifier_site'), generer_url_ecrire('sites_edit',"id_syndic=$id_syndic"), "site-24.gif", "edit.gif",$spip_lang_right) : ""); $haut = + "<div class='bandeau_actions'>$actions</div>". gros_titre($nom_site, '' , false) - . "<a href='$url_site' class='url_site'>$url_affichee</a>" - . "<div class='bandeau_actions'>$actions</div>"; + . "<a href='$url_site' class='url_site'>$url_affichee</a>"; $onglet_contenu = (_INTERFACE_ONGLETS? diff --git a/ecrire/inc/config.php b/ecrire/inc/config.php index 221d9236d0..78117d0838 100644 --- a/ecrire/inc/config.php +++ b/ecrire/inc/config.php @@ -89,7 +89,7 @@ function liste_metas() 'multi_secteurs' => 'non', 'gerer_trad' => 'non', 'langues_multilingue' => '', - 'interface_mode' => '193' + 'interface_mode' => '' ); } diff --git a/ecrire/inc/dater.php b/ecrire/inc/dater.php index 150a58a051..e1eda1bf73 100644 --- a/ecrire/inc/dater.php +++ b/ecrire/inc/dater.php @@ -80,10 +80,14 @@ onchange=\"findObj_forcer('valider_date').style.visibility='visible';\""; $res = block_parfois_visible('datepub', $invite, $res, 'text-align: left'); } else { - $res = "\n<div style='padding-$spip_lang_left:7px;'><b> <span class='verdana1'>" - . _T('texte_date_creation_article') - . "</span>\n" - . majuscules(affdate($date))."</b>".aide('artdate')."</div>"; + if ($type == 'article') + $res = "\n<div style='padding-$spip_lang_left:7px;'><b> <span class='verdana1'>" + . _T('texte_date_creation_article') + . "</span>\n" + . majuscules(affdate($date))."</b>".aide('artdate')."</div>"; + else + $res = "\n<div style='text-align:center;'><b>" + . majuscules(affdate($date))."</b></div>"; } if (($type == 'article') diff --git a/ecrire/inc/presentation.php b/ecrire/inc/presentation.php index 29b2caa8fb..d2e4088202 100644 --- a/ecrire/inc/presentation.php +++ b/ecrire/inc/presentation.php @@ -19,7 +19,7 @@ include_spip('inc/puce_statut'); define('_ACTIVER_PUCE_RAPIDE', true); define('_SIGNALER_ECHOS', true); -define('_INTERFACE_ONGLETS',$GLOBALS['meta']['interface_mode']=='192'?false:true); +define('_INTERFACE_ONGLETS', $GLOBALS['meta']['interface_mode']==='onglets'); // http://doc.spip.org/@echo_log function echo_log($f, $ret) { @@ -1064,10 +1064,6 @@ function f_boite_infos($flux) { include_spip('public/assembler'); $boite .= recuperer_fond("prive/{$type}_infos",$args); - if (!in_array($type,array('article','rubrique','site','breve')) - && autoriser('previsualiser', $type, $id)) - $boite .= voir_en_ligne($type, $id, $row['statut'], 'racine-24.gif', false, false); - $flux['data'] = $boite; return $flux; } @@ -1329,6 +1325,7 @@ function debloquer_article($arg, $texte) { // http://doc.spip.org/@meme_rubrique function meme_rubrique($id_rubrique, $id, $type, $order='date', $limit=NULL, $ajax=false) { + global $spip_lang_right, $spip_lang_left; include_spip('inc/afficher_objets'); if (!$limit) $limit = 10; @@ -1365,23 +1362,29 @@ function meme_rubrique($id_rubrique, $id, $type, $order='date', $limit=NULL, $aj . "'>" . sinon(typo($row['titre']), _T('info_sans_titre')) . "</a>"; - $retour .= "<tr class='tr_liste' style='background-color: #e0e0e0;'><td>$statut</td><td>$href</td><td style='width: 25%;'>$num</td></tr>"; + + // Todo: refaire en css plus sains + $retour .= "\n<div>" + . "\n<div style='float:$spip_lang_right;width: 32%'>" + . $num . "</div>" + . "<div style='float:$spip_lang_left; padding-top:1px; width:18px;'>".$statut ."</div>" + . "<div style='padding-$spip_lang_left:18px;'>".$href."</div>" + . "<div style='clear:both; height: 3px;'></div>" + . "</div>"; } $icone = '<b>' . _T('info_meme_rubrique') . '</b>'; - $bouton = bouton_block_depliable(_T('info_meme_rubrique'),true,'memerub'); + $bouton = bouton_block_depliable(_T('info_meme_rubrique'),true,'memerub'); $retour = debut_cadre('meme-rubriques',"article-24.gif",'',$bouton) . debut_block_depliable(true,'memerub') - . "\n<table style='background-color: #e0e0e0;border: 0px; padding-left:4px; width: 100%;'>" . $retour; // $retour .= (($limit <= 0) ? '' : "<tr><td colspan='3' style='text-align: center'>+ $limit</td></tr>"); - $retour .= "</table>" - . fin_block() + $retour .= fin_block() . fin_cadre('meme-rubriques'); if ($ajax) return $retour; @@ -1434,10 +1437,10 @@ function afficher_hierarchie($id_rubrique) { //. http_style_background("racine-site-12.gif", $style1) . "><li><span><$tag class='racine$on'" . ($tag=='a'?" href='". generer_url_ecrire("naviguer","id_rubrique=$id_rubrique")."'":"") - . ">"._T('lien_racine_site')."</$tag>" - . aide ("rubhier") + . ">"._T('info_racine_site')."</$tag>" . "</span>" . $parents + . aide ("rubhier") . "</li></ul>"; } -- GitLab