Fluid Infusion

pager throws an error on load

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Blocker Blocker
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 0.8
  • Component/s: Pager
  • Labels:
    None
  • Environment:
    FF2, FF3, Opera 9.5, Safari 3 (Mac OS 10.5)
    FF2, FF3, IE6, IE7, Opera 9.5 (Win XP)
    FF3, IE7 (Win Vista)
    FF3, IE6 (Win 2000)

Description

pager throws an error on load

http://build.fluidproject.org/fluid/sample-code/pager/basic/pager.html
http://build.fluidproject.org/fluid/sample-code/pager/renderer/pager-render.html

Error: EL is undefined
File: Fluid.js
Line: 593

It seems that this is breaking both versions of pager and the unit tests for it.

Issue Links

Activity

Hide
Justin Obara added a comment -
Bug Parade 0.8 release
Show
Justin Obara added a comment - Bug Parade 0.8 release
Hide
Antranig Basman added a comment - - edited
Pager-renderer now working again, with the tests
Show
Antranig Basman added a comment - - edited Pager-renderer now working again, with the tests
Hide
Justin Obara added a comment -
tested using:

FF2, FF3, Opera 9.5, Safari 3 (Mac OS 10.5)
FF3, IE6 (Win XP)

It appeared to be working.

Waiting on a code review to close
Show
Justin Obara added a comment - tested using: FF2, FF3, Opera 9.5, Safari 3 (Mac OS 10.5) FF3, IE6 (Win XP) It appeared to be working. Waiting on a code review to close
Hide
Justin Obara added a comment -
Colin verified this

Quote from a message to the fluid-work list

"* I reviewed the series of commits that fix FLUID-2179 and FLUID-2192. The Pager is working nicely again in all of the browsers I tested with, and a number of framework improvements came along with this fix. +1 for inclusion in Infusion 0.8."
Show
Justin Obara added a comment - Colin verified this Quote from a message to the fluid-work list "* I reviewed the series of commits that fix FLUID-2179 and FLUID-2192. The Pager is working nicely again in all of the browsers I tested with, and a number of framework improvements came along with this fix. +1 for inclusion in Infusion 0.8."

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: