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.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.
Opened 13 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)
Change History (3)
Changed 13 years ago by doubleface
comment:1 Changed 13 years ago by laurentj
- review changed from review? to review+
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.
I trust you about this change :-)