# HG changeset patch
# User Vincent Hatakeyama <vincent.hatakeyama@xcg-consulting.fr>
# Date 1680190662 -7200
#      Thu Mar 30 17:37:42 2023 +0200
# Node ID 95175b1d6eed1758c9aa69bdb10f62be5af0f1c5
# Parent  9e4d7f6b9c91e42f22bac486e5fc2e1d811b1c83
📝 move common method in docker_client

diff --git a/odoo_scripts/docker_build_doc.py b/odoo_scripts/docker_build_doc.py
--- a/odoo_scripts/docker_build_doc.py
+++ b/odoo_scripts/docker_build_doc.py
@@ -9,8 +9,7 @@
 
 from .config import Config
 from .docker_build import add_build_options, build_local_image, get_build_options
-from .docker_client import DockerClient
-from .docker_prettier import get_volumes
+from .docker_client import DockerClient, get_volumes
 from .parsing import apply, basic_parser
 
 __version__ = "1.0.1"
diff --git a/odoo_scripts/docker_client.py b/odoo_scripts/docker_client.py
--- a/odoo_scripts/docker_client.py
+++ b/odoo_scripts/docker_client.py
@@ -486,3 +486,23 @@
             return True
         result = self.docker_cli.containers.list(filters={"id": self._container.id})
         return result and result[0].status == "exited"
+
+
+def get_volumes(base_dir, directory: str, write: bool = False):
+    """return volumes to bind"""
+    path = os.path.abspath(os.path.join(base_dir, directory))
+    volumes = {
+        base_dir: {
+            "bind": base_dir,
+            "mode": "rw" if write else "ro",
+        }
+    }
+    # also bind any symbolic link, but only for the given directory or its parent.
+    # all symbolic link could be scanned for but that might take too much time
+    for potential_link in (path, os.path.dirname(path)):
+        if os.path.islink(potential_link):
+            volumes[potential_link] = {
+                "bind": os.path.realpath(potential_link),
+                "mode": "rw" if write else "ro",
+            }
+    return volumes, path
diff --git a/odoo_scripts/docker_isort.py b/odoo_scripts/docker_isort.py
--- a/odoo_scripts/docker_isort.py
+++ b/odoo_scripts/docker_isort.py
@@ -9,8 +9,7 @@
 
 from .config import ODOO_7, ODOO_8, ODOO_9, ODOO_10, ODOO_11, ODOO_12, ODOO_13, Config
 from .docker_build import add_build_options, build_local_image, get_build_options
-from .docker_client import DockerClient
-from .docker_prettier import get_volumes
+from .docker_client import DockerClient, get_volumes
 from .parsing import apply, basic_parser
 
 __version__ = "3.0.2"
diff --git a/odoo_scripts/docker_prettier.py b/odoo_scripts/docker_prettier.py
--- a/odoo_scripts/docker_prettier.py
+++ b/odoo_scripts/docker_prettier.py
@@ -8,7 +8,7 @@
 
 from .config import ODOO_7, ODOO_8, ODOO_9, ODOO_10, Config
 from .docker_build import add_build_options, build_local_image, get_build_options
-from .docker_client import DockerClient
+from .docker_client import DockerClient, get_volumes
 from .parsing import apply, basic_parser
 
 __version__ = "1.1.1"
@@ -104,25 +104,5 @@
     )
 
 
-def get_volumes(base_dir, directory: str, write: bool = False):
-    """return volumes to bind"""
-    path = os.path.abspath(os.path.join(base_dir, directory))
-    volumes = {
-        base_dir: {
-            "bind": base_dir,
-            "mode": "rw" if write else "ro",
-        }
-    }
-    # also bind any symbolic link, but only for the given directory or its parent.
-    # all symbolic link could be scanned for but that might take too much time
-    for potential_link in (path, os.path.dirname(path)):
-        if os.path.islink(potential_link):
-            volumes[potential_link] = {
-                "bind": os.path.realpath(potential_link),
-                "mode": "rw" if write else "ro",
-            }
-    return volumes, path
-
-
 if __name__ == "__main__":
     sys.exit(main())