Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.5.14.1
    • Fix Version/s: 2.5.16
    • Component/s: None
    • Labels:
      None

      Description

      As we use a lot of inheritance on action classes we would appreciate, if the allowed methods would be inherited as well.

      For example:

      We do not declare actions in struts.xml and just use the Convention.

      struts.xml:

      <package name="default" extends="tiles-default"  namespace="/„ >
          ...
          <global-allowed-methods> abort,execute,search,reset,download,refresh,delegate,forward,release,saveComments </global-allowed-methods>
      </package>
      

      Java:

      @AllowedMethods(value = \{"changeLanguage", "deleteAttachment", "uploadMailAttachment"})
      public class SpecifyMailTask extends AbstractTask {
      
      @AllowedMethods(value = \{"saveInputAndSuccess"})
      public abstract class AbstractTask extends AbstractBasicJbpmTask implements ModelDriven<TaskModel>, Preparable \{...}
      
      @AllowedMethods(value = { "abort", "cancel", "claim", "claimAndStart", "claimAndSkip", "complete", "delegate", "forward",
              "incomplete", "release", "skipAndComplete", "start", "suspend", "uploadTaskAttachment"})
      public abstract class AbstractBasicJbpmTask extends AbstractCrmAction implements ModelDriven<TaskModel>, Preparable \{...}
      

      I tried to add the allowed methods in the package-info.java. It worked as long as I did not add an additional method to a certain action, i.e. SpecifyMailTask. If I add here the annotation to allow the methods "changeLanguage", "deleteAttachment", "uploadMailAttachment" additionally, the other methods, which are allowed via the package-info.java, are not allowed anymore.

       

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                lukaszlenart Lukasz Lenart
                Reporter:
                Britta Britta Katzenbach
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: