Add support for extraction + matching on website

PEP8 fix

8.0 Add support for partner bank matching on invoice update (#6)

Add support for partner bank matching on invoice update (before, it was only supported on invoice creation)

8.0 Use try/except when importing external libs (#8)

[FIX] LINT

Use try/except when importing external libs
Remove self.ensure_one() that has nothing to do in an api.model method

Author: alexis-via

base_ubl/README.rst

@@ -24,13 +24,13 @@ Usage
 
 .. image:: https://odoo-community.org/website/image/ir.attachment/5784_f2813bd/datas
    :alt: Try me on Runbot
-   :target: https://runbot.odoo-community.org/runbot/95/8.0
+   :target: https://runbot.odoo-community.org/runbot/226/8.0
 
 Bug Tracker
 ===========
 
 Bugs are tracked on `GitHub Issues
-<https://github.com/OCA/account-invoicing/issues>`_. In case of trouble, please
+<https://github.com/OCA/edi/issues>`_. In case of trouble, please
 check there if your issue has already been reported. If you spotted it first,
 help us smashing it by providing a detailed and welcomed feedback.
 

base_ubl/models/ubl.py

@@ -8,12 +8,15 @@
 from lxml import etree
 from StringIO import StringIO
 from tempfile import NamedTemporaryFile
-import PyPDF2
 import mimetypes
 import logging
-
 logger = logging.getLogger(__name__)
 
+try:
+    import PyPDF2
+except ImportError:
+    logger.debug('Cannot import PyPDF2')
+
 
 class BaseUbl(models.AbstractModel):
     _name = 'base.ubl'
@@ -459,10 +462,13 @@ def ubl_parse_party(self, party_node, ns):
             'cac:Contact/cbc:Telephone', namespaces=ns)
         fax_xpath = party_node.xpath(
             'cac:Contact/cbc:Telefax', namespaces=ns)
+        website_xpath = party_node.xpath(
+            'cbc:WebsiteURI', namespaces=ns)
         partner_dict = {
             'vat': vat_xpath and vat_xpath[0].text or False,
             'name': partner_name_xpath[0].text,
             'email': email_xpath and email_xpath[0].text or False,
+            'website': website_xpath and website_xpath[0].text or False,
             'phone': phone_xpath and phone_xpath[0].text or False,
             'fax': fax_xpath and fax_xpath[0].text or False,
             }
@@ -554,7 +560,7 @@ def get_xml_files_from_pdf(self, pdf_file):
             xmlfiles = {}  # key = filename, value = PDF obj
             for embeddedfile in embeddedfiles[:-1]:
                 mime_res = mimetypes.guess_type(embeddedfile)
-                if mime_res and mime_res[0] in ['application/xml','text/xml']:
+                if mime_res and mime_res[0] in ['application/xml', 'text/xml']:
                     xmlfiles[embeddedfile] = embeddedfiles[i+1]
                 i += 1
             logger.debug('xmlfiles=%s', xmlfiles)