-rw-r--r-- | include/opie/xmltree.h | 2 | ||||
-rw-r--r-- | libopie/xmltree.cc | 42 | ||||
-rw-r--r-- | libopie/xmltree.h | 2 | ||||
-rw-r--r-- | libopie2/opiecore/xmltree.cc | 42 | ||||
-rw-r--r-- | libopie2/opiecore/xmltree.h | 2 |
5 files changed, 40 insertions, 50 deletions
diff --git a/include/opie/xmltree.h b/include/opie/xmltree.h index 7f1b7b7..7f50365 100644 --- a/include/opie/xmltree.h +++ b/include/opie/xmltree.h @@ -84,6 +84,8 @@ public: AttributeMap attributes() const { return m_attributes; } AttributeMap &attributes() { return m_attributes; } + QString attribute(const QString &)const; + void setAttribute( const QString &attr, const QString &value ); void save( QTextStream &stream, uint indent = 0 ); XMLElement *namedItem( const QString &name ); diff --git a/libopie/xmltree.cc b/libopie/xmltree.cc index d5ce74a..3d03cc6 100644 --- a/libopie/xmltree.cc +++ b/libopie/xmltree.cc @@ -17,34 +17,13 @@ Boston, MA 02111-1307, USA. */ - +#include <qpe/stringutil.h> #include <opie/xmltree.h> #include <qxml.h> #include <assert.h> -namespace -{ - -/** - Encode an attribute value upon saving. - replaces '"' with """ - replaces '<' with "<" - replaces '&' with "&" - replaces '>' with ">" -*/ -QString encodeAttr( const QString& str ) -{ - QString encAttr( str ); // cause of the const parameter - encAttr = encAttr.replace( QRegExp( "[<]" ), "<" ); - encAttr = encAttr.replace( QRegExp( "[>]" ), ">" ); - encAttr = encAttr.replace( QRegExp( "[\"]" ), """ ); - encAttr = encAttr.replace( QRegExp( "[&]" ), "&" ); - return encAttr; -} - -} XMLElement::XMLElement() : m_parent( 0 ), m_next( 0 ), m_prev( 0 ), m_first( 0 ), m_last( 0 ) @@ -110,7 +89,18 @@ void XMLElement::insertAfter( XMLElement *newChild, XMLElement *refChild ) if ( next ) next->m_prev = newChild; } - +QString XMLElement::attribute(const QString &attr )const +{ + if ( !m_attributes.contains( attr ) ) + return QString::null; + AttributeMap::ConstIterator it = m_attributes.find( attr ); + return it.data(); +} +void XMLElement::setAttribute(const QString &attr, const QString &value ) +{ + m_attributes.remove( attr ); + m_attributes.insert( attr, value ); +} void XMLElement::insertBefore( XMLElement *newChild, XMLElement *refChild ) { assert( refChild ); @@ -163,7 +153,7 @@ void XMLElement::save( QTextStream &s, uint indent ) { if ( !m_value.isEmpty() ) { - s << encodeAttr( m_value ); + s << Qtopia::escapeString( m_value ); return; } @@ -179,7 +169,7 @@ void XMLElement::save( QTextStream &s, uint indent ) AttributeMap::ConstIterator end = m_attributes.end(); for (; it != end; ++it ) { - s << it.key() << "=\"" << encodeAttr( it.data() ) << "\""; + s << it.key() << "=\"" << Qtopia::escapeString( it.data() ) << "\""; s << " "; } } @@ -324,3 +314,5 @@ XMLElement *XMLElement::load( const QString &fileName ) return handler.root();; } + + diff --git a/libopie/xmltree.h b/libopie/xmltree.h index 7f1b7b7..7f50365 100644 --- a/libopie/xmltree.h +++ b/libopie/xmltree.h @@ -84,6 +84,8 @@ public: AttributeMap attributes() const { return m_attributes; } AttributeMap &attributes() { return m_attributes; } + QString attribute(const QString &)const; + void setAttribute( const QString &attr, const QString &value ); void save( QTextStream &stream, uint indent = 0 ); XMLElement *namedItem( const QString &name ); diff --git a/libopie2/opiecore/xmltree.cc b/libopie2/opiecore/xmltree.cc index d5ce74a..3d03cc6 100644 --- a/libopie2/opiecore/xmltree.cc +++ b/libopie2/opiecore/xmltree.cc @@ -17,34 +17,13 @@ Boston, MA 02111-1307, USA. */ - +#include <qpe/stringutil.h> #include <opie/xmltree.h> #include <qxml.h> #include <assert.h> -namespace -{ - -/** - Encode an attribute value upon saving. - replaces '"' with """ - replaces '<' with "<" - replaces '&' with "&" - replaces '>' with ">" -*/ -QString encodeAttr( const QString& str ) -{ - QString encAttr( str ); // cause of the const parameter - encAttr = encAttr.replace( QRegExp( "[<]" ), "<" ); - encAttr = encAttr.replace( QRegExp( "[>]" ), ">" ); - encAttr = encAttr.replace( QRegExp( "[\"]" ), """ ); - encAttr = encAttr.replace( QRegExp( "[&]" ), "&" ); - return encAttr; -} - -} XMLElement::XMLElement() : m_parent( 0 ), m_next( 0 ), m_prev( 0 ), m_first( 0 ), m_last( 0 ) @@ -110,7 +89,18 @@ void XMLElement::insertAfter( XMLElement *newChild, XMLElement *refChild ) if ( next ) next->m_prev = newChild; } - +QString XMLElement::attribute(const QString &attr )const +{ + if ( !m_attributes.contains( attr ) ) + return QString::null; + AttributeMap::ConstIterator it = m_attributes.find( attr ); + return it.data(); +} +void XMLElement::setAttribute(const QString &attr, const QString &value ) +{ + m_attributes.remove( attr ); + m_attributes.insert( attr, value ); +} void XMLElement::insertBefore( XMLElement *newChild, XMLElement *refChild ) { assert( refChild ); @@ -163,7 +153,7 @@ void XMLElement::save( QTextStream &s, uint indent ) { if ( !m_value.isEmpty() ) { - s << encodeAttr( m_value ); + s << Qtopia::escapeString( m_value ); return; } @@ -179,7 +169,7 @@ void XMLElement::save( QTextStream &s, uint indent ) AttributeMap::ConstIterator end = m_attributes.end(); for (; it != end; ++it ) { - s << it.key() << "=\"" << encodeAttr( it.data() ) << "\""; + s << it.key() << "=\"" << Qtopia::escapeString( it.data() ) << "\""; s << " "; } } @@ -324,3 +314,5 @@ XMLElement *XMLElement::load( const QString &fileName ) return handler.root();; } + + diff --git a/libopie2/opiecore/xmltree.h b/libopie2/opiecore/xmltree.h index 7f1b7b7..7f50365 100644 --- a/libopie2/opiecore/xmltree.h +++ b/libopie2/opiecore/xmltree.h @@ -84,6 +84,8 @@ public: AttributeMap attributes() const { return m_attributes; } AttributeMap &attributes() { return m_attributes; } + QString attribute(const QString &)const; + void setAttribute( const QString &attr, const QString &value ); void save( QTextStream &stream, uint indent = 0 ); XMLElement *namedItem( const QString &name ); |