Merge pull request #695 from ModOrganizer2/Develop

Stage for Release 2.2.0
Update version to 2.2.0
2026-07-27 13:58:24 -07:00 · 2019-05-03 21:32:33 -05:00 · 2019-05-03 21:08:20 -05:00 · 2019-05-02 06:01:11 -05:00 · 2019-05-02 05:57:34 -05:00 · 2019-04-30 05:24:46 -05:00
149 changed files with 28674 additions and 24704 deletions
@@ -1,25 +0,0 @@
-# The problem
-
-Briefly describe the issue you are experiencing (or the feature you want to see added to Mod Organizer). Tell us what you were trying to do and what happened instead. Remember, this is _not_ a place to ask questions. For that, go to our discord channel
-
-# Environment
-
-* Mod Organizer Version that exhibits the issue:
-* Last Mod Organizer Verison that did not exhibit the issue (if applicable):
-* Desktop OS/version used to run Mod Organizer:
-
-# Details
-
-If necessary, describe the problem you have been experiencing in more detail.
-
-# Link to Mod Organizer logs
-
-## USVFS
-Create a [GIST](https://gist.github.com) which is a paste of your USVFS logs, and link them here.
-Do _NOT_ paste your full USVFS logs here, as it will make this issue very long and hard to read!
-If you are reporting a bug, _always_ include the USVFS logs!
-
-## MO Interface
-Create a [GIST](https://gist.github.com) which is a paste of your mo_interface logs, and link them here.
-Do _NOT_ paste your full mo_interface logs here, as it will make this issue very long and hard to read!
-If you are reporting a bug, _always_ include the mo_interface logs!
@@ -0,0 +1,10 @@
+---
+name: Dev Ticket
+about: Reserved for development tasks and management.
+title: ''
+labels: Dev Ticket
+assignees: ''
+
+---
+
+Task:
@@ -0,0 +1,20 @@
+---
+name: Feature request
+about: Suggest an idea for this project
+title: ''
+labels: Feature Request
+assignees: ''
+
+---
+
+**Is your feature request related to a problem? Please describe.**
+A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
+
+**Describe the solution you'd like**
+A clear and concise description of what you want to happen.
+
+**Describe alternatives you've considered**
+A clear and concise description of any alternative solutions or features you've considered.
+
+**Additional context**
+Add any other context or screenshots about the feature request here.
@@ -0,0 +1,41 @@
+---
+name: Issue Report
+about: Create an issue report.
+title: ''
+labels: issue report
+assignees: ''
+
+---
+
+## The problem:
+Briefly describe the issue you are experiencing. Tell us what you were trying to do and what happened instead. Remember, this is _not_ a place to ask questions. For that, go to our [discord channel](https://discord.gg/5tCqt6V)
+
+
+## To Reproduce:
+Steps to reproduce the behavior:
+
+1. Go to '...'
+2. Click on '....'
+
+## Environment:
+
+* Mod Organizer Version that exhibits the issue:
+* Last Mod Organizer Verison that did not exhibit the issue (if applicable):
+* Desktop OS/version used to run Mod Organizer:
+
+## Details:
+If necessary, describe the problem you have been experiencing with more details and pictures.
+
+
+## Link to Mod Organizer logs:
+To access the log files you can press the Open folder icon over the modlist and select "Open MO2 Logs folder". You should close MO2 before sending or examining the logs as they are finished writing only after the program is closed.
+
+### USVFS:
+Create a [GIST](https://gist.github.com) which is a paste of your USVFS logs, and link them here.
+Do _NOT_ paste your full USVFS logs here, as it will make this issue very long and hard to read!
+If you are reporting a bug, _always_ include the USVFS logs!
+
+### MO Interface:
+Create a [GIST](https://gist.github.com) which is a paste of your mo_interface logs, and link them here.
+Do _NOT_ paste your full mo_interface logs here, as it will make this issue very long and hard to read!
+If you are reporting a bug, _always_ include the mo_interface logs!
@@ -0,0 +1,28 @@
+version: 1.0.{build}
+skip_branch_with_pr: true
+image: Visual Studio 2017
+environment:
+  WEBHOOK_URL:
+    secure: gOKbXaZM9ImtMD5XrYITvdyZUW/az082G9OIN1EC1VYoNNsKROB4s6VsQiL6qbLNLc4XQhc8Y9DhdU9Hfh49vhypcjUEQmMsENxnp+1hYMJctTqIYDunVSd767eez5wz6rKHqk+XOHmg5gyVNCV9u4DVcCZJSVe4UTKn1lc47H0=
+build_script:
+- cmd: >-
+    git clone --depth=1 --branch=%APPVEYOR_REPO_BRANCH% https://github.com/ModOrganizer2/modorganizer-umbrella.git c:\projects\modorganizer-umbrella 2> $null
+
+    mkdir c:\projects\modorganizer-build -type directory
+
+    cd c:\projects\modorganizer-umbrella
+
+    C:\Python37-x64\python.exe unimake.py -d c:\projects\modorganizer-build -s Appveyor_Build=True %APPVEYOR_PROJECT_NAME%
+artifacts:
+- path: vsbuild\src\RelWithDebInfo\ModOrganizer.exe
+  name: modorganizer_exe
+- path: vsbuild\src\RelWithDebInfo\ModOrganizer.pdb
+  name: modorganizer_pdb
+on_success:
+  - ps: Set-Location -Path $env:APPVEYOR_BUILD_FOLDER
+  - ps: Invoke-RestMethod https://raw.githubusercontent.com/DiscordHooks/appveyor-discord-webhook/master/send.ps1 -o send.ps1
+  - ps: ./send.ps1 success $env:WEBHOOK_URL
+on_failure:
+  - ps: Set-Location -Path $env:APPVEYOR_BUILD_FOLDER
+  - ps: Invoke-RestMethod https://raw.githubusercontent.com/DiscordHooks/appveyor-discord-webhook/master/send.ps1 -o send.ps1
+  - ps: ./send.ps1 failure $env:WEBHOOK_URL
@@ -1,3 +1,5 @@
+[![Build status](https://ci.appveyor.com/api/projects/status/23qmd9rufv9vpmg7?svg=true)](https://ci.appveyor.com/project/Modorganizer2/modorganizer)
+
 # Mod Organizer

 Mod Organizer (MO) is a tool for managing mod collections of arbitrary size. It is specifically designed for people who like to experiment with mods and thus need an easy and reliable way to install and uninstall them.
@@ -82,4 +84,3 @@ Here is a complete list:
 ### Unused Repositories
 * https://github.com/Modorganizer2/modorganizer-hookdll
 * https://github.com/TanninOne/modorganizer-tool_nmmimport
-
@@ -54,7 +54,6 @@ SET(organizer_SRCS
    executableslist.cpp
    editexecutablesdialog.cpp
    downloadmanager.cpp
-    downloadlistwidgetcompact.cpp
    downloadlistwidget.cpp
    downloadlistsortproxy.cpp
    downloadlist.cpp
@@ -80,7 +79,6 @@ SET(organizer_SRCS
    previewgenerator.cpp
    previewdialog.cpp
    aboutdialog.cpp
-    json.cpp
    modflagicondelegate.cpp
    genericicondelegate.cpp
    organizerproxy.cpp
@@ -92,6 +90,9 @@ SET(organizer_SRCS
    eventfilter.cpp
    moshortcut.cpp
    listdialog.cpp
+    lcdnumber.cpp
+    forcedloaddialog.cpp
+    forcedloaddialogwidget.cpp

    shared/windows_error.cpp
    shared/error_report.cpp
@@ -146,7 +147,6 @@ SET(organizer_HDRS
    executableslist.h
    editexecutablesdialog.h
    downloadmanager.h
-    downloadlistwidgetcompact.h
    downloadlistwidget.h
    downloadlistsortproxy.h
    downloadlist.h
@@ -172,7 +172,6 @@ SET(organizer_HDRS
    previewgenerator.h
    previewdialog.h
    aboutdialog.h
-    json.h
    modflagicondelegate.h
    genericicondelegate.h
    organizerproxy.h
@@ -187,6 +186,9 @@ SET(organizer_HDRS
    descriptionpage.h
    moshortcut.h
    listdialog.h
+    lcdnumber.h
+    forcedloaddialog.h
+    forcedloaddialogwidget.h

    shared/windows_error.h
    shared/error_report.h
@@ -216,8 +218,6 @@ SET(organizer_UIS
    installdialog.ui
    finddialog.ui
    editexecutablesdialog.ui
-    downloadlistwidgetcompact.ui
-    downloadlistwidget.ui
    credentialsdialog.ui
    categoriesdialog.ui
    activatemodsdialog.ui
@@ -228,6 +228,8 @@ SET(organizer_UIS
    browserdialog.ui
    aboutdialog.ui
    listdialog.ui
+    forcedloaddialog.ui
+    forcedloaddialogwidget.ui
  )

 SET(organizer_QRCS
@@ -246,7 +248,8 @@ SOURCE_GROUP(Headers FILES ${organizer_HDRS})
 SOURCE_GROUP(UI FILES ${organizer_UIS})

 # MO projects
-SET(default_project_path "${CMAKE_SOURCE_DIR}/..")
+SET(dependency_project_path "${DEPENDENCIES_DIR}")
+SET(default_project_path "${DEPENDENCIES_DIR}/modorganizer_super")
 GET_FILENAME_COMPONENT(${default_project_path} ${default_project_path} REALPATH)

 SET(project_path "${default_project_path}" CACHE PATH "path to the other mo projects")
@@ -263,6 +266,7 @@ FIND_PACKAGE(Qt5Quick REQUIRED)
 FIND_PACKAGE(Qt5Network REQUIRED)
 FIND_PACKAGE(Qt5WinExtras REQUIRED)
 FIND_PACKAGE(Qt5WebEngineWidgets REQUIRED)
+FIND_PACKAGE(Qt5WebSockets REQUIRED)
 FIND_PACKAGE(Qt5Qml REQUIRED)
 FIND_PACKAGE(Qt5LinguistTools)
 QT5_WRAP_UI(organizer_UIHDRS ${organizer_UIS})
@@ -292,7 +296,7 @@ INCLUDE_DIRECTORIES(${project_path}/uibase/src
                    ${project_path}/bsatk/src
                    ${project_path}/esptk/src
                    ${project_path}/archive/src
-                    ${project_path}/../usvfs/include
+                    ${dependency_project_path}/usvfs/include
                    ${project_path}/game_gamebryo/src/gamebryo
                    ${project_path}/game_gamebryo/src/creation
                    ${project_path}/game_features/src
@@ -321,12 +325,12 @@ ENDIF()
 ADD_EXECUTABLE(ModOrganizer WIN32 ${organizer_HDRS} ${organizer_SRCS} ${organizer_UIHDRS} ${organizer_RCS} ${organizer_RCCPPS} ${organizer_translations_qm})
 TARGET_LINK_LIBRARIES(ModOrganizer
                      Qt5::Widgets Qt5::WinExtras Qt5::WebEngineWidgets Qt5::Quick
-                      Qt5::Qml Qt5::QuickWidgets Qt5::Network
+                      Qt5::Qml Qt5::QuickWidgets Qt5::Network Qt5::WebSockets
                      ${Boost_LIBRARIES}
                      zlibstatic
                      uibase esptk bsatk githubpp
                      ${usvfs_name}
-                      Dbghelp advapi32 Version Shlwapi liblz4)
+                      Dbghelp advapi32 Version Shlwapi liblz4 Crypt32)

 IF (MSVC)
  SET_TARGET_PROPERTIES(ModOrganizer PROPERTIES COMPILE_FLAGS "/std:c++latest")
@@ -359,17 +363,19 @@ INSTALL(FILES ${CMAKE_CURRENT_SOURCE_DIR}/dlls.manifest.qt5
        DESTINATION bin/dlls
        RENAME dlls.manifest)

+INSTALL(FILES ${organizer_translations_qm} DESTINATION bin/translations)
+
+INSTALL(DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}/stylesheets
+                  ${CMAKE_CURRENT_SOURCE_DIR}/tutorials
+        DESTINATION bin)
+
+#We should do this last, since windeployqt.exe scans the folders in bin for required modules e.g QtQuick.2 required by tutorials.
 # use windeployqt.exe to install all required libraries
 SET(windeploy_parameters "--no-translations --plugindir qtplugins --libdir dlls --release-with-debug-info --no-compiler-runtime")
 INSTALL(
    CODE
    "EXECUTE_PROCESS(COMMAND
-        ${qt5bin}/windeployqt.exe ModOrganizer.exe --webenginewidgets ${windeploy_parameters}
-        WORKING_DIRECTORY ${CMAKE_INSTALL_PREFIX}/bin
-    )
-    # run it a second time because on the first run it misses some files
-    EXECUTE_PROCESS(COMMAND
-        ${qt5bin}/windeployqt.exe ModOrganizer.exe --webenginewidgets ${windeploy_parameters}
+        ${qt5bin}/windeployqt.exe ModOrganizer.exe --webenginewidgets --websockets ${windeploy_parameters}
        WORKING_DIRECTORY ${CMAKE_INSTALL_PREFIX}/bin
    )
    EXECUTE_PROCESS(COMMAND
@@ -386,10 +392,4 @@ INSTALL(
 )

 # qdds.dll needs installing manually as Qt no longer ships with it by default.
-INSTALL(FILES ${CMAKE_CURRENT_SOURCE_DIR}/../qdds.dll DESTINATION bin/dlls/imageformats)
-
-install(FILES ${organizer_translations_qm} DESTINATION bin/translations)
-
-INSTALL(DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}/stylesheets
-                  ${CMAKE_CURRENT_SOURCE_DIR}/tutorials
-        DESTINATION bin)
+INSTALL(FILES ${CMAKE_CURRENT_SOURCE_DIR}/../qdds.dll DESTINATION bin/dlls/imageformats)
@@ -157,7 +157,7 @@
         <item row="0" column="0">
          <widget class="QGroupBox" name="groupBox">
           <property name="title">
-            <string>Current Maintainers</string>
+            <string>Lead Developers/ Maintainers</string>
           </property>
           <layout class="QVBoxLayout" name="verticalLayout_6">
            <item>
@@ -175,11 +175,6 @@
                <string notr="true">AL12</string>
               </property>
              </item>
-              <item>
-               <property name="text">
-                <string notr="true">Diana</string>
-               </property>
-              </item>
              <item>
               <property name="text">
                <string notr="true">erasmux</string>
@@ -190,7 +185,7 @@
                <string notr="true">Silarn</string>
               </property>
              </item>
-			  <item>
+              <item>
               <property name="text">
                <string notr="true">LostDragonist</string>
               </property>
@@ -203,11 +198,11 @@
         <item row="1" column="0">
          <widget class="QGroupBox" name="groupBox_2">
           <property name="title">
-            <string>Major Contributors</string>
+            <string>Mo2 devs and Contributors</string>
           </property>
           <layout class="QVBoxLayout" name="verticalLayout_7" stretch="0">
            <item>
-             <widget class="QListWidget" name="listWidget_2">
+             <widget class="QListWidget" name="listWidget_3">
              <property name="selectionMode">
               <enum>QAbstractItemView::NoSelection</enum>
              </property>
@@ -216,6 +211,16 @@
                <string notr="true">AnyOldName3</string>
               </property>
              </item>
+              <item>
+               <property name="text">
+                <string>Project579</string>
+               </property>
+              </item>
+              <item>
+               <property name="text">
+                <string>przester</string>
+               </property>
+              </item>
             </widget>
            </item>
           </layout>
@@ -226,9 +231,9 @@
           <property name="title">
            <string>Translators</string>
           </property>
-           <layout class="QVBoxLayout" name="verticalLayout_6">
+           <layout class="QVBoxLayout" name="verticalLayout_61">
            <item>
-             <widget class="QListWidget" name="listWidget_2">
+             <widget class="QListWidget" name="listWidget_4">
              <property name="selectionMode">
               <enum>QAbstractItemView::NoSelection</enum>
              </property>
@@ -310,6 +315,11 @@
                <string>Brgodfx (Portuguese)</string>
               </property>
              </item>
+              <item>
+               <property name="text">
+                <string>zDas (Portuguese)</string>
+               </property>
+              </item>
              <item>
               <property name="text">
                <string notr="true">tokcdk  (Russian)</string>
@@ -330,6 +340,11 @@
                <string>Jax (Swedish)</string>
               </property>
              </item>
+              <item>
+               <property name="text">
+                <string>yohru (Japanese)</string>
+               </property>
+              </item>
              <item>
               <property name="text">
                <string>...and all other Transifex contributors!</string>
@@ -348,12 +363,17 @@
           <property name="title">
            <string>Other Supporters &amp;&amp; Contributors</string>
           </property>
-           <layout class="QVBoxLayout" name="verticalLayout_7">
+           <layout class="QVBoxLayout" name="verticalLayout_71">
            <item>
             <widget class="QListWidget" name="listWidget">
              <property name="selectionMode">
               <enum>QAbstractItemView::NoSelection</enum>
              </property>
+              <item>
+               <property name="text">
+                <string>Tannin (Original Creator)</string>
+               </property>
+              </item>
              <item>
               <property name="text">
                <string notr="true">blacksol</string>
@@ -88,7 +88,7 @@ public:
                return temp.replace(tagIter->second.first, QString("<font style=\"color: #%1;\">%2</font>").arg(color, content));
              }
            } else {
-              qWarning("don't know how to deal with tag %s", qPrintable(tagName));
+              qWarning("don't know how to deal with tag %s", qUtf8Printable(tagName));
            }
          } else {
            if (tagName == "*") {
@@ -100,7 +100,7 @@ public:
          // expression doesn't match. either the input string is invalid
          // or the expression is
          qWarning("%s doesn't match the expression for %s",
-                   temp.toUtf8().constData(), tagName.toUtf8().constData());
+                   qUtf8Printable(temp), qUtf8Printable(tagName));
          length = 0;
          return QString();
        }
@@ -72,7 +72,7 @@ void CategoryFactory::loadCategories()
            bool ok = false;
            int temp = iter->toInt(&ok);
            if (!ok) {
-              qCritical("invalid id %s", iter->constData());
+              qCritical("invalid category id %s", iter->constData());
            }
            nexusIDs.push_back(temp);
          }
@@ -268,7 +268,7 @@ void CategoryFactory::loadDefaultCategories()
 int CategoryFactory::getParentID(unsigned int index) const
 {
  if (index >= m_Categories.size()) {
-    throw MyException(QObject::tr("invalid index %1").arg(index));
+    throw MyException(QObject::tr("invalid category index: %1").arg(index));
  }

  return m_Categories[index].m_ParentID;
@@ -303,7 +303,7 @@ bool CategoryFactory::isDecendantOf(int id, int parentID) const
 bool CategoryFactory::hasChildren(unsigned int index) const
 {
  if (index >= m_Categories.size()) {
-    throw MyException(QObject::tr("invalid index %1").arg(index));
+    throw MyException(QObject::tr("invalid category index: %1").arg(index));
  }

  return m_Categories[index].m_HasChildren;
@@ -313,7 +313,7 @@ bool CategoryFactory::hasChildren(unsigned int index) const
 QString CategoryFactory::getCategoryName(unsigned int index) const
 {
  if (index >= m_Categories.size()) {
-    throw MyException(QObject::tr("invalid index %1").arg(index));
+    throw MyException(QObject::tr("invalid category index: %1").arg(index));
  }

  return m_Categories[index].m_Name;
@@ -323,7 +323,7 @@ QString CategoryFactory::getCategoryName(unsigned int index) const
 int CategoryFactory::getCategoryID(unsigned int index) const
 {
  if (index >= m_Categories.size()) {
-    throw MyException(QObject::tr("invalid index %1").arg(index));
+    throw MyException(QObject::tr("invalid category index: %1").arg(index));
  }

  return m_Categories[index].m_ID;
@@ -334,7 +334,7 @@ int CategoryFactory::getCategoryIndex(int ID) const
 {
  std::map<int, unsigned int>::const_iterator iter = m_IDMap.find(ID);
  if (iter == m_IDMap.end()) {
-    throw MyException(QObject::tr("invalid category id %1").arg(ID));
+    throw MyException(QObject::tr("invalid category id: %1").arg(ID));
  }
  return iter->second;
 }
@@ -32,16 +32,6 @@ CredentialsDialog::~CredentialsDialog()
  delete ui;
 }

-QString CredentialsDialog::username() const
-{
-  return ui->usernameEdit->text();
-}
-
-QString CredentialsDialog::password() const
-{
-  return ui->passwordEdit->text();
-}
-
 bool CredentialsDialog::store() const
 {
  return ui->rememberCheck->isChecked();
@@ -34,9 +34,6 @@ public:
  explicit CredentialsDialog(QWidget *parent = 0);
  ~CredentialsDialog();

-  QString username() const;
-  QString password() const;
-
  bool store() const;
  bool neverAsk() const;

@@ -23,10 +23,13 @@ along with Mod Organizer.  If not, see <http://www.gnu.org/licenses/>.
 #include "utility.h"
 #include "report.h"
 #include "modinfo.h"
+#include "settings.h"

 #include <QApplication>
 #include <QDir>
 #include <QString>
+#include <QTextCodec>
+#include <gameplugins.h>


 using namespace MOBase;
@@ -83,13 +86,29 @@ void DirectoryRefresher::addModBSAToStructure(DirectoryEntry *directoryStructure
                                              int priority, const QString &directory, const QStringList &archives)
 {
  std::wstring directoryW = ToWString(QDir::toNativeSeparators(directory));
+  IPluginGame *game = qApp->property("managed_game").value<IPluginGame*>();
+
+  GamePlugins *gamePlugins = game->feature<GamePlugins>();
+  QStringList loadOrder = QStringList();
+  gamePlugins->getLoadOrder(loadOrder);

  for (const QString &archive : archives) {
    QFileInfo fileInfo(archive);
    if (m_EnabledArchives.find(fileInfo.fileName()) != m_EnabledArchives.end()) {
+
+      int order = -1;
+
+      for (auto plugin : loadOrder)
+      {
+        QString name = plugin.left(plugin.size() - 4);
+        if (fileInfo.fileName().startsWith(name + " - ", Qt::CaseInsensitive) || fileInfo.fileName().startsWith(name + ".", Qt::CaseInsensitive)) {
+          order = loadOrder.indexOf(plugin);
+        }
+      }
+
      try {
-        //directoryStructure->addFromBSA(ToWString(modName), directoryW,
-        //                                ToWString(QDir::toNativeSeparators(fileInfo.absoluteFilePath())), priority);
+        IPluginGame *game = qApp->property("managed_game").value<IPluginGame*>();
+        directoryStructure->addFromBSA(ToWString(modName), directoryW, ToWString(QDir::toNativeSeparators(fileInfo.absoluteFilePath())), priority, order);
      } catch (const std::exception &e) {
        throw MyException(tr("failed to parse bsa %1: %2").arg(archive, e.what()));
      }
@@ -107,6 +126,10 @@ void DirectoryRefresher::addModFilesToStructure(DirectoryEntry *directoryStructu
    // files to this mod
    FilesOrigin &origin = directoryStructure->createOrigin(ToWString(modName), directoryW, priority);
    for (const QString &filename : stealFiles) {
+      if (filename.isEmpty()) {
+        qWarning("Trying to find file with no name");
+        continue;
+      }
      QFileInfo fileInfo(filename);
      FileEntry::Ptr file = directoryStructure->findFile(ToWString(fileInfo.fileName()));
      if (file.get() != nullptr) {
@@ -115,9 +138,12 @@ void DirectoryRefresher::addModFilesToStructure(DirectoryEntry *directoryStructu
          file->removeOrigin(0);
        }
        origin.addFile(file->getIndex());
-        file->addOrigin(origin.getID(), file->getFileTime(), L"");
+        file->addOrigin(origin.getID(), file->getFileTime(), L"", -1);
      } else {
-        qWarning("%s not found", qPrintable(fileInfo.fileName()));
+        QString warnStr = fileInfo.absolutePath();
+        if (warnStr.isEmpty())
+          warnStr = filename;
+        qWarning("file not found: %s", qUtf8Printable(warnStr));
      }
    }
  } else {
@@ -133,7 +159,10 @@ void DirectoryRefresher::addModToStructure(DirectoryEntry *directoryStructure
                                           , const QStringList &archives)
 {
  addModFilesToStructure(directoryStructure, modName, priority, directory, stealFiles);
-  addModBSAToStructure(directoryStructure, modName, priority, directory, archives);
+  
+  if (Settings::instance().archiveParsing()) {
+    addModBSAToStructure(directoryStructure, modName, priority, directory, archives);
+  }
 }

 void DirectoryRefresher::refresh()
@@ -149,10 +178,9 @@ void DirectoryRefresher::refresh()
  std::wstring dataDirectory = QDir::toNativeSeparators(game->dataDirectory().absolutePath()).toStdWString();
  m_DirectoryStructure->addFromOrigin(L"data", dataDirectory, 0);

-  // TODO what was the point of having the priority in this tuple? the list is already sorted by priority
+  std::sort(m_Mods.begin(), m_Mods.end(), [](auto lhs, auto rhs){return lhs.priority < rhs.priority;});
  auto iter = m_Mods.begin();

-  //TODO i is the priority here, where higher = more important. the input vector is also sorted by priority but inverted!
  for (int i = 1; iter != m_Mods.end(); ++iter, ++i) {
    try {
      addModToStructure(m_DirectoryStructure, iter->modName, i, iter->absolutePath, iter->stealFiles, iter->archives);
@@ -162,6 +190,8 @@ void DirectoryRefresher::refresh()
    emit progress((i * 100) / static_cast<int>(m_Mods.size()) + 1);
  }

+  m_DirectoryStructure->getFileRegister()->sortOrigins();
+
  emit progress(100);

  cleanStructure(m_DirectoryStructure);
@@ -27,6 +27,7 @@ along with Mod Organizer.  If not, see <http://www.gnu.org/licenses/>.
 #include <vector>
 #include <set>
 #include <tuple>
+#include "profile.h"


 /**
@@ -21,6 +21,7 @@
  <file name="Qt5Svg.dll"/>
  <file name="Qt5WebEngineCore.dll"/>
  <file name="Qt5WebEngineWidgets.dll"/>
+  <file name="Qt5WebSockets.dll"/>
  <file name="Qt5Widgets.dll"/>
  <file name="Qt5WinExtras.dll"/>
-</assembly>
+</assembly>
@@ -20,6 +20,8 @@ along with Mod Organizer.  If not, see <http://www.gnu.org/licenses/>.
 #include "downloadlist.h"
 #include "downloadmanager.h"
 #include <QEvent>
+#include <QColor>
+#include <QIcon>

 #include <QSortFilterProxyModel>

@@ -31,6 +33,11 @@ DownloadList::DownloadList(DownloadManager *manager, QObject *parent)
  connect(m_Manager, SIGNAL(aboutToUpdate()), this, SLOT(aboutToUpdate()));
 }

+void DownloadList::setMetaDisplay(bool metaDisplay)
+{
+  m_MetaDisplay = metaDisplay;
+}
+

 int DownloadList::rowCount(const QModelIndex&) const
 {
@@ -62,36 +69,86 @@ QVariant DownloadList::headerData(int section, Qt::Orientation orientation, int
      (orientation == Qt::Horizontal)) {
    switch (section) {
      case COL_NAME: return tr("Name");
-      case COL_FILETIME: return tr("Filetime");
      case COL_SIZE: return tr("Size");
-      default: return tr("Done");
+      case COL_STATUS: return tr("Status");
+      case COL_FILETIME: return tr("Filetime");
+      default: return QVariant();
    }
  } else {
    return QAbstractItemModel::headerData(section, orientation, role);
  }
 }

-
 QVariant DownloadList::data(const QModelIndex &index, int role) const
 {
+  bool pendingDownload = index.row() >= m_Manager->numTotalDownloads();
  if (role == Qt::DisplayRole) {
-    return index.row();
+    if (pendingDownload) {
+      std::tuple<QString, int, int> nexusids = m_Manager->getPendingDownload(index.row() - m_Manager->numTotalDownloads());
+      switch (index.column()) {
+        case COL_NAME: return tr("< game %1 mod %2 file %3 >").arg(std::get<0>(nexusids)).arg(std::get<1>(nexusids)).arg(std::get<2>(nexusids));
+        case COL_SIZE: return tr("Unknown");
+        case COL_STATUS: return tr("Pending");
+      }
+    } else {
+      switch (index.column()) {
+        case COL_NAME: return m_MetaDisplay ? m_Manager->getDisplayName(index.row()) : m_Manager->getFileName(index.row());
+        case COL_SIZE: return sizeFormat(m_Manager->getFileSize(index.row()));
+        case COL_FILETIME: return m_Manager->getFileTime(index.row());
+        case COL_STATUS:
+          switch (m_Manager->getState(index.row())) {
+            // STATE_DOWNLOADING handled by DownloadProgressDelegate
+            case DownloadManager::STATE_STARTED: return tr("Started");
+            case DownloadManager::STATE_CANCELING: return tr("Canceling");
+            case DownloadManager::STATE_PAUSING: return tr("Pausing");
+            case DownloadManager::STATE_CANCELED: return tr("Canceled");
+            case DownloadManager::STATE_PAUSED: return tr("Paused");
+            case DownloadManager::STATE_ERROR: return tr("Error");
+            case DownloadManager::STATE_FETCHINGMODINFO: return tr("Fetching Info");
+            case DownloadManager::STATE_FETCHINGFILEINFO: return tr("Fetching Info");
+            case DownloadManager::STATE_FETCHINGMODINFO_MD5: return tr("Fetching Info");
+            case DownloadManager::STATE_READY: return tr("Downloaded");
+            case DownloadManager::STATE_INSTALLED: return tr("Installed");
+            case DownloadManager::STATE_UNINSTALLED: return tr("Uninstalled");
+          }
+      }
+    }
+  } else if (role == Qt::ForegroundRole && index.column() == COL_STATUS) {
+    if (pendingDownload) {
+      return QColor(Qt::darkBlue);
+    } else {
+      DownloadManager::DownloadState state = m_Manager->getState(index.row());
+      if (state == DownloadManager::STATE_READY)
+        return QColor(Qt::darkGreen);
+      else if (state == DownloadManager::STATE_UNINSTALLED)
+        return QColor(Qt::darkYellow);
+      else if (state == DownloadManager::STATE_PAUSED)
+        return QColor(Qt::darkRed);
+    }
  } else if (role == Qt::ToolTipRole) {
-    if (index.row() < m_Manager->numTotalDownloads()) {
+    if (pendingDownload) {
+      return tr("Pending download");
+    } else {
      QString text = m_Manager->getFileName(index.row()) + "\n";
      if (m_Manager->isInfoIncomplete(index.row())) {
        text += tr("Information missing, please select \"Query Info\" from the context menu to re-retrieve.");
      } else {
        const MOBase::ModRepositoryFileInfo *info = m_Manager->getFileInfo(index.row());
-        return QString("%1 (ID %2) %3<br><span>%4</span>").arg(info->modName).arg(m_Manager->getModID(index.row())).arg(info->version.canonicalString()).arg(info->description);
+        return QString("%1 (ID %2) %3<br><span>%4</span>").arg(info->modName).arg(m_Manager->getModID(index.row())).arg(info->version.canonicalString()).arg(info->description.chopped(4096));
      }
      return text;
-    } else {
-      return tr("pending download");
    }
-  } else {
-    return QVariant();
+  } else if (role == Qt::DecorationRole && index.column() == COL_NAME) {
+    if (!pendingDownload && m_Manager->getState(index.row()) >= DownloadManager::STATE_READY
+        && m_Manager->isInfoIncomplete(index.row()))
+      return QIcon(":/MO/gui/warning_16");
+  } else if (role == Qt::TextAlignmentRole) {
+    if (index.column() == COL_NAME)
+      return QVariant(Qt::AlignVCenter | Qt::AlignLeft);
+    else
+      return QVariant(Qt::AlignVCenter | Qt::AlignRight);
  }
+  return QVariant();
 }


@@ -103,13 +160,28 @@ void DownloadList::aboutToUpdate()

 void DownloadList::update(int row)
 {
-  if (row < 0) {
+  if (row < 0)
    emit endResetModel();
-  } else if (row < this->rowCount()) {
-#pragma message("updating only the one column is a hack")
-    emit dataChanged(this->index(row, 2, QModelIndex()), this->index(row, 2, QModelIndex()));
-  } else {
+  else if (row < this->rowCount())
+    emit dataChanged(this->index(row, 0, QModelIndex()), this->index(row, this->columnCount(QModelIndex())-1, QModelIndex()));
+  else
    qCritical("invalid row %d in download list, update failed", row);
-  }
 }

+QString DownloadList::sizeFormat(quint64 size) const
+{
+  qreal calc = size;
+  QStringList list;
+  list << "MB" << "GB" << "TB";
+
+  QStringListIterator i(list);
+  QString unit("KB");
+
+  calc /= 1024.0;
+  while (calc >= 1024.0 && i.hasNext()) {
+    unit = i.next();
+    calc /= 1024.0;
+  }
+
+  return QString().setNum(calc, 'f', 2) + " " + unit;
+}
@@ -22,7 +22,6 @@ along with Mod Organizer.  If not, see <http://www.gnu.org/licenses/>.

 #include <QAbstractTableModel>

-
 class DownloadManager;


@@ -38,9 +37,9 @@ public:

  enum EColumn {
    COL_NAME = 0,
-    COL_FILETIME,
    COL_STATUS,
-    COL_SIZE
+    COL_SIZE,
+    COL_FILETIME
  };

 public:
@@ -53,6 +52,8 @@ public:
   **/
  explicit DownloadList(DownloadManager *manager, QObject *parent = 0);

+  void setMetaDisplay(bool metaDisplay);
+
  /**
   * @brief retrieve the number of rows to display. Invoked by Qt
   *
@@ -91,7 +92,9 @@ public slots:
 private:

  DownloadManager *m_Manager;
+  bool m_MetaDisplay;

+  QString sizeFormat(quint64 size) const;
 };

 #endif // DOWNLOADLIST_H
@@ -37,18 +37,23 @@ void DownloadListSortProxy::updateFilter(const QString &filter)
 bool DownloadListSortProxy::lessThan(const QModelIndex &left,
                                     const QModelIndex &right) const
 {
-  int leftIndex  = left.data().toInt();
-  int rightIndex = right.data().toInt();
+  int leftIndex  = left.row();
+  int rightIndex = right.row();
  if ((leftIndex < m_Manager->numTotalDownloads())
      && (rightIndex < m_Manager->numTotalDownloads())) {
    if (left.column() == DownloadList::COL_NAME) {
-      return m_Manager->getFileName(leftIndex).compare(m_Manager->getFileName(rightIndex), Qt::CaseInsensitive) < 0;
-    } else if (left.column() == DownloadList::COL_FILETIME) {
-      return m_Manager->getFileTime(leftIndex) < m_Manager->getFileTime(rightIndex);
+      return m_Manager->getFileName(left.row()).compare(m_Manager->getFileName(right.row()), Qt::CaseInsensitive) < 0;
    } else if (left.column() == DownloadList::COL_STATUS) {
-      return m_Manager->getState(leftIndex) < m_Manager->getState(rightIndex);
-    } else if(left.column() == DownloadList::COL_SIZE){
-      return m_Manager->getFileSize(leftIndex) < m_Manager->getFileSize(rightIndex);
+      DownloadManager::DownloadState leftState = m_Manager->getState(left.row());
+      DownloadManager::DownloadState rightState = m_Manager->getState(right.row());
+      if (leftState == rightState)
+        return m_Manager->getFileTime(left.row()) < m_Manager->getFileTime(right.row());
+      else
+        return leftState > rightState;
+    } else if (left.column() == DownloadList::COL_SIZE) {
+      return m_Manager->getFileSize(left.row()) < m_Manager->getFileSize(right.row());
+    } else if (left.column() == DownloadList::COL_FILETIME) {
+      return m_Manager->getFileTime(left.row()) < m_Manager->getFileTime(right.row());
    } else {
      return leftIndex < rightIndex;
    }
@@ -63,11 +68,9 @@ bool DownloadListSortProxy::filterAcceptsRow(int sourceRow, const QModelIndex&)
  if (m_CurrentFilter.length() == 0) {
    return true;
  } else if (sourceRow < m_Manager->numTotalDownloads()) {
-    int downloadIndex = sourceModel()->index(sourceRow, 0).data().toInt();
-
    QString displayedName = Settings::instance().metaDownloads()
-        ? m_Manager->getDisplayName(downloadIndex)
-        : m_Manager->getFileName(downloadIndex);
+        ? m_Manager->getDisplayName(sourceRow)
+        : m_Manager->getFileName(sourceRow);
    return displayedName.contains(m_CurrentFilter, Qt::CaseInsensitive);
  } else {
    return false;
@@ -21,51 +21,64 @@ along with Mod Organizer.  If not, see <http://www.gnu.org/licenses/>.
 #define DOWNLOADLISTWIDGET_H

 #include "downloadmanager.h"
+#include "downloadlist.h"
+#include "downloadlistsortproxy.h"
 #include <QWidget>
 #include <QItemDelegate>
 #include <QLabel>
 #include <QProgressBar>
 #include <QTreeView>
+#include <QHeaderView>
+#include <QStyledItemDelegate>
+

 namespace Ui {
-    class DownloadListWidget;
+  class DownloadListWidget;
 }

-class DownloadListWidget : public QWidget
+class DownloadProgressDelegate : public QStyledItemDelegate
 {
-    Q_OBJECT
-
-public:
-    explicit DownloadListWidget(QWidget *parent = 0);
-    ~DownloadListWidget();
-
-
-private:
-    Ui::DownloadListWidget *ui;
-};
-
-class DownloadManager;
-
-class DownloadListWidgetDelegate : public QItemDelegate
-{
-
  Q_OBJECT

 public:
+  DownloadProgressDelegate(DownloadManager *manager, DownloadListSortProxy *sortProxy, QWidget *parent = 0) : QStyledItemDelegate(parent), m_Manager(manager), m_SortProxy(sortProxy) {}

-  DownloadListWidgetDelegate(DownloadManager *manager, bool metaDisplay, QTreeView *view, QObject *parent = 0);
-  ~DownloadListWidgetDelegate();
+  void paint(QPainter *painter, const QStyleOptionViewItem &option,
+    const QModelIndex &index) const override;

-  virtual void paint(QPainter *painter, const QStyleOptionViewItem &option, const QModelIndex &index) const;
-  virtual QSize sizeHint(const QStyleOptionViewItem &option, const QModelIndex &index) const;
+private:
+  DownloadManager *m_Manager;
+  DownloadListSortProxy *m_SortProxy;
+};

-  void paintPendingDownload(int downloadIndex) const;
-  void paintRegularDownload(int downloadIndex) const;
+class DownloadListHeader : public QHeaderView
+{
+  Q_OBJECT
+
+public:
+  explicit DownloadListHeader(Qt::Orientation orientation, QWidget *parent = nullptr) : QHeaderView(orientation, parent) {}
+  void customResizeSections();
+
+private:
+  void mouseReleaseEvent(QMouseEvent *event) override;
+};
+
+class DownloadListWidget : public QTreeView
+{
+  Q_OBJECT
+
+public:
+  explicit DownloadListWidget(QWidget *parent = 0);
+  ~DownloadListWidget();
+
+  void setManager(DownloadManager *manager);
+  void setSourceModel(DownloadList *sourceModel);
+  void setMetaDisplay(bool metaDisplay);

 signals:
-
  void installDownload(int index);
  void queryInfo(int index);
+  void queryInfoMd5(int index);
  void removeDownload(int index, bool deleteFile);
  void restoreDownload(int index);
  void cancelDownload(int index);
@@ -75,19 +88,10 @@ signals:
  void openFile(int index);
  void openInDownloadsFolder(int index);

-protected:
-
-	QString sizeFormat(quint64 size) const;
-  bool editorEvent(QEvent *event, QAbstractItemModel *model,
-                   const QStyleOptionViewItem &option, const QModelIndex &index);
-
-private:
-
-
-  void drawCache(QPainter *painter, const QStyleOptionViewItem &option, const QPixmap &cache) const;
-
 private slots:
-
+  void onDoubleClick(const QModelIndex &index);
+  void onCustomContextMenu(const QPoint &point);
+  void onHeaderCustomContextMenu(const QPoint &point);
  void issueInstall();
  void issueDelete();
  void issueRemoveFromView();
@@ -106,26 +110,14 @@ private slots:
  void issueRemoveFromViewCompleted();
  void issueRemoveFromViewUninstalled();
  void issueQueryInfo();
-
-  void stateChanged(int row, DownloadManager::DownloadState);
-  void resetCache(int);
+  void issueQueryInfoMd5();

 private:
-
-  DownloadListWidget *m_ItemWidget;
  DownloadManager *m_Manager;
-
-  bool m_MetaDisplay;
-
-  QLabel *m_NameLabel;
-  QLabel *m_SizeLabel;
-  QProgressBar *m_Progress;
-  QLabel *m_InstallLabel;
+  DownloadList *m_SourceModel = 0;
  int m_ContextRow;

-  QTreeView *m_View;
-
-  mutable QMap<int, QPixmap> m_Cache;
+  void resizeEvent(QResizeEvent *event);
 };

 #endif // DOWNLOADLISTWIDGET_H
--- a/Show More
+++ b/Show More