ASF Bugzilla – Attachment 27902 Details for
Bug 50799
Having a non-HTTP sampler in a http test plan prevents multiple header managers from working
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
Remember
[x]
|
Forgot Password
Login:
[x]
[patch]
Fix proposal
BUG_50799.patch (text/plain), 1.74 KB, created by
Philippe Mouawad
on 2011-11-06 17:11:58 UTC
(
hide
)
Description:
Fix proposal
Filename:
MIME Type:
Creator:
Philippe Mouawad
Created:
2011-11-06 17:11:58 UTC
Size:
1.74 KB
patch
obsolete
>Index: src/core/org/apache/jmeter/testelement/AbstractTestElement.java >=================================================================== >--- src/core/org/apache/jmeter/testelement/AbstractTestElement.java (revision 1197646) >+++ src/core/org/apache/jmeter/testelement/AbstractTestElement.java (working copy) >@@ -256,21 +256,29 @@ > return jmp instanceof NullProperty ? defaultValue : jmp.getStringValue(); > } > >- protected void addProperty(JMeterProperty property) { >+ protected void addProperty(JMeterProperty property, boolean clone) { >+ JMeterProperty propertyToPut = property; >+ if(clone) { >+ propertyToPut = property.clone(); >+ } > if (isRunningVersion()) { >- setTemporary(property); >+ setTemporary(propertyToPut); > } else { > clearTemporary(property); > } > JMeterProperty prop = getProperty(property.getName()); > > if (prop instanceof NullProperty || (prop instanceof StringProperty && prop.getStringValue().equals(""))) { >- propMap.put(property.getName(), property); >+ propMap.put(property.getName(), propertyToPut); > } else { >- prop.mergeIn(property); >+ prop.mergeIn(propertyToPut); > } > } > >+ protected void addProperty(JMeterProperty property) { >+ addProperty(property, false); >+ } >+ > protected void clearTemporary(JMeterProperty property) { > if (temporaryProperties != null) { > temporaryProperties.remove(property); >@@ -382,7 +390,7 @@ > PropertyIterator iter = element.propertyIterator(); > while (iter.hasNext()) { > JMeterProperty prop = iter.next(); >- addProperty(prop); >+ addProperty(prop, true); > } > } >
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
Attachments on
bug 50799
:
26671
|
27902