developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#527 closed new feature (fixed)

plugin jtpl History ou Fil d'ariane

Reported by: Lipki Owned by: Lipki
Priority: low Milestone: Jelix 1.1 beta 1
Component: jelix:plugins Version: 1.0.2
Severity: minor Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

Voila j'ai voulu créer quelque choses de concret en utilisant le plugin History.

Cela donne le plugin jtpl history.
Ce plugin affiche une liste des dernières actions effectué.
Une démo plus explicite sur mon site http://jelix.toopi.info/page/ariane.

The plugin jtpl history.
This plugin displays a list of recent actions.
A demo more explicit on my site http://jelix.toopi.info/page/ariane.

Attachments (4)

function.history.php (1.9 KB) - added by Lipki 12 years ago.
ariane.2.diff (2.5 KB) - added by Lipki 12 years ago.
ariane.diff (2.4 KB) - added by Lipki 12 years ago.
jelix-trunk-#527.patch (4.1 KB) - added by bibo 12 years ago.
bug fixes

Download all attachments as: .zip

Change History (22)

Changed 12 years ago by Lipki

comment:1 Changed 12 years ago by Lipki

  • Owner set to Lipki
  • Status changed from new to assigned

comment:2 Changed 12 years ago by laurentj

  • Milestone set to Jelix 1.1
  • review changed from review? to review-
  • Pourquoi mettre le label dans la liste ?
  • il manque une petite vérification sur l'existance de la variable de session

comment:3 Changed 12 years ago by Lipki

Pourquoi mettre le label dans la liste ?

Je ne comprend pas la question, il faut bien mettre le label, pour pouvoir cliquez sur quelque chose.

Sinon changement du nom pour "Ariane"

comment:4 Changed 12 years ago by Lipki

  • review changed from review- to review+

comment:5 Changed 12 years ago by Lipki

  • review changed from review+ to review?

comment:6 Changed 12 years ago by laurentj

  • review changed from review? to review-

Je parle du $label que l'on passe en paramètre:

if( $label ) echo '<li value="0" class="label">'.$label.'</li>'; 

À priori, il s'agit du titre de la liste, c'est ça ? Dans ce cas, pourquoi le mettre dans la liste. D'ailleurs, pourquoi donner un label en paramètre ?

Il faut indiquer jUrl::XMLSTRING à jUrl

jUrl::get($page['action'], $page['params'],jUrl::XMLSTRING)

Il faudrait aussi passer à la ligne aprés chaque if, pour que le code soit plus lisible.

Sinon le reste me parait ok.

comment:7 Changed 12 years ago by Lipki

Le paramètre label permet de placer une première ligne dans la liste si l'on veut, mais ce n'est qu'une option.

on peut l'utiliser

{ariane '>>', 'Your path : ', 5}

ce qui permet comme sur mon site un rendu comme ceci.

Your path : Accueil >> History >> Swf >> Accueil

ou pas

{ariane '>>', false, 5}

Peut être que tu préférerez placer les paramètres dans cette ordre

{ariane [ longeur [, separataur [, label ]]]}

{ariane 5}
{ariane 5, '>>'}
{ariane 5, '>>', 'Your path : '}

Ce qui permet de spécifier ce label seulement si on le souhait.

comment:8 Changed 12 years ago by laurentj

Et faire ça, c'est pas pareil ?

<div> Your Path: {ariane '>>', 5}</div>

Avec quelques styles css bien placé, ça revient au même. Ça fait bizarre de mettre le titre d'une liste DANS la liste. Semantiquement parlant, ce n'est pas vraiment logique.

En tout cas, cet argument devrait être le dernier.

comment:9 Changed 12 years ago by Lipki

je me plis

Changed 12 years ago by Lipki

Changed 12 years ago by Lipki

comment:10 Changed 12 years ago by Lipki

  • review changed from review- to review?

comment:11 Changed 12 years ago by laurentj

  • review changed from review? to review+

ok pour ariane.diff

comment:12 Changed 12 years ago by Lipki

  • Resolution set to fixed
  • Status changed from assigned to closed

commmiiit

comment:13 Changed 12 years ago by bballizlife

I'm a little bit late with this, sorry. I'm ok with that jTpl plugin but i think the comments in the header of the file should say that this plugin depends and needs the History coord plugin. Don't you think too ?

comment:14 Changed 12 years ago by Lipki

yes it's true.

comment:15 Changed 12 years ago by bibo

  • Resolution fixed deleted
  • Status changed from closed to reopened

I reopen the bug to attach a new patch. this one will add :

  • the missing ariane plugin file to the manifest
  • raise an exception if it does not find history coord plugin
  • use history session name defined in config file

Changed 12 years ago by bibo

bug fixes

comment:16 Changed 12 years ago by bibo

  • review changed from review+ to review?

comment:17 Changed 12 years ago by laurentj

  • Resolution set to fixed
  • review review? deleted
  • Status changed from reopened to closed

please stop reopen ticket which are close just to add feature or fix bugs. Create new tickets !

comment:18 Changed 12 years ago by bibo

sorry for spam. I created http://developer.jelix.org/ticket/560

Note: See TracTickets for help on using tickets.