Skip to content

Commit 45929ad

Browse files
committed
Improve version display
- show the actual PMD version in title - make the pmd.designer.version show up when running in IntelliJ (and no pom.properties file is found)
1 parent 2f08ccf commit 45929ad

File tree

5 files changed

+8
-5
lines changed

5 files changed

+8
-5
lines changed

CONTRIBUTING.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
paste the following:
2626
```shell
2727
git update-index --skip-worktree -- .idea/misc.xml pmd-designer.iml # Ignore some config files
28-
mvn process-resources # Generate CSS resources
28+
./mvnw process-resources # Generate CSS resources
2929
```
3030

3131
4. [Synchronize the directory contents](https://stackoverflow.com/a/4599243/6245827) to pick-up on the new CSS files

pom.xml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,9 @@
5454
</developers>
5555

5656
<properties>
57+
<pmd.designer.version>${project.version}</pmd.designer.version> <!-- see DesignerVersion -->
5758
<pmd.core.version>7.0.0-SNAPSHOT</pmd.core.version>
59+
5860
<openjfx.version>11.0.2</openjfx.version>
5961
<java.version>8</java.version>
6062
<kotlin.version>1.7.20</kotlin.version>

src/main/java/net/sourceforge/pmd/util/fxdesigner/Designer.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
import java.util.Objects;
1111
import java.util.stream.Collectors;
1212

13+
import net.sourceforge.pmd.PMDVersion;
1314
import net.sourceforge.pmd.util.fxdesigner.app.DesignerParams;
1415
import net.sourceforge.pmd.util.fxdesigner.app.DesignerRoot;
1516
import net.sourceforge.pmd.util.fxdesigner.app.DesignerRootImpl;
@@ -72,7 +73,7 @@ public void stop() {
7273
public void start(Stage stage, DesignerRoot owner) throws IOException {
7374
this.designerRoot = owner;
7475

75-
stage.setTitle("PMD Rule Designer (v " + DesignerVersion.getCurrentVersion() + ')');
76+
stage.setTitle("PMD Rule Designer " + DesignerVersion.getCurrentVersion() + " (with PMD " + PMDVersion.VERSION + ")");
7677
setIcons(stage);
7778

7879
System.out.println(stage.getTitle() + " initializing... ");

src/main/java/net/sourceforge/pmd/util/fxdesigner/DesignerVersion.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,10 +25,10 @@ public final class DesignerVersion {
2525
private static final String UNKNOWN_VERSION = "unknown";
2626

2727
/**
28-
* Determines the version from maven's generated pom.properties file.
28+
* Determines the version from maven's generated designer.properties file.
2929
*/
3030
static {
31-
VERSION = readProperty("/META-INF/maven/net.sourceforge.pmd/pmd-designer/pom.properties", "version").orElse(UNKNOWN_VERSION);
31+
VERSION = readProperty(ResourceUtil.resolveResource("designer.properties"), "pmd.designer.version").orElse(UNKNOWN_VERSION);
3232
PMD_CORE_MIN_VERSION = readProperty(ResourceUtil.resolveResource("designer.properties"), "pmd.core.version").orElse(UNKNOWN_VERSION);
3333
}
3434

src/main/java/net/sourceforge/pmd/util/fxdesigner/popups/SimplePopups.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -210,7 +210,7 @@ public static void showAboutPopup(DesignerRoot root) {
210210
String sb =
211211
"PMD core version:\t\t" + PMDVersion.VERSION + "\n"
212212
+ "Designer version:\t\t\t" + DesignerVersion.getCurrentVersion()
213-
+ " (supports PMD core " + DesignerVersion.getPmdCoreMinVersion() + ")\n"
213+
+ " (min. supported PMD core " + DesignerVersion.getPmdCoreMinVersion() + ")\n"
214214
+ "Designer settings dir:\t\t"
215215
+ root.getService(DesignerRoot.DISK_MANAGER).getSettingsDirectory() + "\n"
216216
+ "Available languages:\t\t"

0 commit comments

Comments
 (0)