last update

Author: VitorVieiraZ

core/merginapi.cpp

@@ -1453,7 +1453,7 @@ bool MerginApi::validateAuth()
     return false;
   }
 
-  if ( mUserAuth->authToken().isEmpty() || mUserAuth->tokenExpiration() < QDateTime().currentDateTime().toUTC() )
+  if ( mUserAuth->authToken().isEmpty() || mUserAuth->tokenExpiration() < QDateTime().currentDateTimeUtc() )
   {
     authorize( mUserAuth->username(), mUserAuth->password() );
     CoreUtils::log( QStringLiteral( "MerginApi" ), QStringLiteral( "Requesting authorization because of missing or expired token." ) );
@@ -3966,12 +3966,10 @@ DownloadQueueItem::DownloadQueueItem( const QString &fp, qint64 s, int v, qint64
 
 void MerginApi::reloadProjectRole( const QString &projectFullName )
 {
-  if ( projectFullName.isEmpty() || !isLoggedIn() )
-  {
+  if ( projectFullName.isEmpty() )
     return;
-  }
 
-  QNetworkReply *reply = getProjectInfo( projectFullName );
+  QNetworkReply *reply = getProjectInfo( projectFullName, mUserAuth->isLoggedIn() ); //withAuth depends on whether user is logged in or not
   if ( !reply )
     return;
 
@@ -4053,8 +4051,3 @@ void MerginApi::setNetworkManager( QNetworkAccessManager *manager )
 
   emit networkManagerChanged();
 }
-
-bool MerginApi::isLoggedIn() const
-{
-  return mUserAuth->hasAuthData() && !mUserAuth->authToken().isEmpty() && !( mUserAuth->tokenExpiration() < QDateTime().currentDateTime().toUTC() );
-}

core/merginapi.h

@@ -593,11 +593,6 @@ class MerginApi: public QObject
      */
     void setNetworkManager( QNetworkAccessManager *manager );
 
-    /**
-     * Returns whether user is currently logged in
-    */
-    Q_INVOKABLE bool isLoggedIn() const;
-
   signals:
     void apiSupportsSubscriptionsChanged();
     void supportsSelectiveSyncChanged();

core/merginuserauth.cpp

@@ -134,3 +134,8 @@ bool MerginUserAuth::hasValidToken() const
 {
   return !mAuthToken.isEmpty() && mTokenExpiration >= QDateTime().currentDateTimeUtc();
 }
+
+bool MerginUserAuth::isLoggedIn()
+{
+  return hasAuthData() && hasValidToken();
+}

core/merginuserauth.h

@@ -38,6 +38,11 @@ class MerginUserAuth: public QObject
     //! i.e. we should be good to do authenticated requests.
     Q_INVOKABLE bool hasValidToken() const;
 
+    /**
+     * Returns whether user is currently logged in
+    */
+    Q_INVOKABLE bool isLoggedIn();
+
     void clear();
 
     QString username() const;