Merge pull request #26 from mluis7/pxx-9

pxx-9 upload version 0.3.3 to PyPI

Author: mluis7

README.md

@@ -2,13 +2,11 @@
 # pyxml2xpath
 Parse an XML document with [lxml](https://lxml.de/) and build XPath expressions corresponding to its structure.
 
-<h3> &#x24D8; Project status: BETA (RC2) </h3>
-
 Table of contents
 =================
 
 * [Basic usage](#description)
-* [Build and install](#build-and-install)
+* [Installation](#installation)
 * [Command line usage](#command-line-usage)
 * [Module usage](#module-usage)
 * [Method parse(...)](#method-parse)
@@ -43,9 +41,15 @@ text node types can be used in predicates but not on path
 
 It can be used as a [command line utility](#command-line-usage) or as a [module](#module-usage).
 
-A spin off of [xml2xpath Bash script](https://github.com/mluis7/xml2xpath). Both projects rely on [libxml2](https://gitlab.gnome.org/GNOME/libxml2/-/wikis/home) implementation.
+> A spin off of [xml2xpath Bash script](https://github.com/mluis7/xml2xpath). Both projects rely on [libxml2](https://gitlab.gnome.org/GNOME/libxml2/-/wikis/home) implementation.
+
+## Installation
+Installing from PyPi
+
+`pip3.9 install pyxml2xpath`
+
+Or building from source repo
 
-## Build and install
 ```bash
 git clone https://github.com/mluis7/pyxml2xpath.git
 cd pyxml2xpath
@@ -59,8 +63,6 @@ Alternative without cloning the repo yourself
 pip3.9 install git+https://github.com/mluis7/pyxml2xpath.git
 ```
 
-Soon on PyPi!
-
 ## Command line usage
 `pyxml2xpath <file path> [mode] [initial xpath expression] [with count] [max elements] [without banners]`
 

VERSION

@@ -1 +1 @@
-0.3.2rc2
+0.3.3

pyproject.toml

@@ -19,7 +19,7 @@ readme = "README.md"
 license = {file = "LICENSE"}
 keywords = ["xpath", "xml"]
 classifiers = [
-  "Development Status :: 4 - Beta",
+  "Development Status :: 5 - Production/Stable",
   "Programming Language :: Python"
 ]
 

src/xml2xpath/xml2xpath.py

@@ -125,7 +125,7 @@ def parse_mixed_ns(tree: etree._ElementTree,
     xpath_base: str
         Xpath expression to start from
     with_count: bool
-        add count of found elements (performance cost on large documents.
+        add count of found elements (performance cost on large documents).
     max_items: int
         max number of elements to parse. Default: 100000'''
 
@@ -162,12 +162,12 @@ def parse_mixed_ns(tree: etree._ElementTree,
             # parent may exist even if xpath_base is a relative path: //soapenv:Body
             prnt = ele.getparent()
             if prnt is not None:
-                pqname = etree.QName(prnt.tag)
-                # parent's (unqualified) xpath
-                xpp = tree.getpath(prnt)
                 # type(ele): etree._Element
                 if type(ele.tag) is str:
                     qname = etree.QName(ele.tag)
+                    pqname = etree.QName(prnt.tag)
+                    # parent's (unqualified) xpath
+                    xpp = tree.getpath(prnt)
                     # parent of current element was already parsed so
                     # just append current qualified name
                     if xpp in xmap: