1658: Fix null handling in file navigation and enhance readability.

Improved null safety by ensuring proper checks around file navigation logic. Refactored UI interaction and popup messaging to adhere to threading best practices, enhancing code clarity and preventing race conditions. Updated changelog to reflect the fixed issue.

Author: VitaliyBoyko

CHANGELOG.md

@@ -12,6 +12,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0).
 - "Copy Path/Reference" does not show the preview value [#2497](https://github.com/magento/magento2-phpstorm-plugin/pull/2497)
 - Must not start write action from within read action in the other thread [#2498](https://github.com/magento/magento2-phpstorm-plugin/pull/2498)
 - URN map generation during indexing [#2499](https://github.com/magento/magento2-phpstorm-plugin/pull/2499)
+- Cannot invoke "com.intellij.psi.PsiDirectory.getName() [#2500](https://github.com/magento/magento2-phpstorm-plugin/pull/2500)
 
 ## 2025.0.0
 

src/main/java/com/magento/idea/magento2plugin/actions/generation/generator/OverrideTemplateInThemeGenerator.java

@@ -76,13 +76,19 @@ public void execute(final PsiFile baseFile, final String themeName) {
         directory = getTargetDirectory(directory, pathComponents);
 
         if (directory.findFile(baseFile.getName()) != null) {
-            PsiDirectory finalDirectory1 = directory;
+            final PsiDirectory finalDirectory1 = directory;
             ApplicationManager.getApplication().invokeLater(() -> {
                 JBPopupFactory.getInstance()
-                        .createMessage(validatorBundle.message("validator.file.alreadyExists", baseFile.getName()))
-                        .showCenteredInCurrentWindow(project);
+                        .createMessage(
+                                validatorBundle.message(
+                                        "validator.file.alreadyExists",
+                                        baseFile.getName()
+                                )
+                        ).showCenteredInCurrentWindow(project);
                 ApplicationManager.getApplication().invokeLater(() -> {
-                    Objects.requireNonNull(finalDirectory1.findFile(baseFile.getName())).navigate(true);
+                    Objects.requireNonNull(
+                            finalDirectory1.findFile(baseFile.getName())
+                    ).navigate(true);
                 });
             });
             return;