From 64483c4488c9184705ef08ae59e994a0d706cec1 Mon Sep 17 00:00:00 2001
From: alessandro ceglie <sauzher@gmail.com>
Date: Thu, 16 Feb 2023 22:40:12 +0100
Subject: [PATCH 1/4] Python3 fixes

---
 collective/embedly/browser/controlpanel.py |  9 ++++-----
 collective/embedly/setuphandlers.py        |  4 ++--
 collective/embedly/transform.py            | 16 ++++++++--------
 docs/HISTORY.rst                           |  2 ++
 4 files changed, 16 insertions(+), 15 deletions(-)

diff --git a/collective/embedly/browser/controlpanel.py b/collective/embedly/browser/controlpanel.py
index 3f532f6..6d00d06 100644
--- a/collective/embedly/browser/controlpanel.py
+++ b/collective/embedly/browser/controlpanel.py
@@ -1,5 +1,5 @@
-from zope.interface import implements
-from zope.component import adapts
+from zope.interface import implementer
+from zope.component import adapter
 from zope.component import getUtility
 from zope.formlib import form
 
@@ -13,11 +13,10 @@
 from collective.embedly.interfaces import IEmbedlySettings
 
 
+@implementer(IEmbedlySettings)
+@adapter(IPloneSiteRoot)
 class EmbedlyControlPanelAdapter(SchemaAdapterBase):
 
-    adapts(IPloneSiteRoot)
-    implements(IEmbedlySettings)
-
     def __init__(self, context):
         super(EmbedlyControlPanelAdapter, self).__init__(context)
         registry = getUtility(IRegistry)
diff --git a/collective/embedly/setuphandlers.py b/collective/embedly/setuphandlers.py
index f2d49cb..3d53cf5 100644
--- a/collective/embedly/setuphandlers.py
+++ b/collective/embedly/setuphandlers.py
@@ -1,4 +1,4 @@
-from zope.interface import implements
+from zope.interface import implementer
 from Products.CMFCore.utils import getToolByName
 from Products.CMFPlone.interfaces import INonInstallable
 from zope.component import getUtility
@@ -16,8 +16,8 @@
 })
 
 
+#implementer(INonInstallable)
 class HiddenProfiles(object):
-    implements(INonInstallable)
 
     def getNonInstallableProfiles(self):
         """
diff --git a/collective/embedly/transform.py b/collective/embedly/transform.py
index bc1abae..b9ede1f 100644
--- a/collective/embedly/transform.py
+++ b/collective/embedly/transform.py
@@ -1,17 +1,17 @@
-import urllib2
+import urllib
 import json
 import re
 import logging
 from Products.CMFCore.utils import getToolByName
 from Products.CMFPlone.utils import safe_unicode
 from Products.PortalTransforms.interfaces import ITransform
-from zope.interface import implements
+from zope.interface import implementer
 from zope.component import getUtility, queryUtility
 from plone.registry.interfaces import IRegistry
 from plone.memoize import ram
 from plone.memoize.interfaces import ICacheChooser
 from time import time
-from urlparse import urlparse, parse_qsl
+from urllib.parse import urlparse, parse_qsl
 try:
     from zope.app.component.hooks import getSite
 except ImportError:
@@ -48,7 +48,7 @@ def clear_cache():
 
 
 def update_services():
-    resp = urllib2.urlopen('http://api.embed.ly/1/services/python')
+    resp = urllib.request.urlopen('http://api.embed.ly/1/services/python')
     if resp.getcode() == 200:
         list_exp = []
         for service in json.loads(resp.read()):
@@ -152,12 +152,12 @@ def get_oembed(url, api_key=None, doc=None):
     if embedly_query:
         embedly_query = '&' + embedly_query
     fetch_url = 'http://api.embed.ly/1/oembed?%surl=%s%s&format=json' % \
-        (api_key_string, urllib2.quote(curl.encode('utf-8')), embedly_query)
+        (api_key_string, urllib.parse.quote(curl.encode('utf-8')), embedly_query)
     logger.debug("HREF:%s URL:%s" % (url, fetch_url))
     try:
-        result = urllib2.urlopen(fetch_url).read()
+        result = urllib.request.urlopen(fetch_url).read()
         logger.debug("Response: %s" % result)
-    except urllib2.HTTPError, e:
+    except urllib.HTTPError as e:
         if doc:
             logger.error("Unexpected response from embedly API (%d: %s) while processing %s in %s" % (e.code, e.msg, url, doc))
         else:
@@ -202,8 +202,8 @@ def replace(matchobj):
     return link_url_re.sub(replace, text)
 
 
+@implementer(ITransform)
 class EmbedlyTransform:
-    implements(ITransform)
 
     __name__ = "embedly_transform"
 
diff --git a/docs/HISTORY.rst b/docs/HISTORY.rst
index 2c4ff00..e1ac732 100644
--- a/docs/HISTORY.rst
+++ b/docs/HISTORY.rst
@@ -3,6 +3,8 @@ Changelog
 
 dev
 ---
+- Py3 compatibility
+  [sauzher]
 
 - Don't attempt to decode unicode strings in transform.
   [alecm]

From 8c278c83935da3904cdb41e242fcc44982054764 Mon Sep 17 00:00:00 2001
From: Alessandro Ceglie <sauzher@gmail.com>
Date: Mon, 20 Feb 2023 11:00:21 +0100
Subject: [PATCH 2/4] typo

---
 collective/embedly/setuphandlers.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/collective/embedly/setuphandlers.py b/collective/embedly/setuphandlers.py
index 3d53cf5..653878c 100644
--- a/collective/embedly/setuphandlers.py
+++ b/collective/embedly/setuphandlers.py
@@ -16,7 +16,7 @@
 })
 
 
-#implementer(INonInstallable)
+@implementer(INonInstallable)
 class HiddenProfiles(object):
 
     def getNonInstallableProfiles(self):

From c8f3f1051662b84127b2719328492018483efa58 Mon Sep 17 00:00:00 2001
From: "alessandro.ceglie" <sauzher@gmail.com>
Date: Mon, 20 Feb 2023 12:05:00 +0100
Subject: [PATCH 3/4] backward compatibility for Py2 refactoring Controlpanel
 with  p.a.registry RegistryEditForm wrapper (z3c.form)

---
 collective/embedly/browser/configure.zcml  |  4 --
 collective/embedly/browser/controlpanel.py | 71 ++++------------------
 collective/embedly/tests/patch.py          | 24 +++++---
 collective/embedly/tests/tests.py          |  8 ++-
 collective/embedly/transform.py            | 17 ++++--
 5 files changed, 44 insertions(+), 80 deletions(-)

diff --git a/collective/embedly/browser/configure.zcml b/collective/embedly/browser/configure.zcml
index e0eb8de..8b27360 100644
--- a/collective/embedly/browser/configure.zcml
+++ b/collective/embedly/browser/configure.zcml
@@ -4,9 +4,6 @@
     xmlns:zcml="http://namespaces.zope.org/zcml"
     i18n_domain="collective.embedly">
 
-  <adapter factory=".controlpanel.EmbedlyControlPanelAdapter"
-           zcml:condition="installed Products.CMFDefault.formlib" />
-
   <browser:resource
     name="embedly.css"
     file="embedly.css"
@@ -28,7 +25,6 @@
       for="Products.CMFPlone.interfaces.IPloneSiteRoot"
       class=".controlpanel.EmbedlyControlPanel"
       permission="cmf.ManagePortal"
-      zcml:condition="installed Products.CMFDefault.formlib"
       />
 
   <browser:resourceDirectory
diff --git a/collective/embedly/browser/controlpanel.py b/collective/embedly/browser/controlpanel.py
index 6d00d06..ae46401 100644
--- a/collective/embedly/browser/controlpanel.py
+++ b/collective/embedly/browser/controlpanel.py
@@ -1,66 +1,17 @@
-from zope.interface import implementer
-from zope.component import adapter
-from zope.component import getUtility
-from zope.formlib import form
-
-from Products.CMFDefault.formlib.schema import SchemaAdapterBase
-from Products.CMFPlone.interfaces import IPloneSiteRoot
-
-from plone.registry.interfaces import IRegistry
-from plone.app.controlpanel.form import ControlPanelForm
-
 from collective.embedly import embedlyMessageFactory as _
 from collective.embedly.interfaces import IEmbedlySettings
+from plone.app.registry.browser.controlpanel import ControlPanelFormWrapper
+from plone.app.registry.browser.controlpanel import RegistryEditForm
+from plone.registry.interfaces import IRegistry
+from plone.z3cform import layout
+from zope.component import getUtility
 
 
-@implementer(IEmbedlySettings)
-@adapter(IPloneSiteRoot)
-class EmbedlyControlPanelAdapter(SchemaAdapterBase):
-
-    def __init__(self, context):
-        super(EmbedlyControlPanelAdapter, self).__init__(context)
-        registry = getUtility(IRegistry)
-        self.settings = registry.forInterface(IEmbedlySettings, False)
-
-    def getAPIKey(self):
-        return self.settings.api_key
-
-    def setAPIKey(self, value):
-        self.settings.api_key = value
-
-    def getUseServicesRegexp(self):
-        return self.settings.use_services_regexp
-
-    def setUseServicesRegexp(self, value):
-        self.settings.use_services_regexp = value
-
-    def getPersistentCache(self):
-        return self.settings.persistent_cache
-
-    def setPersistentCache(self, value):
-        self.settings.persistent_cache = value
-
-    def getCacheTimeout(self):
-        return self.settings.cache_timeout
-
-    def setCacheTimeout(self, value):
-        self.settings.cache_timeout = value
-
-    api_key = property(getAPIKey,
-                       setAPIKey)
-
-    use_services_regexp = property(getUseServicesRegexp,
-                                   setUseServicesRegexp)
-
-    persistent_cache = property(getPersistentCache,
-                                setPersistentCache)
-
-    cache_timeout = property(getCacheTimeout,
-                             setCacheTimeout)
-
-
-class EmbedlyControlPanel(ControlPanelForm):
-
+class EmbedlyControlPanelForm(RegistryEditForm):
     label = _("Embedly settings")
     description = _("Lets you change the settings of collective.embedly")
-    form_fields = form.FormFields(IEmbedlySettings)
+    schema = IEmbedlySettings
+
+
+EmbedlyControlPanel = layout.wrap_form(
+    EmbedlyControlPanelForm, ControlPanelFormWrapper)
\ No newline at end of file
diff --git a/collective/embedly/tests/patch.py b/collective/embedly/tests/patch.py
index c6c33bf..3d20691 100644
--- a/collective/embedly/tests/patch.py
+++ b/collective/embedly/tests/patch.py
@@ -1,8 +1,16 @@
-import urllib2
 import re
-from urlparse import urlparse, parse_qsl
 from collective.embedly.transform import get_services_regexp
 
+try:
+    from urllib.request import urlopen
+    from urllib.parse import urlparse, parse_qsl
+    from urllib.error import HTTPError
+except ImportError:
+    from urllib2 import urlopen
+    from urlparse import urlparse, parse_qsl
+    from urllib2 import HTTPError
+
+
 
 embedly_data = '{"provider_url": "http://www.youtube.com/", "description": "HAYDAMAKY message", "title": "HAYDAMAKY message", "url": "http://www.youtube.com/watch?v=L1NPLlhFTVk", "author_name": "eastblok", "height": 360, "width": 640, "html": "<object width=\\"640\\" height=\\"360\\"><param name=\\"movie\\" value=\\"http://www.youtube.com/v/L1NPLlhFTVk?version=3\\"><param name=\\"allowFullScreen\\" value=\\"true\\"><param name=\\"allowscriptaccess\\" value=\\"always\\"><embed src=\\"http://www.youtube.com/v/L1NPLlhFTVk?version=3\\" type=\\"application/x-shockwave-flash\\" width=\\"640\\" height=\\"360\\" allowscriptaccess=\\"always\\" allowfullscreen=\\"true\\"></embed></object>", "thumbnail_width": 480, "version": "1.0", "provider_name": "YouTube", "thumbnail_url": "http://i1.ytimg.com/vi/L1NPLlhFTVk/hqdefault.jpg", "type": "video", "thumbnail_height": 360, "author_url": "http://www.youtube.com/user/eastblok"}'
 
@@ -39,8 +47,8 @@ def read(self):
                 service_url = p[1]
 
         if key is not None and len(key) != 32:
-            from StringIO import StringIO
-            raise urllib2.HTTPError(self.url, 401, 'Unauthorized', None,
+            from six import StringIO
+            raise HTTPError(self.url, 401, 'Unauthorized', None,
                                     StringIO('Invalid key or oauth_consumer_key provided: %s' % key))
         if re.match(get_services_regexp(), service_url):
             if service_url.endswith('.jpg'):
@@ -53,14 +61,14 @@ def read(self):
     def close(self):
         pass
 
-original_urlopen = urllib2.urlopen
+original_urlopen = urlopen
 
 
 def patch_urlopen():
     global original_urlopen
-    original_urlopen = urllib2.urlopen
-    urllib2.urlopen = dummy_urlopen
+    original_urlopen = urlopen
+    urlopen = dummy_urlopen
 
 
 def unpatch_urlopen():
-    urllib2.urlopen = original_urlopen
+    urlopen = original_urlopen
diff --git a/collective/embedly/tests/tests.py b/collective/embedly/tests/tests.py
index fd8f836..7a2bad3 100644
--- a/collective/embedly/tests/tests.py
+++ b/collective/embedly/tests/tests.py
@@ -1,4 +1,3 @@
-import urllib2
 import json
 import robotsuite
 import unittest2 as unittest
@@ -24,6 +23,11 @@
 from collective.embedly.tests.layer import EMBEDLY_ACCEPTANCE_TESTING
 from collective.embedly.tests.patch import json_result
 
+try:
+    from urllib.request import urlopen
+except ImportError:
+    from urllib import urlopen
+
 
 embedly_filter = lambda i: i['path'] == 'collective.embedly.transform.get_oembed'
 
@@ -41,7 +45,7 @@ def test_store_services(self):
         view = portal.unrestrictedTraverse("@@update_embedly_services")
         view.__of__(portal)()
 
-        res = urllib2.urlopen('http://api.embed.ly/1/services/python')
+        res = urlopen('http://api.embed.ly/1/services/python')
         list_exp = []
         for service in json.loads(res.read()):
             list_exp.append('|'.join(service.get('regex', [])))
diff --git a/collective/embedly/transform.py b/collective/embedly/transform.py
index b9ede1f..045878b 100644
--- a/collective/embedly/transform.py
+++ b/collective/embedly/transform.py
@@ -1,4 +1,3 @@
-import urllib
 import json
 import re
 import logging
@@ -11,7 +10,13 @@
 from plone.memoize import ram
 from plone.memoize.interfaces import ICacheChooser
 from time import time
-from urllib.parse import urlparse, parse_qsl
+try:
+    from urllib.request import urlopen
+    from urllib.error import HTTPError
+    from urllib.parse import urlparse, parse_qsl, quote
+except ImportError:
+    from urlparse import urlparse, parse_qsl
+    from urllib2 import urlopen, HTTPError, quote
 try:
     from zope.app.component.hooks import getSite
 except ImportError:
@@ -48,7 +53,7 @@ def clear_cache():
 
 
 def update_services():
-    resp = urllib.request.urlopen('http://api.embed.ly/1/services/python')
+    resp = urlopen('http://api.embed.ly/1/services/python')
     if resp.getcode() == 200:
         list_exp = []
         for service in json.loads(resp.read()):
@@ -152,12 +157,12 @@ def get_oembed(url, api_key=None, doc=None):
     if embedly_query:
         embedly_query = '&' + embedly_query
     fetch_url = 'http://api.embed.ly/1/oembed?%surl=%s%s&format=json' % \
-        (api_key_string, urllib.parse.quote(curl.encode('utf-8')), embedly_query)
+        (api_key_string, quote(curl.encode('utf-8')), embedly_query)
     logger.debug("HREF:%s URL:%s" % (url, fetch_url))
     try:
-        result = urllib.request.urlopen(fetch_url).read()
+        result = urlopen(fetch_url).read()
         logger.debug("Response: %s" % result)
-    except urllib.HTTPError as e:
+    except HTTPError as e:
         if doc:
             logger.error("Unexpected response from embedly API (%d: %s) while processing %s in %s" % (e.code, e.msg, url, doc))
         else:

From 43ca054f3b0bc017c486a0d485a32eb752c0f049 Mon Sep 17 00:00:00 2001
From: Alessandro Ceglie <sauzher@gmail.com>
Date: Mon, 20 Feb 2023 13:43:20 +0100
Subject: [PATCH 4/4] typo

---
 collective/embedly/tests/tests.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/collective/embedly/tests/tests.py b/collective/embedly/tests/tests.py
index 7a2bad3..bdbb23e 100644
--- a/collective/embedly/tests/tests.py
+++ b/collective/embedly/tests/tests.py
@@ -26,7 +26,7 @@
 try:
     from urllib.request import urlopen
 except ImportError:
-    from urllib import urlopen
+    from urllib2 import urlopen
 
 
 embedly_filter = lambda i: i['path'] == 'collective.embedly.transform.get_oembed'