You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by cz...@apache.org on 2008/03/04 14:07:36 UTC
svn commit: r633467 -
/felix/trunk/scrplugin/src/main/java/org/apache/felix/scrplugin/xml/ComponentDescriptorIO.java
Author: cziegeler
Date: Tue Mar 4 05:07:35 2008
New Revision: 633467
URL: http://svn.apache.org/viewvc?rev=633467&view=rev
Log:
IN PROGRESS - issue FELIX-507: properties and service names of extended components not always included
https://issues.apache.org/jira/browse/FELIX-507
Modified:
felix/trunk/scrplugin/src/main/java/org/apache/felix/scrplugin/xml/ComponentDescriptorIO.java
Modified: felix/trunk/scrplugin/src/main/java/org/apache/felix/scrplugin/xml/ComponentDescriptorIO.java
URL: http://svn.apache.org/viewvc/felix/trunk/scrplugin/src/main/java/org/apache/felix/scrplugin/xml/ComponentDescriptorIO.java?rev=633467&r1=633466&r2=633467&view=diff
==============================================================================
--- felix/trunk/scrplugin/src/main/java/org/apache/felix/scrplugin/xml/ComponentDescriptorIO.java (original)
+++ felix/trunk/scrplugin/src/main/java/org/apache/felix/scrplugin/xml/ComponentDescriptorIO.java Tue Mar 4 05:07:35 2008
@@ -427,7 +427,7 @@
public void characters(char[] ch, int start, int length) throws SAXException {
if ( this.pendingProperty != null ) {
final String text = new String(ch, start, length);
- if ( this.pendingProperty.getValue() != null ) {
+ if ( this.pendingProperty.getValue() == null ) {
this.pendingProperty.setValue(text);
} else {
this.pendingProperty.setValue(this.pendingProperty.getValue() + text);