LDEV-6449 fix getDisplayPath() returning null in debugger

- Ensure PageSourceImpl.getDisplayPath() never returns null by providing fallbacks
  (archive path or relPath) when physical file doesn't exist
- Add defensive null/empty check in DebuggerImpl.writeOut() before calling getPageSources()
- Fixes NullPointerException: Cannot invoke String.replace() because realPath is null

Author: michaeloffner

core/src/main/java/lucee/runtime/PageSourceImpl.java

@@ -535,13 +535,16 @@ public String getDisplayPath() {
 		if (dspPath != null) return dspPath;
 
 		if (!mapping.hasArchive()) {
-			return dspPath = StringUtil.toString(getPhyscalFile(), null);
+			String result = StringUtil.toString(getPhyscalFile(), null);
+			return dspPath = result != null ? result : relPath;
 		}
 		else if (isLoad(LOAD_PHYSICAL)) {
-			return dspPath = StringUtil.toString(getPhyscalFile(), null);
+			String result = StringUtil.toString(getPhyscalFile(), null);
+			return dspPath = result != null ? result : getArchiveSourcePath();
 		}
 		else if (isLoad(LOAD_ARCHIVE)) {
-			return dspPath = StringUtil.toString(getArchiveSourcePath(), null);
+			String result = StringUtil.toString(getArchiveSourcePath(), null);
+			return dspPath = result != null ? result : relPath;
 		}
 		else {
 			boolean pse = physcalExists();
@@ -550,11 +553,11 @@ else if (isLoad(LOAD_ARCHIVE)) {
 			if (mapping.isPhysicalFirst()) {
 				if (pse) return dspPath = getPhyscalFile().toString();
 				else if (ase) return dspPath = getArchiveSourcePath();
-				return dspPath = getPhyscalFile().toString();
+				return dspPath = relPath;
 			}
 			if (ase) return dspPath = getArchiveSourcePath();
 			else if (pse) return dspPath = getPhyscalFile().toString();
-			return dspPath = getArchiveSourcePath();
+			return dspPath = relPath;
 		}
 	}
 

core/src/main/java/lucee/runtime/debug/DebuggerImpl.java

@@ -384,23 +384,25 @@ public void writeOut(PageContext pc) throws IOException {
 
 					// load Pagesource of the template
 					String path = debugEntry.getPath();
-					PageSource[] arr = ((PageContextImpl) pc).getPageSources(path);
-					Page p = PageSourceImpl.loadPage(pc, arr, null);
-
-					String fullname = debugEntry.getFullname();
-					if (p != null) {
-						arr = ((PageContextImpl) pc).getPageSources(path);
-						p = PageSourceImpl.loadPage(pc, arr);
-						pc.addPageSource(p.getPageSource(), true);
-						// load info with debug template
-						try {
-							Component c = pc.loadComponent(fullname);
-							ModernAppListener.info(pc, c, args);
+					if (!StringUtil.isEmpty(path)) {
+						PageSource[] arr = ((PageContextImpl) pc).getPageSources(path);
+						Page p = PageSourceImpl.loadPage(pc, arr, null);
+
+						String fullname = debugEntry.getFullname();
+						if (p != null) {
+							arr = ((PageContextImpl) pc).getPageSources(path);
+							p = PageSourceImpl.loadPage(pc, arr);
+							pc.addPageSource(p.getPageSource(), true);
+							// load info with debug template
+							try {
+								Component c = pc.loadComponent(fullname);
+								ModernAppListener.info(pc, c, args);
+							}
+							finally {
+								pc.removeLastPageSource(true);
+							}
+							return;
 						}
-						finally {
-							pc.removeLastPageSource(true);
-						}
-						return;
 					}
 				}
 			}

loader/build.xml

@@ -2,7 +2,7 @@
 <project default="core" basedir="." name="Lucee" 
   xmlns:resolver="antlib:org.apache.maven.resolver.ant">
 
-  <property name="version" value="8.0.0.173-SNAPSHOT"/>
+  <property name="version" value="8.0.0.174-SNAPSHOT"/>
 
   <taskdef uri="antlib:org.apache.maven.resolver.ant" resource="org/apache/maven/resolver/ant/antlib.xml">
     <classpath>

loader/pom.xml

@@ -3,7 +3,7 @@
 
   <groupId>org.lucee</groupId>
   <artifactId>lucee</artifactId>
-  <version>8.0.0.173-SNAPSHOT</version>
+  <version>8.0.0.174-SNAPSHOT</version>
   <packaging>jar</packaging>
 
   <name>Lucee Loader Build</name>