Ticket #527 (closed new feature: fixed)

Opened 4 months ago

Last modified 4 weeks ago

plugin jtpl History ou Fil d'ariane

Reported by: Lipki Assigned to: Lipki
Priority: low Milestone: Jelix 1.1 beta 1
Component: jelix:plugins Version: 1.0.2
Severity: minor Keywords:
Cc: Php version:
Review: Hosting Provider:
Documentation needed: 0 Blocking:

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

function.history.php (1.9 kB) - added by Lipki on 03/28/08 23:46:21.
ariane.2.diff (2.5 kB) - added by Lipki on 04/25/08 22:09:09.
ariane.diff (2.4 kB) - added by Lipki on 04/25/08 22:12:01.
jelix-trunk-#527.patch (4.1 kB) - added by bibo on 06/23/08 12:38:55.
bug fixes

Change History

03/28/08 23:46:21 changed by Lipki

  • attachment function.history.php added.

03/28/08 23:47:15 changed by Lipki

  • owner set to Lipki.
  • status changed from new to assigned.

04/14/08 23:37:29 changed by laurentj

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

04/23/08 22:11:32 changed 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"

04/23/08 22:12:11 changed by Lipki

  • review changed from review- to review+.

04/23/08 22:20:13 changed by Lipki

  • review changed from review+ to review?.

04/25/08 09:40:46 changed 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.

04/25/08 13:35:26 changed 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.

04/25/08 21:45:27 changed 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.

04/25/08 21:57:37 changed by Lipki

je me plis

04/25/08 22:09:09 changed by Lipki

  • attachment ariane.2.diff added.

04/25/08 22:12:01 changed by Lipki

  • attachment ariane.diff added.

04/25/08 22:12:43 changed by Lipki

  • review changed from review- to review?.

04/30/08 23:29:30 changed by laurentj

  • review changed from review? to review+.

ok pour ariane.diff

04/30/08 23:44:51 changed by Lipki

  • status changed from assigned to closed.
  • resolution set to fixed.

commmiiit

05/02/08 10:31:03 changed 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 ?

05/02/08 14:11:08 changed by Lipki

yes it's true.

06/23/08 12:32:25 changed by bibo

  • status changed from closed to reopened.
  • resolution deleted.

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

06/23/08 12:38:55 changed by bibo

  • attachment jelix-trunk-#527.patch added.

bug fixes

06/23/08 12:39:27 changed by bibo

  • review changed from review+ to review?.

06/23/08 13:21:54 changed by laurentj

  • status changed from reopened to closed.
  • review deleted.
  • resolution set to fixed.

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

06/23/08 14:40:20 changed by bibo

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

Download in other formats: Comma-delimited Text Tab-delimited Text RSS Feed