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

#629 closed bug (fixed)

history tpl plugin : bug fixes

Reported by: bibo Owned by: bibo
Priority: normal Milestone: Jelix 1.1 beta 1
Component: jelix:plugins Version: trunk
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

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 (1)

jelix-trunk-#629.patch (4.1 KB) - added by bibo 12 years ago.

Download all attachments as: .zip

Change History (7)

Changed 12 years ago by bibo

comment:1 Changed 12 years ago by bibo

  • review set to review?

comment:2 Changed 12 years ago by bibo

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

comment:3 Changed 12 years ago 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" ?

comment:4 Changed 12 years ago by laurentj

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

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

r+

comment:5 Changed 12 years ago by bibo

I'll rename it before commit.

comment:6 Changed 12 years ago by bibo

  • Resolution set to fixed
  • Status changed from assigned to closed
Note: See TracTickets for help on using tickets.