Uploaded image for project: 'Weld'
  1. Weld
  2. WELD-687

Refactoring InstantiatorFactory to allow per-deployment configuration

    Details

    • Similar Issues:
      Show 10 results 

      Description

      InstantiatorFactory is currently using static fields for storing the enabled/disabled status for unsafe instantiators, as well as the instantiator that should be used. In a shared library scenario, the instantiator factory will be enabled if at least one of the deployed applications contains the META-INF/org.jboss.weld.enableUnsafeProxies resource.

        Gliffy Diagrams

          Activity

          Hide
          marius.bogoevici Marius Bogoevici added a comment -

          Actually. it's more complicated than this. InstantiatorFactory will decide on whether unsafe proxies are enabled or not depending on whether the first CDI application deployed on the server had unsafe proxies enabled or not.

          Show
          marius.bogoevici Marius Bogoevici added a comment - Actually. it's more complicated than this. InstantiatorFactory will decide on whether unsafe proxies are enabled or not depending on whether the first CDI application deployed on the server had unsafe proxies enabled or not.
          Hide
          marius.bogoevici Marius Bogoevici added a comment -

          Marked as a bug, this should not work only when weld is shared.

          Show
          marius.bogoevici Marius Bogoevici added a comment - Marked as a bug, this should not work only when weld is shared.
          Hide
          pmuir Pete Muir added a comment -

          Disabled

          org.jboss.weld.tests.proxy.privateconstructor.ProxyConstructorTest
          org.jboss.weld.tests.proxy.weld56.ProxyTest
          org.jboss.weld.tests.proxy.weld56.ProxyTest

          until this issue is fixed.

          Show
          pmuir Pete Muir added a comment - Disabled org.jboss.weld.tests.proxy.privateconstructor.ProxyConstructorTest org.jboss.weld.tests.proxy.weld56.ProxyTest org.jboss.weld.tests.proxy.weld56.ProxyTest until this issue is fixed.
          Hide
          pmuir Pete Muir added a comment -

          We should alter InstantiatorFactory to be a per-BDA internal-only service.

          Show
          pmuir Pete Muir added a comment - We should alter InstantiatorFactory to be a per-BDA internal-only service.
          Hide
          pmuir Pete Muir added a comment -

          Nik, can you take a look at this?

          Show
          pmuir Pete Muir added a comment - Nik, can you take a look at this?
          Hide
          pmuir Pete Muir added a comment -

          Nik, any hope you can look at this?

          Show
          pmuir Pete Muir added a comment - Nik, any hope you can look at this?

            People

            • Assignee:
              alesj Ales Justin
              Reporter:
              marius.bogoevici Marius Bogoevici
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development