WIP add spinner and raise questions

Author: julien-lang

python/tank/authentication/login_dialog.py

@@ -337,6 +337,17 @@ def __init__(
 
         self.confirm_box.setStyleSheet(self.styleSheet())
 
+        # Init UI Spinner
+        self.ui.message.setText("Retrieving the site information...")
+        self.ui.spinner_movie = QtGui.QMovie(
+            ":/spinning_wheel/spinning_wheel.gif"
+        )
+        if self.ui.spinner_movie.loopCount() != -1:
+            self.ui.spinner_movie.finished.connect(self.ui.spinner_movie.start)
+
+        self.ui.spinner.setMovie(self.ui.spinner_movie)
+
+
     def __del__(self):
         """
         Destructor.
@@ -472,6 +483,7 @@ def _link_activated(self, site=None):
             )
 
     def _toggle_web(self, method_selected=None):
+        return
         """
         Sets up the dialog GUI according to the use of web login or not.
         """
@@ -701,13 +713,18 @@ def result(self):
 
             return self._sso_saml2.get_session_data()
 
-        # We want to wait until we know what is supported by the site, to avoid
-        # flickering GUI.
-        if not self._query_task.wait(THREAD_WAIT_TIMEOUT_MS):
-            logger.warning(
-                "Timed out awaiting requesting information: %s"
-                % self._get_current_site()
-            )
+        # # We want to wait until we know what is supported by the site, to avoid
+        # # flickering GUI.
+        # if not self._query_task.wait(THREAD_WAIT_TIMEOUT_MS):
+        #     logger.warning(
+        #         "Timed out awaiting requesting information: %s"
+        #         % self._get_current_site()
+        #     )
+
+        # Configure the GUI according to what we know: site and user preferences
+        # TODO ->        self._toggle_web....
+
+        self.ui.spinner_movie.start()
 
         res = self.exec_()
         if res != QtGui.QDialog.Accepted:

python/tank/authentication/site_info.py

@@ -40,6 +40,13 @@ def _get_site_infos(url, http_proxy=None):
     :returns:   A dictionary with the site infos.
     """
 
+    time.sleep(3)
+    return {
+        "user_authentication_method": "oxygen",
+        "unified_login_flow_enabled": True,
+        "authentication_app_session_launcher_enabled": True,
+    }
+
     # logger.info("Sleep for 10s")
     # time.sleep(30)
     # logger.info("done sleeping")
@@ -115,6 +122,14 @@ def reload(self, url, http_proxy=None):
             logger.debug("Unable to connect with %s, got exception '%s'", url, exc)
             return
 
+
+        # TODO emit a signal with the infos dict instead of waiting for the comsumer to retrieve it.
+        # Because the thread might already run with different URL at that point....
+
+
+        # ALSO, the following logs should only run if needed
+        # ALSO, why don't we consume the cache here ?!
+
         self._url = url
         self._infos = infos