Merge branch 'dev' of https://github.com/huderlem/porymap into metatile-layers

Author: GriffinRichards

CHANGELOG.md

@@ -5,6 +5,10 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project somewhat adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).  The MAJOR version number is bumped when there are **"Breaking Changes"** in the pret projects. For more on this, see [the manual page on breaking changes](https://huderlem.github.io/porymap/manual/breaking-changes.html).
 
 ## [Unreleased]
+### Added
+- Add an option under `Preferences` to include common scripts in the autocomplete for Script labels.
+- A link to Porymap's manual is now available under `Help`.
+
 ### Changed
 - The scroll position of the map view now remains the same between the Connections tab and the Map/Events tabs.
 - The Move tool now behaves more like a traditional pan tool (with no momentum).

forms/mainwindow.ui

@@ -2926,6 +2926,7 @@
      <string>Help</string>
     </property>
     <addaction name="actionAbout_Porymap"/>
+    <addaction name="actionOpen_Manual"/>
     <addaction name="actionOpen_Log_File"/>
     <addaction name="actionOpen_Config_Folder"/>
     <addaction name="actionCheck_for_Updates"/>
@@ -3267,6 +3268,11 @@
     <string>Alt+Right</string>
    </property>
   </action>
+  <action name="actionOpen_Manual">
+   <property name="text">
+    <string>Open Manual</string>
+   </property>
+  </action>
  </widget>
  <layoutdefault spacing="6" margin="11"/>
  <customwidgets>

forms/preferenceeditor.ui

@@ -104,7 +104,7 @@
                <item row="3" column="0">
                 <spacer name="verticalSpacer_5">
                  <property name="orientation">
-                  <enum>Qt::Vertical</enum>
+                  <enum>Qt::Orientation::Vertical</enum>
                  </property>
                  <property name="sizeHint" stdset="0">
                   <size>
@@ -159,7 +159,7 @@
                <item row="0" column="3">
                 <spacer name="horizontalSpacer">
                  <property name="orientation">
-                  <enum>Qt::Horizontal</enum>
+                  <enum>Qt::Orientation::Horizontal</enum>
                  </property>
                  <property name="sizeHint" stdset="0">
                   <size>
@@ -209,7 +209,7 @@
                <item>
                 <spacer name="verticalSpacer_4">
                  <property name="orientation">
-                  <enum>Qt::Vertical</enum>
+                  <enum>Qt::Orientation::Vertical</enum>
                  </property>
                  <property name="sizeHint" stdset="0">
                   <size>
@@ -253,13 +253,42 @@
          </widget>
         </item>
         <item>
-         <widget class="QCheckBox" name="checkBox_AutocompleteAllScripts">
-          <property name="toolTip">
-           <string>&lt;html&gt;&lt;head/&gt;&lt;body&gt;&lt;p&gt;If checked, the list of suggestions when typing in an Event's Script field will include all global script labels in the project. Enabling this setting will make Porymap's startup slower.&lt;/p&gt;&lt;/body&gt;&lt;/html&gt;</string>
-          </property>
-          <property name="text">
-           <string>Autocomplete Script labels using all possible scripts</string>
+         <widget class="QGroupBox" name="groupBox_ScriptAutocomplete">
+          <property name="title">
+           <string>Script label autocomplete</string>
           </property>
+          <layout class="QVBoxLayout" name="verticalLayout_7">
+           <item>
+            <widget class="QRadioButton" name="radioButton_AutocompleteAllScripts">
+             <property name="toolTip">
+              <string>&lt;html&gt;&lt;head/&gt;&lt;body&gt;&lt;p&gt;If checked, the list of suggestions when typing in an Event's Script field will include all global script labels in the project. This is the slowest option for Porymap's project opening.&lt;/p&gt;&lt;/body&gt;&lt;/html&gt;</string>
+             </property>
+             <property name="text">
+              <string>All possible scripts</string>
+             </property>
+            </widget>
+           </item>
+           <item>
+            <widget class="QRadioButton" name="radioButton_AutocompleteCommonScripts">
+             <property name="toolTip">
+              <string>&lt;html&gt;&lt;head/&gt;&lt;body&gt;&lt;p&gt;If checked, the list of suggestions when typing in an Event's Script field will include script labels from the current map's scripts file, scripts in-use by the map's other events, and all script files in the &lt;span style=&quot; font-family:'SFMono-Regular','Menlo','Monaco','Consolas','Liberation Mono','Courier New','Courier','monospace'; font-size:11px; color:#e74c3c; background-color:#ffffff;&quot;&gt;data_scripts_folders &lt;/span&gt;folder.&lt;/p&gt;&lt;/body&gt;&lt;/html&gt;</string>
+             </property>
+             <property name="text">
+              <string>Current map, and global script files</string>
+             </property>
+            </widget>
+           </item>
+           <item>
+            <widget class="QRadioButton" name="radioButton_AutocompleteMapScripts">
+             <property name="toolTip">
+              <string>&lt;html&gt;&lt;head/&gt;&lt;body&gt;&lt;p&gt;If checked, the list of suggestions when typing in an Event's Script field will only include script labels from the current map's scripts file and scripts in-use by the map's other events. This is the fastest option for Porymap's project opening.&lt;/p&gt;&lt;/body&gt;&lt;/html&gt;</string>
+             </property>
+             <property name="text">
+              <string>Current map only</string>
+             </property>
+            </widget>
+           </item>
+          </layout>
          </widget>
         </item>
         <item>
@@ -294,7 +323,7 @@
         <item>
          <spacer name="verticalSpacer_3">
           <property name="orientation">
-           <enum>Qt::Vertical</enum>
+           <enum>Qt::Orientation::Vertical</enum>
           </property>
           <property name="sizeHint" stdset="0">
            <size>
@@ -323,7 +352,7 @@
         <item>
          <widget class="QScrollArea" name="scrollArea_TextEditor">
           <property name="frameShape">
-           <enum>QFrame::NoFrame</enum>
+           <enum>QFrame::Shape::NoFrame</enum>
           </property>
           <property name="widgetResizable">
            <bool>true</bool>
@@ -333,21 +362,21 @@
             <rect>
              <x>0</x>
              <y>0</y>
-             <width>476</width>
-             <height>343</height>
+             <width>495</width>
+             <height>376</height>
             </rect>
            </property>
            <layout class="QGridLayout" name="gridLayout_2">
             <property name="sizeConstraint">
-             <enum>QLayout::SetMinimumSize</enum>
+             <enum>QLayout::SizeConstraint::SetMinimumSize</enum>
             </property>
             <item row="6" column="0" colspan="2">
              <widget class="QLabel" name="label_TextEditorGotoLineHelp">
               <property name="text">
                <string>&lt;html&gt;&lt;head/&gt;&lt;body&gt;&lt;p&gt;When this command is set a button will appear next to the &lt;span style=&quot; font-weight:600; font-style:italic;&quot;&gt;Script&lt;/span&gt; combo-box in the &lt;span style=&quot; font-weight:600; font-style:italic;&quot;&gt;Events&lt;/span&gt; tab which executes this command.&lt;span style=&quot; font-weight:600;&quot;&gt; %F&lt;/span&gt; will be substituted with the file path of the script and &lt;span style=&quot; font-weight:600;&quot;&gt;%L&lt;/span&gt; will be substituted with the line number of the script in that file. &lt;span style=&quot; font-weight:600;&quot;&gt;%F &lt;/span&gt;&lt;span style=&quot; font-style:italic;&quot;&gt;must&lt;/span&gt; be given if &lt;span style=&quot; font-weight:600;&quot;&gt;%L&lt;/span&gt; is given. If &lt;span style=&quot; font-weight:600;&quot;&gt;%F&lt;/span&gt; is &lt;span style=&quot; font-style:italic;&quot;&gt;not&lt;/span&gt; given then the script's file path will be added to the end of the command. If the script can't be found then the current map's scripts file is opened.&lt;/p&gt;&lt;/body&gt;&lt;/html&gt;</string>
               </property>
               <property name="alignment">
-               <set>Qt::AlignLeading|Qt::AlignLeft|Qt::AlignTop</set>
+               <set>Qt::AlignmentFlag::AlignLeading|Qt::AlignmentFlag::AlignLeft|Qt::AlignmentFlag::AlignTop</set>
               </property>
               <property name="wordWrap">
                <bool>true</bool>
@@ -380,7 +409,7 @@
                <string>&lt;html&gt;&lt;head/&gt;&lt;body&gt;&lt;p&gt;This is the command that is executed when clicking &lt;span style=&quot; font-weight:600; font-style:italic;&quot;&gt;Open Project in Text Editor&lt;/span&gt; in the &lt;span style=&quot; font-weight:600; font-style:italic;&quot;&gt;Tools&lt;/span&gt; menu. &lt;span style=&quot; font-weight:600;&quot;&gt;%D&lt;/span&gt; will be substituted with the project's root directory. If &lt;span style=&quot; font-weight:600;&quot;&gt;%D&lt;/span&gt; is &lt;span style=&quot; font-style:italic;&quot;&gt;not&lt;/span&gt; specified then the project directory will be added to the end of the command.&lt;/p&gt;&lt;/body&gt;&lt;/html&gt;</string>
               </property>
               <property name="alignment">
-               <set>Qt::AlignLeading|Qt::AlignLeft|Qt::AlignTop</set>
+               <set>Qt::AlignmentFlag::AlignLeading|Qt::AlignmentFlag::AlignLeft|Qt::AlignmentFlag::AlignTop</set>
               </property>
               <property name="wordWrap">
                <bool>true</bool>
@@ -410,10 +439,10 @@
             <item row="2" column="0" colspan="2">
              <spacer name="verticalSpacer_2">
               <property name="orientation">
-               <enum>Qt::Vertical</enum>
+               <enum>Qt::Orientation::Vertical</enum>
               </property>
               <property name="sizeType">
-               <enum>QSizePolicy::Fixed</enum>
+               <enum>QSizePolicy::Policy::Fixed</enum>
               </property>
               <property name="sizeHint" stdset="0">
                <size>
@@ -426,7 +455,7 @@
             <item row="7" column="0" colspan="2">
              <spacer name="verticalSpacer">
               <property name="orientation">
-               <enum>Qt::Vertical</enum>
+               <enum>Qt::Orientation::Vertical</enum>
               </property>
               <property name="sizeHint" stdset="0">
                <size>
@@ -447,7 +476,7 @@
     <item>
      <widget class="QDialogButtonBox" name="buttonBox">
       <property name="standardButtons">
-       <set>QDialogButtonBox::Apply|QDialogButtonBox::Cancel|QDialogButtonBox::Ok</set>
+       <set>QDialogButtonBox::StandardButton::Apply|QDialogButtonBox::StandardButton::Cancel|QDialogButtonBox::StandardButton::Ok</set>
       </property>
      </widget>
     </item>

include/config.h

@@ -27,6 +27,12 @@ extern const QVersionNumber porymapVersion;
 
 #define CONFIG_BACKWARDS_COMPATABILITY
 
+enum ScriptAutocompleteMode {
+    MapOnly,
+    MapAndCommon,
+    All,
+};
+
 class KeyValueConfigBase
 {
 public:
@@ -119,7 +125,7 @@ class PorymapConfig: public KeyValueConfigBase
     QString textEditorGotoLine;
     int paletteEditorBitDepth;
     int projectSettingsTab;
-    bool loadAllEventScripts;
+    ScriptAutocompleteMode scriptAutocompleteMode;
     bool warpBehaviorWarningDisabled;
     bool eventDeleteWarningDisabled;
     bool eventOverlayEnabled;

include/core/map.h

@@ -148,7 +148,7 @@ class Map : public QObject
     void modified();
     void scriptsModified();
     void mapDimensionsChanged(const QSize &size);
-    void openScriptRequested(QString label);
+    void openScriptRequested(const QString &label);
     void connectionAdded(MapConnection*);
     void connectionRemoved(MapConnection*);
     void layoutChanged();

include/editor.h

@@ -209,7 +209,8 @@ class Editor : public QObject
 
 public slots:
     void openMapScripts() const;
-    void openScript(const QString &scriptLabel) const;
+    bool openScript(const QString &scriptLabel) const;
+    bool openScriptInFile(const QString &scriptLabel, const QString &filepath) const;
     void openProjectInTextEditor() const;
     void maskNonVisibleConnectionTiles();
     void onBorderMetatilesChanged();

include/mainwindow.h

@@ -290,6 +290,7 @@ private slots:
     void on_spinBox_SelectedCollision_valueChanged(int collision);
     void on_actionRegion_Map_Editor_triggered();
     void on_actionPreferences_triggered();
+    void on_actionOpen_Manual_triggered();
     void on_actionCheck_for_Updates_triggered();
     void togglePreferenceSpecificUi();
     void on_actionProject_Settings_triggered();

include/project.h

@@ -215,7 +215,11 @@ class Project : public QObject
 
     static QString getScriptFileExtension(bool usePoryScript);
     QString getScriptDefaultString(bool usePoryScript, QString mapName) const;
-    QStringList getEventScriptsFilepaths() const;
+
+    QStringList getAllEventScriptsFilepaths() const;
+    QStringList getMapScriptsFilepaths() const;
+    QStringList getCommonEventScriptsFilepaths() const;
+    QStringList findScriptsFiles(const QString &searchDir, const QStringList &fileNames = {"*"}) const;
     void insertGlobalScriptLabels(QStringList &scriptLabels) const;
 
     QString getDefaultPrimaryTilesetLabel() const;

include/ui/preferenceeditor.h

@@ -2,6 +2,7 @@
 #define PREFERENCES_H
 
 #include <QMainWindow>
+#include "config.h"
 
 class NoScrollComboBox;
 class QAbstractButton;
@@ -23,7 +24,7 @@ class PreferenceEditor : public QMainWindow
 signals:
     void preferencesSaved();
     void themeChanged(const QString &theme);
-    void scriptSettingsChanged(bool on);
+    void scriptSettingsChanged(ScriptAutocompleteMode mode);
     void reloadProjectRequested();
 
 private:

src/config.cpp

@@ -355,7 +355,7 @@ void PorymapConfig::reset() {
     this->textEditorGotoLine = "";
     this->paletteEditorBitDepth = 24;
     this->projectSettingsTab = 0;
-    this->loadAllEventScripts = false;
+    this->scriptAutocompleteMode = ScriptAutocompleteMode::MapOnly;
     this->warpBehaviorWarningDisabled = false;
     this->eventDeleteWarningDisabled = false;
     this->eventOverlayEnabled = false;
@@ -490,8 +490,13 @@ void PorymapConfig::parseConfigKeyValue(QString key, QString value) {
         }
     } else if (key == "project_settings_tab") {
         this->projectSettingsTab = getConfigInteger(key, value, 0);
+#ifdef CONFIG_BACKWARDS_COMPATABILITY
+    // Old setting replaced by script_autocomplete_mode
     } else if (key == "load_all_event_scripts") {
-        this->loadAllEventScripts = getConfigBool(key, value);
+        this->scriptAutocompleteMode = getConfigBool(key, value) ? ScriptAutocompleteMode::All : ScriptAutocompleteMode::MapOnly;
+#endif
+    } else if (key == "script_autocomplete_mode") {
+        this->scriptAutocompleteMode = static_cast<ScriptAutocompleteMode>(getConfigInteger(key, value, ScriptAutocompleteMode::MapOnly, ScriptAutocompleteMode::All));
     } else if (key == "warp_behavior_warning_disabled") {
         this->warpBehaviorWarningDisabled = getConfigBool(key, value);
     } else if (key == "event_delete_warning_disabled") {
@@ -613,7 +618,7 @@ QMap<QString, QString> PorymapConfig::getKeyValueMap() {
     map.insert("text_editor_goto_line", this->textEditorGotoLine);
     map.insert("palette_editor_bit_depth", QString::number(this->paletteEditorBitDepth));
     map.insert("project_settings_tab", QString::number(this->projectSettingsTab));
-    map.insert("load_all_event_scripts", QString::number(this->loadAllEventScripts));
+    map.insert("script_autocomplete_mode", QString::number(this->scriptAutocompleteMode));
     map.insert("warp_behavior_warning_disabled", QString::number(this->warpBehaviorWarningDisabled));
     map.insert("event_delete_warning_disabled", QString::number(this->eventDeleteWarningDisabled));
     map.insert("event_overlay_enabled", QString::number(this->eventOverlayEnabled));