Skip to content

Commit ca788a5

Browse files
ahornaceVladimir Kotal
authored andcommitted
Add check for null project
1 parent 0fbbebc commit ca788a5

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

src/org/opensolaris/opengrok/web/api/v1/controller/ProjectsController.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -210,6 +210,7 @@ public void markIndexed(@PathParam("project") final String projectName) throws E
210210
}
211211
}
212212
}
213+
SuggesterServiceFactory.getDefault().refresh(projectName);
213214
} else {
214215
logger.log(Level.WARNING, "cannot find project {0} to mark as indexed", projectName);
215216
}
@@ -219,8 +220,6 @@ public void markIndexed(@PathParam("project") final String projectName) throws E
219220
env.maybeRefreshIndexSearchers(Collections.singleton(projectName));
220221

221222
env.refreshDateForLastIndexRun();
222-
223-
SuggesterServiceFactory.getDefault().refresh(projectName);
224223
}
225224

226225
@PUT

src/org/opensolaris/opengrok/web/suggester/provider/service/impl/SuggesterServiceImpl.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -169,6 +169,10 @@ public void refresh(final String project) {
169169
Configuration config = env.getConfiguration();
170170

171171
Project p = config.getProjects().get(project);
172+
if (p == null) {
173+
logger.log(Level.WARNING, "Cannot refresh suggester because project for name {0} was not found", project);
174+
return;
175+
}
172176
suggester.rebuild(Collections.singletonList(Paths.get(config.getDataRoot(), IndexDatabase.INDEX_DIR, p.getPath())));
173177
}
174178

0 commit comments

Comments
 (0)