Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
[FIXED JENKINS-8298] Reloading persona does not reloads existing pers…
…ona quotes

When using the reload persona url, it does not reload personas quotes. It only loads potentialy new personas.
Looks like a Persona.reload() is missing in hudson.plugins.persona.xml.XmlPersonaReloader.doIndex(StaplerResponse rsp).

Signed-off-by: Whren <whren@free.fr>
  • Loading branch information
whren authored and ssogabe committed Feb 25, 2012
1 parent cd21364 commit bf35780
Showing 1 changed file with 7 additions and 1 deletion.
Expand Up @@ -7,6 +7,7 @@
import hudson.model.Hudson;
import hudson.model.RootAction;
import hudson.plugins.persona.Persona;
import org.dom4j.DocumentException;
import org.kohsuke.stapler.StaplerResponse;

import java.io.IOException;
Expand Down Expand Up @@ -39,7 +40,12 @@ public void doIndex(StaplerResponse rsp) throws IOException {
ExtensionList<Persona> all = Persona.all();

for (XmlBasedPersona p : Util.filter(all,XmlBasedPersona.class)) {
w.println("Reloaded "+p.xml);
try {
p.reload();
w.println("Reloaded "+p.xml);
} catch (DocumentException de) {
w.println("Error on reloading " + p.xml);
}
}

// find new personas
Expand Down

0 comments on commit bf35780

Please sign in to comment.