From 282c546c2a2fe7d0d704684fabf6663db9184ab2 Mon Sep 17 00:00:00 2001 From: Xavier Le Vourch Date: Sun, 2 Dec 2007 20:19:32 +0000 Subject: [PATCH] bug fix: changing properties didn't trigger a rebuild prompt Both the working set and "ruleSetStoredInProject" properties were ignored Fixes: [ 1661524 ] Working set functionality doesn't work git-svn-id: https://pmd.svn.sourceforge.net/svnroot/pmd/trunk@5685 51baf565-9d33-0410-a72c-fc3788e3496d --- .../pmd/runtime/properties/impl/ProjectPropertiesImpl.java | 2 ++ 1 file changed, 2 insertions(+) diff --git a/pmd-eclipse-plugin/pmd-eclipse-runtime/src/net/sourceforge/pmd/runtime/properties/impl/ProjectPropertiesImpl.java b/pmd-eclipse-plugin/pmd-eclipse-runtime/src/net/sourceforge/pmd/runtime/properties/impl/ProjectPropertiesImpl.java index 88febecfa7..630dae9931 100644 --- a/pmd-eclipse-plugin/pmd-eclipse-runtime/src/net/sourceforge/pmd/runtime/properties/impl/ProjectPropertiesImpl.java +++ b/pmd-eclipse-plugin/pmd-eclipse-runtime/src/net/sourceforge/pmd/runtime/properties/impl/ProjectPropertiesImpl.java @@ -200,6 +200,7 @@ public class ProjectPropertiesImpl implements IProjectProperties { */ public void setRuleSetStoredInProject(final boolean ruleSetStoredInProject) throws PropertiesException { log.debug("Set rule set stored in project for project " + this.project.getName() + ": " + ruleSetStoredInProject); + this.needRebuild |= this.ruleSetStoredInProject != ruleSetStoredInProject; this.ruleSetStoredInProject = ruleSetStoredInProject; if ((this.ruleSetStoredInProject) && (!isRuleSetFileExist())) { throw new PropertiesException("The project ruleset file cannot be found for project " + this.project.getName()); // TODO NLS @@ -220,6 +221,7 @@ public class ProjectPropertiesImpl implements IProjectProperties { log.debug("Set working set for project " + this.project.getName() + ": " + (projectWorkingSet == null ? "none" : projectWorkingSet.getName())); + this.needRebuild |= (this.projectWorkingSet == null)?(projectWorkingSet != null):!this.projectWorkingSet.equals(projectWorkingSet); this.projectWorkingSet = projectWorkingSet; }