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

#791 closed bug (fixed)

check for addJSLink, addCSSLink support before calling it

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

Description

See ticket #548 for details.

Attachments (2)

#791-trunk-addJsLink.patch (1.3 KB) - added by bibo 12 years ago.
check if response format is HTML
#791-trunk-addJsLink.2.patch (2.5 KB) - added by bibo 12 years ago.
fixes regression, trunk only

Download all attachments as: .zip

Change History (8)

Changed 12 years ago by bibo

check if response format is HTML

comment:1 Changed 12 years ago by bibo

  • Summary changed from link_to_remote shoud check for addJSLink support to check for addJSLink, addCSSLink support before calling it

jForms builders should also check addJSLink, addCSSLink support in current response before calling it. This is a regression from #548.

comment:2 Changed 12 years ago by bibo

Ok, This regressed in r1030, on the merging of jforms-group branch.

Patch in a minute for trunk and branch.

Changed 12 years ago by bibo

fixes regression, trunk only

comment:3 Changed 12 years ago by bibo

  • review set to review?

comment:4 Changed 12 years ago by laurentj

  • review changed from review? to review+

for trunk+1.1.x branch

comment:5 Changed 12 years ago by laurentj

  • Component changed from jelix:tpl to jelix:forms

comment:6 Changed 12 years ago by laurentj

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

Landed in the trunk and 1.1.x branch. r1259.

Note: See TracTickets for help on using tickets.