Skip to content

Commit 888039c

Browse files
committed
Add back backwards compatible IProjectPRoperties.getResolvedRuleSetFile()
1 parent 68ee0cc commit 888039c

File tree

3 files changed

+28
-9
lines changed

3 files changed

+28
-9
lines changed

net.sourceforge.pmd.eclipse.plugin/src/main/java/net/sourceforge/pmd/eclipse/runtime/properties/IProjectProperties.java

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,8 +106,16 @@ public interface IProjectProperties {
106106
/**
107107
* @return Returns the resolved RuleSet File suitable for loading a rule
108108
* set.
109+
* @deprecated use {@link #getResolvedRuleSetFiles()} instead
109110
*/
110-
List<File> getResolvedRuleSetFile() throws PropertiesException;
111+
@Deprecated
112+
File getResolvedRuleSetFile() throws PropertiesException;
113+
114+
/**
115+
* @return Returns the resolved RuleSet Files suitable for loading multiple
116+
* rulesets.
117+
*/
118+
List<File> getResolvedRuleSetFiles() throws PropertiesException;
111119

112120
/**
113121
* @return Returns the project Working Set.

net.sourceforge.pmd.eclipse.plugin/src/main/java/net/sourceforge/pmd/eclipse/runtime/properties/impl/ProjectPropertiesImpl.java

Lines changed: 18 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -189,7 +189,7 @@ public void setProjectRuleSet(final RuleSet projectRuleSet) throws PropertiesExc
189189
this.needRebuild |= !this.projectRuleSet.getRules().equals(projectRuleSet.getRules());
190190
this.projectRuleSet = projectRuleSet;
191191
if (this.ruleSetStoredInProject) {
192-
for (File f : getResolvedRuleSetFile()) {
192+
for (File f : getResolvedRuleSetFiles()) {
193193
if (f != null) {
194194
if (projectRuleFileLastModified < f.lastModified()) {
195195
projectRuleFileLastModified = f.lastModified();
@@ -220,7 +220,7 @@ public void setRuleSetStoredInProject(final boolean ruleSetStoredInProject) thro
220220
throw new PropertiesException(
221221
"The project ruleset file cannot be found for project " + this.project.getName());
222222
}
223-
for (File f : getResolvedRuleSetFile()) {
223+
for (File f : getResolvedRuleSetFiles()) {
224224
if (f != null) {
225225
if (projectRuleFileLastModified < f.lastModified()) {
226226
projectRuleFileLastModified = f.lastModified();
@@ -251,7 +251,7 @@ public void setRuleSetFile(String ruleSetFile) throws PropertiesException {
251251
throw new PropertiesException(
252252
"The project ruleset file cannot be found for project " + project.getName());
253253
}
254-
for (File f : getResolvedRuleSetFile()) {
254+
for (File f : getResolvedRuleSetFiles()) {
255255
if (f != null) {
256256
if (projectRuleFileLastModified < f.lastModified()) {
257257
projectRuleFileLastModified = f.lastModified();
@@ -283,12 +283,13 @@ public void setProjectWorkingSet(final IWorkingSet projectWorkingSet) {
283283
/**
284284
* @see net.sourceforge.pmd.eclipse.runtime.properties.IProjectProperties#isNeedRebuild()
285285
*/
286+
@Override
286287
public boolean isNeedRebuild() {
287288
LOG.debug("Query if project " + project.getName() + " need rebuild : " + (pmdEnabled && needRebuild));
288289
LOG.debug(" PMD Enabled = " + pmdEnabled);
289290
LOG.debug(" Project need rebuild = " + needRebuild);
290291
if (ruleSetStoredInProject) {
291-
for (File f : getResolvedRuleSetFile()) {
292+
for (File f : getResolvedRuleSetFiles()) {
292293
if (f != null) {
293294
needRebuild |= f.lastModified() > projectRuleFileLastModified;
294295
}
@@ -310,18 +311,28 @@ public void setNeedRebuild(final boolean needRebuild) {
310311
*/
311312
public final boolean isRuleSetFileExist() {
312313
boolean ret = true;
313-
for (File f : getResolvedRuleSetFile()) {
314+
for (File f : getResolvedRuleSetFiles()) {
314315
if (!f.exists()) {
315316
ret = false;
316317
}
317318
}
318319
return ret;
319320
}
320321

322+
@Deprecated
323+
@Override
324+
public File getResolvedRuleSetFile() throws PropertiesException {
325+
if (isRuleSetFileExist()) {
326+
return getResolvedRuleSetFiles().get(0);
327+
}
328+
return null;
329+
}
330+
321331
/**
322-
* @see net.sourceforge.pmd.eclipse.runtime.properties.IProjectProperties#getResolvedRuleSetFile()
332+
* @see net.sourceforge.pmd.eclipse.runtime.properties.IProjectProperties#getResolvedRuleSetFiles()
323333
*/
324-
public List<File> getResolvedRuleSetFile() {
334+
@Override
335+
public List<File> getResolvedRuleSetFiles() {
325336
// Check as project-relative path
326337
List<File> files = new ArrayList<File>();
327338
for (String ruleSetFile : getRuleSetFile().split(",")) {

net.sourceforge.pmd.eclipse.plugin/src/main/java/net/sourceforge/pmd/eclipse/runtime/properties/impl/ProjectPropertiesManagerImpl.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -173,7 +173,7 @@ private void loadRuleSetFromProject(IProjectProperties projectProperties) throws
173173
try {
174174
final RuleSetFactory factory = new RuleSetFactory();
175175
RuleSet allRuleSets = null;
176-
for (final File ruleSetFile : projectProperties.getResolvedRuleSetFile()) {
176+
for (final File ruleSetFile : projectProperties.getResolvedRuleSetFiles()) {
177177
RuleSet rs = factory.createRuleSets(ruleSetFile.getPath()).getAllRuleSets()[0];
178178
if (allRuleSets == null) {
179179
/* The first ruleset file */

0 commit comments

Comments
 (0)