Skip to content
Permalink
Browse files

[FIXED JENKINS-18589]

Use DescribableList to handle the copy-on-write semantics correctly. The vector class just doesn't cut it, and we've been setting a new value to this field, which will violates all sorts of the concurrent programming practice.

This change has the nice side effect of removing {{class="vector"}} from the persisted XML. A test is added to make sure we can still read back such an XML.

Originally-Committed-As: 7facc7733c7040536d4074a2c5805b75ee1d8f35
  • Loading branch information...
kohsuke committed Jul 2, 2013
1 parent 4f70763 commit a779e6769320d88abc905a454f2fe7f45f5ed5f0
Showing with 1 addition and 1 deletion.
  1. +1 −1 src/main/java/hudson/maven/AbstractMavenProject.java
@@ -184,7 +184,7 @@ protected AbstractMavenProject(ItemGroup parent, String name) {
addTransientActionsFromBuild(getLastBuild(),r,added);
addTransientActionsFromBuild(getLastSuccessfulBuild(),r,added);

for (Trigger<?> trigger : triggers)
for (Trigger<?> trigger : triggers())
r.addAll(trigger.getProjectActions());

return r;

0 comments on commit a779e67

Please sign in to comment.
You can’t perform that action at this time.