updates as per code review

Crunch-io · Feb 5, 2025 · c69785e · c69785e
1 parent 47edc6a
commit c69785e
Showing 1 changed file with 7 additions and 6 deletions.
diff --git a/scrunch/datasets.py b/scrunch/datasets.py
@@ -2380,7 +2380,7 @@ def fork(self, description=None, name=None, is_published=False,
         :returns _fork: scrunch.datasets.BaseDataset
         """
         from scrunch.mutable_dataset import MutableDataset
-        
+
         # Handling project vs owner conflict
         owner = kwargs.get("owner")
 
@@ -2426,14 +2426,15 @@ def fork(self, description=None, name=None, is_published=False,
             if project:
                 # Create fork in given Project path.
                 try:
-                    project = get_project(project).url
+                    project_url = get_project(project).url
                 except KeyError:
                     # Creating full project URL for sub-folders
-                    connection = _default_connection(connection=None)
-                    site_url = connection.session.site_url
-                    project = site_url + "/projects/{}/".format(project)
+                    project_url = "{}/projects/{}/".format(
+                        self.resource.session.site_url,
+                        project
+                    )
                 finally:
-                    body["project"] = project
+                    body["project"] = project_url
             else:
                 raise ValueError(
                     "Project parameter should be provided when preserve_owner=False."