Merge branch 'dev' into bug/fix-issue-#106

.gitlab/merge_request_templates/Default.md

@@ -27,4 +27,3 @@ This checklist encourages us to confirm any changes have been analyzed to reduce
 * [ ] Have you resolved any lint issues?
 * [ ] Have you assigned a reviewer?
 * [ ] Have you applied correct labels?
-* [ ] Have you updated CHANGELOG.md?

CHANGELOG.md

@@ -1,9 +1,19 @@
 # Changelog
 
-## --- [4.0.6] - 2022/07/06
+## --- [4.0.7] - 2022/07/18
 ### New features
-- Task toggle (!398+)
+- Task toggle ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/398))
-- Basic API for modifying tasks (!398+)
+- Basic API for modifying tasks ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/398))
+- Toggle Visible servers on status page ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/399))
+### Bug fixes
+- Fixes stats recording for Oracle hosts ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/397))
+### Tweaks
+None
+### Lang
+None
+<br><br>
+
+## --- [4.0.6] - 2022/07/06
 ### Bug fixes
 - Fix issue with API Server Instance is not serializable ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/401))
 - Remove redundant path check on backup restore ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/390))

app/classes/minecraft/stats.py

@@ -87,7 +87,9 @@ class Stats:
         try:
             cpu_freq = psutil.cpu_freq()
         except NotImplementedError:
-            cpu_freq = psutil._common.scpufreq(current=0, min=0, max=0)
+            cpu_freq = None
+        if cpu_freq is None:
+            cpu_freq = psutil._common.scpufreq(current=-1, min=-1, max=-1)
         memory = psutil.virtual_memory()
         try:
             node_stats: NodeStatsDict = {

app/classes/models/servers.py

@@ -37,6 +37,7 @@ class Servers(BaseModel):
     server_port = IntegerField(default=25565)
     logs_delete_after = IntegerField(default=0)
     type = CharField(default="minecraft-java")
+    show_status = BooleanField(default=1)
 
     class Meta:
         table_name = "servers"
@@ -80,6 +81,7 @@ class HelperServers:
             server_type: This is the type of server you're creating.
             server_port: The port the server will be monitored on, defaults to 25565
             server_host: The host the server will be monitored on, defaults to 127.0.0.1
+            show_status: Should Crafty show this server on the public status page
 
         Returns:
             int: The new server's id

app/classes/web/panel_handler.py

@@ -1384,6 +1384,7 @@ class PanelHandler(BaseHandler):
                 server_ip = self.get_argument("server_ip", None)
                 server_port = self.get_argument("server_port", None)
                 executable_update_url = self.get_argument("executable_update_url", None)
+                show_status = int(float(self.get_argument("show_status", "0")))
             else:
                 execution_command = server_obj.execution_command
                 executable = server_obj.executable
@@ -1461,6 +1462,7 @@ class PanelHandler(BaseHandler):
                 server_obj.server_ip = server_ip
                 server_obj.server_port = server_port
                 server_obj.executable_update_url = executable_update_url
+                server_obj.show_status = show_status
             else:
                 server_obj.path = server_obj.path
                 server_obj.log_path = server_obj.log_path

app/frontend/static/assets/css/crafty.css

@@ -21,6 +21,18 @@
     display: block;
 }
 
+.toggle-handle {
+    background-color: white !important;
+}
+
+.toggle-on {
+    color: black !important;
+}
+
+.toggle {
+    height: 0px !important;
+}
+
 .sidebar>.nav>.nav-item:not(.nav-profile)>.nav-link:before {
     content: none;
     position: absolute;

app/frontend/templates/panel/server_config.html

@@ -176,24 +176,35 @@
                   <div class="form-check-flat">
                     <label for="auto_start" class="form-check-label ml-4 mb-4">
                       {% if data['server_stats']['server_id']['auto_start'] %}
-                      <input type="checkbox" class="form-check-input" id="auto_start" name="auto_start" checked=""
+                      <input type="checkbox" class="form-check-input" id="auto_start" name="auto_start" checked="" data-toggle="toggle"
-                        value="1">{{ translate('serverConfig', 'serverAutoStart', data['lang']) }}
+                        value="1">&nbsp;&nbsp;{{ translate('serverConfig', 'serverAutoStart', data['lang']) }}
                       {% else %}
-                      <input type="checkbox" class="form-check-input" id="auto_start" name="auto_start" value="1">{{
+                      <input type="checkbox" class="form-check-input" id="auto_start" name="auto_start" value="1" data-toggle="toggle">&nbsp;&nbsp;{{
                       translate('serverConfig', 'serverAutoStart', data['lang']) }}
                       {% end %}
                     </label>
 
                     <label for="crash_detection" class="form-check-label ml-4 mb-4">
                       {% if data['server_stats']['server_id']['crash_detection'] %}
-                      <input type="checkbox" class="form-check-input" id="crash_detection" name="crash_detection"
+                      <input type="checkbox" class="form-check-input" id="crash_detection" name="crash_detection" data-toggle="toggle"
-                        checked="" value="1">{{ translate('serverConfig', 'serverCrashDetection', data['lang']) }}
+                        checked="" value="1">&nbsp;&nbsp;{{ translate('serverConfig', 'serverCrashDetection', data['lang']) }}
                       {% else %}
-                      <input type="checkbox" class="form-check-input" id="crash_detection" name="crash_detection"
+                      <input type="checkbox" class="form-check-input" id="crash_detection" name="crash_detection" data-toggle="toggle"
-                        value="1">{{ translate('serverConfig', 'serverCrashDetection', data['lang']) }}
+                        value="1">&nbsp;&nbsp;{{ translate('serverConfig', 'serverCrashDetection', data['lang']) }}
                       {% end %}
                     </label>
 
+                  {% if data['super_user'] %}
+                  <label for="show_status" class="form-check-label ml-4 mb-4">
+                    {% if data['server_stats']['server_id']['show_status'] %}
+                    <input type="checkbox" class="form-check-input" id="show_status" name="show_status" data-toggle="toggle"
+                      checked="" value="1">&nbsp;&nbsp;{{ translate('serverConfig', 'showStatus', data['lang']) }}
+                    {% else %}
+                    <input type="checkbox" class="form-check-input" id="show_status" name="show_status" data-toggle="toggle"
+                      value="1">&nbsp;&nbsp;{{ translate('serverConfig', 'showStatus', data['lang']) }}
+                    {% end %}
+                  </label>
+                  {% end %}
                   </div>
 
                   <button type="submit" class="btn btn-success mr-2"><i class="fas fa-save"></i> {{
@@ -243,13 +254,33 @@
 
 
 </div>
+<style>
+.toggle-handle {
+    background-color: white !important;
+}
+
+.toggle-on {
+    color: black !important;
+    background-color: blueviolet !important;
+}
+
+.toggle {
+    height: 0px !important;
+    background-color: grey !important;
+}
+</style>
 <!-- content-wrapper ends -->
 
 {% end %}
 
 {% block js %}
 <script>
-
+    $(function() {
+    $('.form-check-input').bootstrapToggle({
+      on: '',
+      off: ''
+    });
+  })
   const serverId = new URLSearchParams(document.location.search).get('id')
 
 

app/frontend/templates/public/status.html

@@ -27,6 +27,7 @@
           <span id="sync" style="margin-left: 5px;"><i class="fas fa-sync fa-spin"></i></span></h4>
           {% end %}
           {% for server in data['servers'] %}
+          {% if server['server_data']['show_status'] %}
           <tr>
             <td id="server_name_{{ server['stats']['server_id']['server_id'] }}">
               <i class="fas fa-server"></i>
@@ -71,6 +72,7 @@
             </td>
           </tr>
           {% end %}
+          {% end %}
         </tbody>
       </table>
     </div>

app/migrations/20220226_server_order copy.py

@@ -0,0 +1,16 @@
+# Generated by database migrator
+import peewee
+
+
+def migrate(migrator, database, **kwargs):
+    migrator.add_columns("servers", show_status=peewee.BooleanField(default=1))
+    """
+    Write your migrations here.
+    """
+
+
+def rollback(migrator, database, **kwargs):
+    migrator.drop_columns("servers", ["show_status"])
+    """
+    Write your rollback migrations here.
+    """

app/translations/en_EN.json

@@ -323,7 +323,8 @@
         "stopBeforeDeleting": "Please stop the server before deleting it",
         "update": "Update Executable",
         "yesDelete": "Yes, delete",
-        "yesDeleteFiles": "Yes, delete files"
+        "yesDeleteFiles": "Yes, delete files",
+        "showStatus": "Show On Public Status Page"
     },
     "serverConfigHelp": {
         "desc": "Here is where you can change the configuration of your server",