Ticket #602 (closed bug: fixed)

Opened 3 months ago

Last modified 2 months ago

jClasses::createBinded does not work

Reported by: doubleface Assigned to: doubleface
Priority: normal Milestone: Jelix 1.1 beta 1
Component: jelix:utils Version: trunk
Severity: major Keywords:
Cc: Php version:
Review: review+ Hosting Provider:
Documentation needed: 0 Blocking:

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

createBinded.patch (4.1 kB) - added by doubleface on 05/26/08 12:01:09.

Change History

05/26/08 12:01:09 changed by doubleface

  • attachment createBinded.patch added.

05/28/08 11:18:01 changed by laurentj

  • review changed from review? to review+.

I trust you about this change :-)

06/15/08 15:50:53 changed by bastnic

  • status changed from new to closed.
  • resolution set to fixed.

Commited.

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

Download in other formats: Comma-delimited Text Tab-delimited Text RSS Feed