Sharing: change coding style of enum

From all upper case to camel case

This hopefully fix the Windows build which fails because DELETE seems
to be a macro

diff --git a/src/gui/ocssharejob.cpp b/src/gui/ocssharejob.cpp
index dea9b530cd4adc5450463dc4a2a708d53e79452b..ea0529bd24ca70c4c9bf6620d14bd7acd2945880 100644 (file)
--- a/src/gui/ocssharejob.cpp
+++ b/src/gui/ocssharejob.cpp
@@ -79,7 +79,7 @@ void OcsShareJob::setPublicUpload(int shareId, bool publicUpload)
     start();
 }
 
-void OcsShareJob::createShare(const QString &path, SHARETYPE shareType, const QString &password, const QDate &date)
+void OcsShareJob::createShare(const QString &path, ShareType shareType, const QString &password, const QDate &date)
 {
     setVerb("POST");
 

diff --git a/src/gui/ocssharejob.h b/src/gui/ocssharejob.h
index 1f16bf2516e7575114d1cdbe7f83d04d043d532a..87043c08e64e46134f8cb4cd6be381bdfb5ec7fd 100644 (file)
--- a/src/gui/ocssharejob.h
+++ b/src/gui/ocssharejob.h
@@ -35,20 +35,20 @@ public:
     /**
      * Support sharetypes
      */
-    enum class SHARETYPE : int {
-        LINK = 3
+    enum class ShareType : int {
+        Link = 3
     };
 
     /**
      * Possible permissions
      */
-    enum class PERMISSION : int {
-        READ = 1,
-        UPDATE = 2,
-        CREATE = 4,
-        DELETE = 8,
-        SHARE = 16,
-        ALL = 31
+    enum class Permission : int {
+        Read = 1,
+        Update = 2,
+        Create = 4,
+        Delete = 8,
+        Share = 16,
+        All = 31
     };
 
     /**
@@ -99,7 +99,7 @@ public:
      * @param password Optionally a password for the share
      * @param date Optionally an expire date for the share
      */
-    void createShare(const QString& path, SHARETYPE shareType, const QString& password = "", const QDate& date = QDate());
+    void createShare(const QString& path, ShareType shareType, const QString& password = "", const QDate& date = QDate());
 };
 
 }

diff --git a/src/gui/sharedialog.cpp b/src/gui/sharedialog.cpp
index 8c8dd57f495b5d023da92948923b683d81241998..e59888e8ed5d7a8bea135657c1901d908faa6a2b 100644 (file)
--- a/src/gui/sharedialog.cpp
+++ b/src/gui/sharedialog.cpp
@@ -248,7 +248,7 @@ void ShareDialog::setPassword(const QString &password)
             date = _ui->calendar->date();
         }
 
-        job->createShare(_sharePath, OcsShareJob::SHARETYPE::LINK, password, date);
+        job->createShare(_sharePath, OcsShareJob::ShareType::Link, password, date);
     }
     _passwordJobRunning = true;
 }
@@ -304,7 +304,7 @@ void ShareDialog::slotSharesFetched(const QVariantMap &reply)
     Q_FOREACH(auto share, ShareDialog::_shares) {
         QVariantMap data = share.toMap();
 
-        if (data.value("share_type").toInt() == static_cast<int>(OcsShareJob::SHARETYPE::LINK)) {
+        if (data.value("share_type").toInt() == static_cast<int>(OcsShareJob::ShareType::Link)) {
             _public_share_id = data.value("id").toULongLong();
             _ui->pushButton_copy->show();
 
@@ -340,8 +340,8 @@ void ShareDialog::slotSharesFetched(const QVariantMap &reply)
                  * For public links the server sets CREATE and UPDATE permissions.
                  */
                 if (!_isFile && 
-                       (permissions & static_cast<int>(OcsShareJob::PERMISSION::UPDATE)) && 
-                       (permissions & static_cast<int>(OcsShareJob::PERMISSION::CREATE))) {
+                       (permissions & static_cast<int>(OcsShareJob::Permission::Update)) &&
+                       (permissions & static_cast<int>(OcsShareJob::Permission::Create))) {
                     _ui->checkBox_editing->setChecked(true);
                 }
             }
@@ -468,7 +468,7 @@ void ShareDialog::slotCheckBoxShareLinkClicked()
 
         OcsShareJob *job = new OcsShareJob(_account, this);
         connect(job, SIGNAL(jobFinished(QVariantMap)), this, SLOT(slotCreateShareFetched(QVariantMap)));
-        job->createShare(_sharePath, OcsShareJob::SHARETYPE::LINK);
+        job->createShare(_sharePath, OcsShareJob::ShareType::Link);
     } else {
         _pi_link->startAnimation();
         OcsShareJob *job = new OcsShareJob(_account, this);