Uploaded image for project: 'Magnolia'
  1. Magnolia
  2. MAGNOLIA-5231

Don't use org.apache.commons.lang.exception.NestableException any longer

    XMLWordPrintable

Details

    • Task
    • Resolution: Fixed
    • Neutral
    • 5.0.2, 5.1
    • 5.0.1
    • None
    • None

    Description

      NestableException had been introduced before Exceptions could be nested out of the box (introduced with Java 1.4). Looks like we overlooked one place where we still used it: MgnlCommand#executeSynchronzide

      try {
          success = cmd.execute((info.magnolia.context.Context) ctx);
      } catch (Exception e) {
          throw new NestableException("Exception during executing command", e);
      } finally {
          cmd.release();
      }
      

      The ugly consequence is that - in case of exceptions - we get unnecessarily long stack traces.

      Checklists

        Acceptance criteria

        Attachments

          Issue Links

            Activity

              People

                dlipp Daniel Lipp
                dlipp Daniel Lipp
                Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved:

                  Checklists

                    Task DoR