Uploaded image for project: 'Sling'
  1. Sling
  2. SLING-7139

ScriptEngineManagerFactory does not register ScriptEngineFactory references

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Critical
    • Resolution: Fixed
    • None
    • Scripting Core 2.0.48
    • Scripting
    • None

    Description

      It seems that the bind methods are not called any more, the simple fix that I found was

      diff --git a/bundles/scripting/core/src/main/java/org/apache/sling/scripting/core/impl/ScriptEngineManagerFactory.java b/bundles/scripting/core/src/main/java/org/apache/sling/scripting/core/impl/ScriptEngineManagerFactory.java
      index fe60415c17..79fef581eb 100644
      --- a/bundles/scripting/core/src/main/java/org/apache/sling/scripting/core/impl/ScriptEngineManagerFactory.java
      +++ b/bundles/scripting/core/src/main/java/org/apache/sling/scripting/core/impl/ScriptEngineManagerFactory.java
      @@ -60,7 +60,9 @@ import org.slf4j.LoggerFactory;
               name = "ScriptEngineFactory",
               service = ScriptEngineFactory.class,
               policy = ReferencePolicy.DYNAMIC,
      -        cardinality = ReferenceCardinality.MULTIPLE
      +        cardinality = ReferenceCardinality.MULTIPLE,
      +        bind = "bindScriptEngineFactory",
      +        unbind = "unbindScriptEngineFactory"
           )
       )
       public class ScriptEngineManagerFactory implements BundleListener {
      

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            rombert Robert Munteanu
            rombert Robert Munteanu
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment