Skip to content

Commit 66b4808

Browse files
author
Vitaliy Boyko
committed
Removed caching the project global object in singletons
1 parent 14ac491 commit 66b4808

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

41 files changed

+81
-96
lines changed

src/com/magento/idea/magento2plugin/actions/generation/dialog/CreateAPluginDialog.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -232,7 +232,7 @@ public static void open(
232232
}
233233

234234
private void createUIComponents() {
235-
final List<String> allModulesList = ModuleIndex.getInstance(project)
235+
final List<String> allModulesList = new ModuleIndex(project)
236236
.getEditableModuleNames();
237237

238238
this.pluginModule = new FilteredComboBox(allModulesList);

src/com/magento/idea/magento2plugin/actions/generation/dialog/CreateAnObserverDialog.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -192,7 +192,7 @@ public String getObserverModule() {
192192
}
193193

194194
private void createUIComponents() { //NOPMD
195-
final List<String> allModulesList = ModuleIndex.getInstance(project)
195+
final List<String> allModulesList = new ModuleIndex(project)
196196
.getEditableModuleNames();
197197

198198
this.observerModule = new FilteredComboBox(allModulesList);

src/com/magento/idea/magento2plugin/actions/generation/dialog/NewModuleDialog.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,7 @@ public NewModuleDialog(
123123
this.project = project;
124124
this.initialBaseDir = initialBaseDir;
125125
this.camelCaseToHyphen = CamelCaseToHyphen.getInstance();
126-
this.moduleIndex = ModuleIndex.getInstance(project);
126+
this.moduleIndex = new ModuleIndex(project);
127127
detectPackageName(initialBaseDir);
128128
setContentPane(contentPane);
129129
setModal(true);

src/com/magento/idea/magento2plugin/actions/generation/dialog/OverrideClassByAPreferenceDialog.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -235,7 +235,7 @@ public static void open(final @NotNull Project project, final PhpClass targetCla
235235
}
236236

237237
private void createUIComponents() {
238-
final List<String> allModulesList = ModuleIndex.getInstance(project)
238+
final List<String> allModulesList = new ModuleIndex(project)
239239
.getEditableModuleNames();
240240

241241
this.preferenceModule = new FilteredComboBox(allModulesList);

src/com/magento/idea/magento2plugin/actions/generation/dialog/OverrideInThemeDialog.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@ public static void open(final @NotNull Project project, final PsiFile psiFile) {
131131
}
132132

133133
private void fillThemeOptions() {
134-
final List<String> themeNames = ModuleIndex.getInstance(project).getEditableThemeNames();
134+
final List<String> themeNames = new ModuleIndex(project).getEditableThemeNames();
135135
for (final String themeName: themeNames) {
136136
theme.addItem(themeName);
137137
}

src/com/magento/idea/magento2plugin/actions/generation/generator/CLICommandClassGenerator.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ public CLICommandClassGenerator(
4848
this.phpClassData = phpClassData;
4949

5050
this.dirGenerator = DirectoryGenerator.getInstance();
51-
this.fileGenerator = FileFromTemplateGenerator.getInstance(project);
51+
this.fileGenerator = new FileFromTemplateGenerator(project);
5252
this.validatorBundle = new ValidatorBundle();
5353
this.commonBundle = new CommonBundle();
5454
}
@@ -83,7 +83,7 @@ private PhpFile createCLICommandClass(final String actionName) {
8383
private PsiDirectory getParentDirectory() {
8484
final String moduleName = this.phpClassData.getModuleName();
8585
final String[] subDirectories = this.phpClassData.getParentDirectory().split("/");
86-
PsiDirectory parentDirectory = ModuleIndex.getInstance(project)
86+
PsiDirectory parentDirectory = new ModuleIndex(project)
8787
.getModuleDirectoryByModuleName(moduleName);
8888
for (final String subDirectory : subDirectories) {
8989
parentDirectory = dirGenerator.findOrCreateSubdirectory(parentDirectory, subDirectory);

src/com/magento/idea/magento2plugin/actions/generation/generator/CronjobClassGenerator.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ public CronjobClassGenerator(Project project, @NotNull CronjobClassData cronjobC
3434
this.cronjobClassData = cronjobClassData;
3535

3636
this.directoryGenerator = DirectoryGenerator.getInstance();
37-
this.fileFromTemplateGenerator = FileFromTemplateGenerator.getInstance(project);
37+
this.fileFromTemplateGenerator = new FileFromTemplateGenerator(project);
3838
this.validatorBundle = new ValidatorBundle();
3939
}
4040

@@ -82,7 +82,7 @@ private PhpFile createCronjobClass(String actionName) {
8282
String cronjobClassName = this.cronjobClassData.getClassName();
8383
String moduleName = this.cronjobClassData.getModuleName();
8484
String[] cronjobSubDirectories = this.cronjobClassData.getDirectory().split("/");
85-
PsiDirectory parentDirectory = ModuleIndex.getInstance(project).getModuleDirectoryByModuleName(moduleName);
85+
PsiDirectory parentDirectory = new ModuleIndex(project).getModuleDirectoryByModuleName(moduleName);
8686

8787
for (String cronjobSubDirectory: cronjobSubDirectories) {
8888
parentDirectory = directoryGenerator.findOrCreateSubdirectory(parentDirectory, cronjobSubDirectory);

src/com/magento/idea/magento2plugin/actions/generation/generator/DataModelGenerator.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ public DataModelGenerator(final Project project, final DataModelData modelData)
4444
this.project = project;
4545
this.modelData = modelData;
4646
this.directoryGenerator = DirectoryGenerator.getInstance();
47-
this.fileFromTemplateGenerator = FileFromTemplateGenerator.getInstance(project);
47+
this.fileFromTemplateGenerator = new FileFromTemplateGenerator(project);
4848
this.getFirstClassOfFile = GetFirstClassOfFile.getInstance();
4949
this.validatorBundle = new ValidatorBundle();
5050
this.commonBundle = new CommonBundle();
@@ -122,7 +122,7 @@ private List<String> getUses() {
122122
}
123123

124124
private PhpClass createModel(final String actionName) {
125-
PsiDirectory parentDirectory = ModuleIndex.getInstance(project)
125+
PsiDirectory parentDirectory = new ModuleIndex(project)
126126
.getModuleDirectoryByModuleName(modelData.getModuleName());
127127
final PsiFile interfaceFile;
128128
final Properties attributes = getAttributes();

src/com/magento/idea/magento2plugin/actions/generation/generator/DataModelInterfaceGenerator.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ public DataModelInterfaceGenerator(
4444
this.project = project;
4545
this.interfaceData = interfaceData;
4646
this.directoryGenerator = DirectoryGenerator.getInstance();
47-
this.fileFromTemplateGenerator = FileFromTemplateGenerator.getInstance(project);
47+
this.fileFromTemplateGenerator = new FileFromTemplateGenerator(project);
4848
this.getFirstClassOfFile = GetFirstClassOfFile.getInstance();
4949
this.validatorBundle = new ValidatorBundle();
5050
this.commonBundle = new CommonBundle();
@@ -101,7 +101,7 @@ protected void fillAttributes(final Properties attributes) {
101101
}
102102

103103
private PhpClass createInterface(final String actionName) {
104-
PsiDirectory parentDirectory = ModuleIndex.getInstance(project)
104+
PsiDirectory parentDirectory = new ModuleIndex(project)
105105
.getModuleDirectoryByModuleName(interfaceData.getModuleName());
106106
final PsiFile interfaceFile;
107107
final Properties attributes = getAttributes();

src/com/magento/idea/magento2plugin/actions/generation/generator/MessageQueueClassGenerator.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ public MessageQueueClassGenerator(
5050

5151
this.messageQueueClassDataName = messageQueueClassData;
5252
this.directoryGenerator = DirectoryGenerator.getInstance();
53-
this.fileFromTemplateGenerator = FileFromTemplateGenerator.getInstance(project);
53+
this.fileFromTemplateGenerator = new FileFromTemplateGenerator(project);
5454
this.validatorBundle = new ValidatorBundle();
5555
this.commonBundle = new CommonBundle();
5656
this.getFirstClassOfFile = GetFirstClassOfFile.getInstance();
@@ -112,7 +112,7 @@ protected void fillAttributes(final Properties attributes) {
112112
}
113113

114114
private PhpClass createHandlerClass(final String actionName) {
115-
PsiDirectory parentDirectory = ModuleIndex.getInstance(project)
115+
PsiDirectory parentDirectory = new ModuleIndex(project)
116116
.getModuleDirectoryByModuleName(this.moduleName);
117117
final PsiFile handlerFile;
118118
final String[] handlerDirectories = messageQueueClassDataName.getPath().split(

0 commit comments

Comments
 (0)