diff --git a/src/gui/sharedialog.cpp b/src/gui/sharedialog.cpp
index 2d3daf9a7c..d4d645510c 100644
--- a/src/gui/sharedialog.cpp
+++ b/src/gui/sharedialog.cpp
@@ -65,7 +65,7 @@ void ShareDialog::setExpireDate(const QString &date)
     job->start();
 }
 
-void ShareDialog::slotExpireSet(const QString &reply)
+void ShareDialog::slotExpireSet(const QString & /* reply */)
 {
     _ui->labelCalendarSpinner->hide();
 }
@@ -114,7 +114,7 @@ void ShareDialog::setPassword(QString password)
     job->start();
 }
 
-void ShareDialog::slotPasswordSet(const QString &reply)
+void ShareDialog::slotPasswordSet(const QString & /* reply */)
 {
     _ui->labelPasswordSpinner->hide();
 }
@@ -172,7 +172,7 @@ void ShareDialog::slotSharesFetched(const QString &reply)
     }
 }
 
-void ShareDialog::slotDeleteShareFetched(const QString &reply)
+void ShareDialog::slotDeleteShareFetched(const QString & /* reply */)
 {
     _ui->labelShareSpinner->hide();
     _ui->lineEdit_shareLink->hide();
@@ -260,7 +260,7 @@ void ShareDialog::slotCheckBoxExpireClicked()
     }
 }
 
-OcsShareJob::OcsShareJob(const QByteArray &verb, const QUrl &url, const QUrl &postData, Account* account, QObject* parent)
+OcsShareJob::OcsShareJob(const QByteArray &verb, const QUrl url, const QUrl postData, AccountPtr account, QObject* parent)
 : AbstractNetworkJob(account, "", parent),
   _verb(verb),
   _url(url),
diff --git a/src/gui/sharedialog.h b/src/gui/sharedialog.h
index 2467f246c4..91344ab2bf 100644
--- a/src/gui/sharedialog.h
+++ b/src/gui/sharedialog.h
@@ -9,7 +9,7 @@ namespace OCC {
 class OcsShareJob : public AbstractNetworkJob {
     Q_OBJECT
 public:
-    explicit OcsShareJob(const QByteArray &verb, const QUrl &url, const QUrl &postData, Account *account, QObject* parent = 0);
+    explicit OcsShareJob(const QByteArray &verb, const QUrl url, const QUrl postData, AccountPtr account, QObject* parent = 0);
 public slots:
     void start() Q_DECL_OVERRIDE;
 signals: