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

#794 closed bug (fixed)

datepicker cant work when ui.core.js loaded twice

Reported by: foxmask Owned by:
Priority: normal Milestone: jelix 1.1
Component: jelix:forms Version: 1.0RC1
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

When we need to use the ui.tabs.min.js, we also have to load the ui.core.min.js before ( otherwise we have a jquery error ) and everything works great.

Now if in the same form we use the datapicker + ui.tabs, the datepicker does not work anymore. As the ui.core.min.js is loaded twice :

  • once for ui.tabs.min.js,
  • once from datapicker.js,

the functionnality of datapicker is broken.

To make datepicker work we have to comment line 23 of jelix\js\jforms\datepickers\default\init.js

/* this._jsFiles.push(this._config.basePath+'jelix/jquery/ui/ui.core.min.js'); */

Attachments (1)

794-jForms-ui.core-should-not-load-twice.diff (1.5 KB) - added by Julien 12 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 12 years ago by Julien

Hello,

it's not possible to comment this line, because we need ui.core to make the date picker work.

I'll see if there's a way to check if ui.core is already loaded.

comment:2 Changed 12 years ago by foxmask

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

tested and working !

comment:3 Changed 12 years ago by foxmask

  • Resolution fixed deleted
  • Status changed from closed to reopened

may be i close the bug too ealier and the patch should be put in review+ ?

comment:4 Changed 12 years ago by Julien

  • Resolution set to fixed
  • review set to review+
  • Status changed from reopened to closed

that's ok

committed in the trunk and 1.1.x branch, r1249

comment:5 Changed 12 years ago by laurentj

  • Component changed from jelix to jelix:forms
  • Milestone set to jelix 1.1
Note: See TracTickets for help on using tickets.