Uploaded image for project: 'Magnolia Data Module (closed)'
  1. Magnolia Data Module (closed)
  2. MGNLDATA-154

The property automatedExecution/enabled should be optional when creating an importer

XMLWordPrintable

    • Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Neutral Neutral
    • 1.7.3
    • 1.7.2
    • None

      I created an importer for a data type. The importer works great but was giving me a NullPointerException when starting the server. It's because I did not create a content node and data node automatedExecution/enabled=>false. I did not create the node because I don't need scheduled execution. The importer is throwing a NPE because of this line of code in DataModule.java in the initImporter method:
      if (importer.getAutomatedExecution().isEnabled()) { //getAutomatedExecution() return null

      The property automatedExecution/enabled will not be mandatory if you replace with something like this:
      if (importer.getAutomatedExecution() != null && importer.getAutomatedExecution().isEnabled()) {
      You can also initialize AutomatedExecutionConfig to disable by default instead of leaving it null.

      thanks

        Acceptance criteria

              ochytil Ondrej Chytil
              dunand Jean-Francois Nadeau
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated:
                Resolved:

                  Estimated:
                  Original Estimate - 10m
                  10m
                  Remaining:
                  Remaining Estimate - 10m
                  10m
                  Logged:
                  Time Spent - Not Specified
                  Not Specified