Convert hard coded shortcuts to standard keys

[dolphin.git] / src / views / dolphinremoteencoding.cpp

diff --git a/src/views/dolphinremoteencoding.cpp b/src/views/dolphinremoteencoding.cpp
index 397eaccc098babbe5819ca3b81c78ce345b93af0..269cc3085ffd03d8732dcbb23ea9da049c8fd752 100644 (file)
--- a/src/views/dolphinremoteencoding.cpp
+++ b/src/views/dolphinremoteencoding.cpp
@@ -16,44 +16,42 @@
  *   Free Software Foundation, Inc.,                                       *
  *   51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA            *
  ***************************************************************************/
  *   Free Software Foundation, Inc.,                                       *
  *   51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA            *
  ***************************************************************************/

- 
+

  /*
  * This code is largely based on the kremoteencodingplugin
  * Copyright (c) 2003 Thiago Macieira <thiago.macieira@kdemail.net>
  * Distributed under the same terms.
  */
  /*
  * This code is largely based on the kremoteencodingplugin
  * Copyright (c) 2003 Thiago Macieira <thiago.macieira@kdemail.net>
  * Distributed under the same terms.
  */

- 
+

 #include "dolphinremoteencoding.h"
 #include "dolphinremoteencoding.h"

+

 #include "dolphinviewactionhandler.h"
 #include "dolphinviewactionhandler.h"

+#include "dolphindebug.h"

 
 

-#include <kdebug.h>
-#include <kactionmenu.h>
-#include <kactioncollection.h>
-#include <kicon.h>
-#include <klocale.h>
-#include <kglobal.h>
-#include <kmimetype.h>
-#include <kconfig.h>
-#include <kcharsets.h>
-#include <kmenu.h>
-#include <kprotocolinfo.h>
-#include <kprotocolmanager.h>
-#include <kio/slaveconfig.h>
-#include <kio/scheduler.h>
-#include <kconfiggroup.h>
-
-#define DATA_KEY        QLatin1String("Charset")
+#include <KActionCollection>
+#include <KActionMenu>
+#include <KCharsets>
+#include <KConfig>
+#include <KConfigGroup>
+#include <KIO/Scheduler>
+#include <KLocalizedString>
+#include <KProtocolInfo>
+#include <KProtocolManager>
+
+#include <QMenu>
+
+#define DATA_KEY        QStringLiteral("Charset")

 
 DolphinRemoteEncoding::DolphinRemoteEncoding(QObject* parent, DolphinViewActionHandler* actionHandler)
    :QObject(parent),
     m_actionHandler(actionHandler),
     m_loaded(false),
 
 DolphinRemoteEncoding::DolphinRemoteEncoding(QObject* parent, DolphinViewActionHandler* actionHandler)
    :QObject(parent),
     m_actionHandler(actionHandler),
     m_loaded(false),

-    m_idDefault(0)    
+    m_idDefault(0)

 {
 {

-    m_menu = new KActionMenu(KIcon("character-set"), i18n("Select Remote Charset"), this);
-    m_actionHandler->actionCollection()->addAction("change_remote_encoding", m_menu);
-    connect(m_menu->menu(), SIGNAL(aboutToShow()),
-          this, SLOT(slotAboutToShow()));
+    m_menu = new KActionMenu(QIcon::fromTheme(QStringLiteral("character-set")), i18n("Select Remote Charset"), this);
+    m_actionHandler->actionCollection()->addAction(QStringLiteral("change_remote_encoding"), m_menu);
+    connect(m_menu->menu(), &QMenu::aboutToShow,
+          this, &DolphinRemoteEncoding::slotAboutToShow);

 
     m_menu->setEnabled(false);
     m_menu->setDelayed(false);
 
     m_menu->setEnabled(false);
     m_menu->setDelayed(false);


@@ -71,22 +69,22 @@ void DolphinRemoteEncoding::slotReload()
 void DolphinRemoteEncoding::loadSettings()
 {
     m_loaded = true;
 void DolphinRemoteEncoding::loadSettings()
 {
     m_loaded = true;

-    m_encodingDescriptions = KGlobal::charsets()->descriptiveEncodingNames();
+    m_encodingDescriptions = KCharsets::charsets()->descriptiveEncodingNames();

 
     fillMenu();
 }
 
 void DolphinRemoteEncoding::slotAboutToOpenUrl()
 {
 
     fillMenu();
 }
 
 void DolphinRemoteEncoding::slotAboutToOpenUrl()
 {

-    KUrl oldURL = m_currentURL;
+    QUrl oldURL = m_currentURL;

     m_currentURL = m_actionHandler->currentView()->url();
 
     m_currentURL = m_actionHandler->currentView()->url();
 

-    if (m_currentURL.protocol() != oldURL.protocol()) {
+    if (m_currentURL.scheme() != oldURL.scheme()) {

         // This plugin works on ftp, fish, etc.
         // everything whose type is T_FILESYSTEM except for local files
         if (!m_currentURL.isLocalFile() &&
             KProtocolManager::outputType(m_currentURL) == KProtocolInfo::T_FILESYSTEM) {
         // This plugin works on ftp, fish, etc.
         // everything whose type is T_FILESYSTEM except for local files
         if (!m_currentURL.isLocalFile() &&
             KProtocolManager::outputType(m_currentURL) == KProtocolInfo::T_FILESYSTEM) {

-          
+

             m_menu->setEnabled(true);
             loadSettings();
         } else {
             m_menu->setEnabled(true);
             loadSettings();
         } else {


@@ -102,10 +100,10 @@ void DolphinRemoteEncoding::slotAboutToOpenUrl()
 
 void DolphinRemoteEncoding::fillMenu()
 {
 
 void DolphinRemoteEncoding::fillMenu()
 {

-    KMenu* menu = m_menu->menu();
+    QMenu* menu = m_menu->menu();

     menu->clear();
 
     menu->clear();
 

- 
+

     for (int i = 0; i < m_encodingDescriptions.size();i++) {
         QAction* action = new QAction(m_encodingDescriptions.at(i), this);
         action->setCheckable(true);
     for (int i = 0; i < m_encodingDescriptions.size();i++) {
         QAction* action = new QAction(m_encodingDescriptions.at(i), this);
         action->setCheckable(true);


@@ -117,8 +115,8 @@ void DolphinRemoteEncoding::fillMenu()
     menu->addAction(i18n("Reload"), this, SLOT(slotReload()), 0);
     menu->addAction(i18n("Default"), this, SLOT(slotDefault()), 0)->setCheckable(true);
     m_idDefault = m_encodingDescriptions.size() + 2;
     menu->addAction(i18n("Reload"), this, SLOT(slotReload()), 0);
     menu->addAction(i18n("Default"), this, SLOT(slotDefault()), 0)->setCheckable(true);
     m_idDefault = m_encodingDescriptions.size() + 2;

-  
-    connect(menu, SIGNAL(triggered(QAction*)), this, SLOT(slotItemSelected(QAction*)));
+
+    connect(menu, &QMenu::triggered, this, &DolphinRemoteEncoding::slotItemSelected);

 }
 
 void DolphinRemoteEncoding::updateMenu()
 }
 
 void DolphinRemoteEncoding::updateMenu()


@@ -126,37 +124,35 @@ void DolphinRemoteEncoding::updateMenu()
     if (!m_loaded) {
         loadSettings();
     }
     if (!m_loaded) {
         loadSettings();
     }

-  
+

     // uncheck everything
     for (int i =  0; i < m_menu->menu()->actions().count(); i++) {
         m_menu->menu()->actions().at(i)->setChecked(false);
     }
     // uncheck everything
     for (int i =  0; i < m_menu->menu()->actions().count(); i++) {
         m_menu->menu()->actions().at(i)->setChecked(false);
     }

-  
-    QString charset = KIO::SlaveConfig::self()->configData(m_currentURL.protocol(),
-                                                           m_currentURL.host(), DATA_KEY);

 
 

+    const QString charset = KCharsets::charsets()->descriptionForEncoding(KProtocolManager::charsetFor(m_currentURL));

     if (!charset.isEmpty()) {
         int id = 0;
         bool isFound = false;
         for (int i = 0; i < m_encodingDescriptions.size(); i++) {
     if (!charset.isEmpty()) {
         int id = 0;
         bool isFound = false;
         for (int i = 0; i < m_encodingDescriptions.size(); i++) {

-            if (m_encodingDescriptions.at(i).contains(charset)) {
+            if (m_encodingDescriptions.at(i) == charset) {

                 isFound = true;
                 id = i;
                 break;
             }
         }
                 isFound = true;
                 id = i;
                 break;
             }
         }

-      
-        kDebug() << "URL=" << m_currentURL << " charset=" << charset;
+
+        qCDebug(DolphinDebug) << "URL=" << m_currentURL << " charset=" << charset;

 
         if (!isFound) {
 
         if (!isFound) {

-            kWarning() << "could not find entry for charset=" << charset ;
+            qCWarning(DolphinDebug) << "could not find entry for charset=" << charset ;

         } else {
             m_menu->menu()->actions().at(id)->setChecked(true);
         }
     } else {
         m_menu->menu()->actions().at(m_idDefault)->setChecked(true);
     }
         } else {
             m_menu->menu()->actions().at(id)->setChecked(true);
         }
     } else {
         m_menu->menu()->actions().at(m_idDefault)->setChecked(true);
     }

-    
+

 }
 
 void DolphinRemoteEncoding::slotAboutToShow()
 }
 
 void DolphinRemoteEncoding::slotAboutToShow()


@@ -169,13 +165,13 @@ void DolphinRemoteEncoding::slotAboutToShow()
 
 void DolphinRemoteEncoding::slotItemSelected(QAction* action)
 {
 
 void DolphinRemoteEncoding::slotItemSelected(QAction* action)
 {

-    if (action != 0) {
+    if (action) {

         int id = action->data().toInt();
         int id = action->data().toInt();

-    
-        KConfig config(("kio_" + m_currentURL.protocol() + "rc").toLatin1());
+
+        KConfig config(("kio_" + m_currentURL.scheme() + "rc").toLatin1());

         QString host = m_currentURL.host();
         if (m_menu->menu()->actions().at(id)->isChecked()) {
         QString host = m_currentURL.host();
         if (m_menu->menu()->actions().at(id)->isChecked()) {

-            QString charset = KGlobal::charsets()->encodingForName(m_encodingDescriptions.at(id));
+            QString charset = KCharsets::charsets()->encodingForName(m_encodingDescriptions.at(id));

             KConfigGroup cg(&config, host);
             cg.writeEntry(DATA_KEY, charset);
             config.sync();
             KConfigGroup cg(&config, host);
             cg.writeEntry(DATA_KEY, charset);
             config.sync();


@@ -190,7 +186,7 @@ void DolphinRemoteEncoding::slotDefault()
 {
     // We have no choice but delete all higher domain level
     // settings here since it affects what will be matched.
 {
     // We have no choice but delete all higher domain level
     // settings here since it affects what will be matched.

-    KConfig config(("kio_" + m_currentURL.protocol() + "rc").toLatin1());
+    KConfig config(("kio_" + m_currentURL.scheme() + "rc").toLatin1());

 
     QStringList partList = m_currentURL.host().split('.', QString::SkipEmptyParts);
     if (!partList.isEmpty()) {
 
     QStringList partList = m_currentURL.host().split('.', QString::SkipEmptyParts);
     if (!partList.isEmpty()) {


@@ -200,7 +196,7 @@ void DolphinRemoteEncoding::slotDefault()
         // Remove the exact name match...
         domains << m_currentURL.host();
 
         // Remove the exact name match...
         domains << m_currentURL.host();
 

-        while (partList.count()) {
+        while (!partList.isEmpty()) {

             if (partList.count() == 2) {
                 if (partList[0].length() <= 2 && partList[1].length() == 2) {
                     break;
             if (partList.count() == 2) {
                 if (partList[0].length() <= 2 && partList[1].length() == 2) {
                     break;


@@ -211,12 +207,12 @@ void DolphinRemoteEncoding::slotDefault()
                 break;
             }
 
                 break;
             }
 

-            domains << partList.join(".");
+            domains << partList.join(QLatin1Char('.'));

             partList.erase(partList.begin());
         }
 
         for (QStringList::const_iterator it = domains.constBegin(); it != domains.constEnd();++it) {
             partList.erase(partList.begin());
         }
 
         for (QStringList::const_iterator it = domains.constBegin(); it != domains.constEnd();++it) {

-            kDebug() << "Domain to remove: " << *it;
+            qCDebug(DolphinDebug) << "Domain to remove: " << *it;

             if (config.hasGroup(*it)) {
                 config.deleteGroup(*it);
             } else if (config.group("").hasKey(*it)) {
             if (config.hasGroup(*it)) {
                 config.deleteGroup(*it);
             } else if (config.group("").hasKey(*it)) {


@@ -238,4 +234,3 @@ void DolphinRemoteEncoding::updateView()
     m_actionHandler->currentView()->reload();
 }
 
     m_actionHandler->currentView()->reload();
 }
 

-#include "dolphinremoteencoding.moc"