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

#602 closed bug (fixed)

jClasses::createBinded does not work

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

Description

I found a big probleme in my own implementation of dependency injection for jClasses::createBinded. It does not work at all... and the unit tests were not good enough to see it.

This is the only part of bindings that I don't use in my own projects and here is what you get...

Here is a patch wich corrects the problem and adds some unit tests.

The patch is not made from the trunk (still no internet at home) but I think it is good since I am the only on who modified theses files.

Attachments (1)

createBinded.patch (4.1 KB) - added by doubleface 12 years ago.

Download all attachments as: .zip

Change History (3)

Changed 12 years ago by doubleface

comment:1 Changed 12 years ago by laurentj

  • review changed from review? to review+

I trust you about this change :-)

comment:2 Changed 12 years ago by bastnic

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

Commited.

doubleface : it would better if it was a real svn diff ;).

Note: See TracTickets for help on using tickets.