WIP add spinner and raise questions

Author: julien-lang

python/tank/authentication/login_dialog.py

@@ -337,6 +337,19 @@ def __init__(
 
         self.confirm_box.setStyleSheet(self.styleSheet())
 
+        # Init UI Spinner
+        self.ui.spinner_movie = QtGui.QMovie(":/spinning_wheel.gif")
+        if self.ui.spinner_movie.loopCount() != -1:
+            self.ui.spinner_movie.finished.connect(self.ui.spinner_movie.start)
+            # TODO get a better GIF that loop forever
+
+        self.ui.refresh_site_info_spinner.setMovie(self.ui.spinner_movie)
+
+        # TODO: Init which widget are display by default
+        # IF not host at all in the recent box: nothing but the site widget
+        # IF host exists: spinner mode
+
+
     def __del__(self):
         """
         Destructor.
@@ -399,6 +412,16 @@ def _update_ui_according_to_site_support(self):
         TODO the name and description of this method are not accurate!!!!!
         """
 
+        self.ui.login.setVisible(False)
+        self.ui.password.setVisible(False)
+        self.ui.message.setVisible(False)
+        self.ui.forgot_password_link.setVisible(False)
+        self.ui.button_options.setVisible(False)
+        self.ui.sign_in.setVisible(False) # maybe setEnable instead?
+
+        self.ui.refresh_site_info_spinner.setVisible(True)
+        self.ui.refresh_site_info_label.setVisible(True)
+
         logger.debug("_update_ui_according_to_site_support")
         self._query_task.url_to_test = self._get_current_site()
         self._query_task.start()
@@ -558,6 +581,11 @@ def _toggle_web(self, method_selected=None):
 
         self.method_selected = method_selected
 
+        self.ui.refresh_site_info_spinner.setVisible(False)
+        self.ui.refresh_site_info_label.setVisible(False)
+        self.ui.message.setVisible(True)
+        self.ui.sign_in.setVisible(True)
+
         # if we are switching from one mode (using the web) to another (not using
         # the web), or vice-versa, we need to update the GUI.
         # In web-based authentication, the web form is in charge of obtaining
@@ -701,13 +729,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