headwirecom / aemdc Goto Github PK

File operations are all relative to the startup directory of AEMDC. For IntelliJ this is not going to work.

I added this to the Config:

private static File projectRoot = new File(".");

public static String getProjectRootPath() {
return projectRoot.getPath();
}

To make config work relative to my project but creating components does not work as for example CopyDIrCommand does copy it to the startup folder of IntelliJ. All file operation have to be made relative to a given project root.

Tomorrow I will clean up my changes, create a branch and commit.

in a lazybones project, when using aemdc config the TARGET_JAVA_PACKAGE is created incorrectly

in the stored-params.properties the setting is

slingModelsPackage=com.headwire.aem.base.models

referencing the models package, not the root java package for the project

suggestion:

TARGET_JAVA_PACKAGE set to the parent package of slingModelsPackage
introduction of
TARGET_JAVA_MODEL_PACKAGE set to slingModelsPackage

Nice tool 👍

Playing around w/ it, my biggest challenges were

1. providing property key/vals
2. switching out of intellij to command line

While i use intellij, i dont customize it too much -- but was wondering if you guys know off the top of your heads if creating a custom external tool config in intellij, and creating commands for atleast some of the more basic stuff (aemdc Model, Servlet, etc.) sounds feasible.

Wanted to just pick your brains before messing around w it.

Generating a safemethods servlet and settings package to "test.two" will create the file inside com.headwire.aemdc.sample.servlets/two but that Servlet's package is set to test.two.

Java Path and Package are both the same (dots -> slashes): com.headwire.aemdc.samples.

I would expect that the file is copied into "com/headwire/aemdc/samples/servlets/test/two" sub folder and package set to "com.headwire.aemdc.samples.servlets.test.two".

While creating a Dialog to let the user edit the aemdc-config.properties I found the structure of that file cumbersome as it does not just contain the changeable values but other tool related items.

I suggest that we change it to a file that only contains configurable items. This way I don't have to search and replace strings in that file but can just write properties-name=properties-value to the file.

These are the values I found so far:

• aemdc-files location
• UI Folder (content package)
• Apps Folder
• Conf Folder
• Designs Folder
• Java Folder (service bundle)
• Java Package
• Java Path
• OSGi Configuration

Thanks - Andy

I just downloaded and tried to build aemdc on my Mac OS machine with Java 8. Maven reports:

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-assembly-plugin:3.0.0:single (resource-zips) on project aemdc: Execution resource-zips of goal org.apache.maven.plugins:maven-assembly-plugin:3.0.0:single failed: group id '334330370' is too big ( > 2097151 ). Use STAR or POSIX extensions to overcome this limit -> [Help 1]

The problem is that some users get very large group id's. The solution is to use posix mode as noted in maven-assembly-plugin doc

bld_log.txt

Using MainApp.getScene() and adding it to a JFXPanel shows one view. Removing the enclosing Swing component and reading to a TabbedPane or hiding / showing the View makes the scene appear twice (side by side). I could fix the tab issue by re-creating the JFX Panel but the hiding and showing the view is not calling my container.

There are two different issues with the System Out / Err redirection when using AEMDC inside IntelliJ:

1. In IntelliJ there are dependencies that write to the System Out / Err which are not inside a JavaFX thread causing unexpected exceptions and making the plugin fail

2. The redirect is redirecting output that does not belong to AEMDC and hide it from the Plugin console.

Contemplating multiple solutions I thing this might be the right one:

1. Create a JavaFX output stream wrapper that takes the original System Out and Error stream and the Console
2. That wrapper then checks if the thread is JavaFx and then either writes it to the original stream or to the Console
3. Console should be changed to writing bigger chunks that a single character as the test would add additional time

Let me know if that works for you.