Grapher doesn't do @Inject(optional=true) properly

[GoogleCodeExporter]

We attempted to graph our app, but it exploded because we were missing some
bindings. But the injection-points for those bindings were optional, and
Guice proper would have ignored them.

When the graphing code calls getBinding() for an optional injection point,
it should catch the ConfigurationException that may be thrown. In that
case, it should pretend like the optional binding does not exist.

Original issue reported on code.google.com by limpbizkit on 2 Feb 2009 at 10:27

[GoogleCodeExporter]

Do you have a stack trace (perhaps best to send internally) or better repro? I 
tried a adding an unsatisfied 
optional injection to the demo app and it graphed (the dependency was not shown 
on the graph, but it didn't 
explode).

Original comment by phopkins on 4 Feb 2009 at 4:05

• Changed state: Accepted

[GoogleCodeExporter]

Original comment by limpbizkit on 26 Apr 2009 at 9:27

• Added labels: Extension-Grapher, Priority-Medium

[GoogleCodeExporter]

Closing because I couldn't repro.

Original comment by phopkins on 29 Apr 2009 at 1:30

• Changed state: Invalid