Ticket #629 (closed bug: fixed)

Opened 2 months ago

Last modified 1 month ago

history tpl plugin : bug fixes

Reported by: bibo Assigned to: bibo
Priority: normal Milestone: Jelix 1.1 beta 1
Component: jelix:plugins Version: trunk
Severity: normal Keywords:
Cc: Php version:
Review: review+ Hosting Provider:
Documentation needed: 0 Blocking:

Description

there is a number of small bugs in history tpl plugin (fil d'ariane in french):

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

Attachments

jelix-trunk-#629.patch (4.1 kB) - added by bibo on 06/23/08 14:38:35.

Change History

06/23/08 14:38:35 changed by bibo

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

06/23/08 14:42:07 changed by bibo

  • review set to review?.

06/23/08 14:43:02 changed by bibo

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

07/02/08 14:17:32 changed by bballizlife

I guess this patch is ok, but i don't know if i'm able to give you a review+...

Maybe a little bit off topic but : does the name "ariane" used in english too ? Isn't it called "breadcrumb" ?

07/07/08 11:46:33 changed by laurentj

  • review changed from review? to review+.
  • milestone set to Jelix 1.1 beta 1.

Yes, I think we should rename ariane plugin with "breadcrumb".

r+

07/07/08 12:04:40 changed by bibo

I'll rename it before commit.

07/07/08 17:26:49 changed by bibo

  • status changed from assigned to closed.
  • resolution set to fixed.
Download in other formats: Comma-delimited Text Tab-delimited Text RSS Feed