• Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Neutral Neutral
    • 1.3.6, 1.4.2
    • 1.4.1
    • build
    • None

      The current STK pom files declare a dependency to the forum module, which afaik is not used nor needed. (It would be needed for commenting, which is only added with the ETK)

      Consequences:

      • forum module is added to the stk-bundle (unneeded?)
      • modules depending on stk transitively depend on the forum, thus requiring several tedious exclusions.

      For the record, here's the output of mvn dependency:analyze, which shows more missing and superfluous dependencies (with a pinch of salt, as the dependency to the taglib is needed, for instance - it's used in the template, but not in code, which is why mvn dependency:analyze considers it unneeded)

      [INFO] [dependency:analyze {execution: default-cli}]
      [WARNING] Used undeclared dependencies found:
      [WARNING]    commons-lang:commons-lang:jar:2.4:provided
      [WARNING]    javax.jcr:jcr:jar:1.0:provided
      [WARNING]    org.slf4j:slf4j-api:jar:1.5.11:provided
      [WARNING]    commons-collections:commons-collections:jar:3.2.1:provided
      [WARNING]    rome:rome:jar:1.0:compile
      [WARNING]    log4j:log4j:jar:1.2.16:provided
      [WARNING]    info.magnolia:magnolia-gui:jar:4.4.1:provided
      [WARNING]    javax.servlet:servlet-api:jar:2.4:provided
      [WARNING]    commons-beanutils:commons-beanutils:jar:1.7.0:provided
      [WARNING]    info.magnolia:magnolia-module-exchange-simple:jar:4.4.1:provided
      [WARNING] Unused declared dependencies found:
      [WARNING]    info.magnolia:magnolia-module-forum:jar:1.2.1:compile
      [WARNING]    org.easymock:easymockclassextension:jar:2.4:test
      [WARNING]    com.mockrunner:mockrunner:jar:0.3.1:test
      [WARNING]    info.magnolia:magnolia-module-fckeditor:jar:4.4.1:provided
      [WARNING]    info.magnolia:magnolia-taglib-utility:jar:4.4.1:provided
      [WARNING]    info.magnolia:magnolia-module-form:jar:1.2:compile
      [WARNING]    info.magnolia:magnolia-taglib-cms:jar:4.4.1:provided
      [WARNING]    cglib:cglib-full:jar:2.0.2:compile
      [WARNING]    javax.servlet:jsp-api:jar:2.0:provided
      [WARNING]    info.magnolia:magnolia-module-scheduler:jar:1.4:compile
      

      Additionally, we also see side effects, for instance in user role bootstrap files:

      $ grep -ri forum .
      ./src/main/resources/mgnl-bootstrap/standard-templating-kit/userroles.contact-base.xml:  <sv:node sv:name="acl_forum">
      

        Acceptance criteria

              ochytil Ondrej Chytil
              gjoseph Magnolia International
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

                Created:
                Updated:
                Resolved: