fix: properly set modpacks as hidden on creation

Fixes #734
TechnicPack · Dec 3, 2024 · f4e6ff1 · f4e6ff1
1 parent e178360
commit f4e6ff1
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 4 deletions.
diff --git a/app/Http/Controllers/ModpackController.php b/app/Http/Controllers/ModpackController.php
@@ -250,6 +250,7 @@ public function postCreate(): RedirectResponse
         $rules = [
             'name' => 'required|unique:modpacks',
             'slug' => 'required|unique:modpacks',
+            'hidden' => 'sometimes|required',
         ];
 
         $messages = [
@@ -266,7 +267,7 @@ public function postCreate(): RedirectResponse
         $modpack = new Modpack;
         $modpack->name = Request::input('name');
         $modpack->slug = Str::slug(Request::input('slug'));
-        $modpack->hidden = Request::input('hidden') ? false : true;
+        $modpack->hidden = request()->boolean('hidden');
         $modpack->icon_md5 = null;
         $modpack->icon_url = URL::asset('/resources/default/icon.png');
         $modpack->logo_md5 = null;

diff --git a/resources/views/modpack/create.blade.php b/resources/views/modpack/create.blade.php
@@ -30,6 +30,17 @@
                             <label for="slug">Modpack Slug</label>
                             <input type="text" class="form-control" name="slug" id="slug">
                         </div>
+                        <div class="form-group">
+                            <div class="checkbox">
+                                <label>
+                                    <input type="checkbox" name="hidden" id="hidden" checked>
+                                    Hide modpack
+                                </label>
+                                <p class="help-block">Hidden modpacks will not show up in the API response for the
+                                    modpack list. However, anyone with the modpack's slug can access all of its
+                                    information.</p>
+                            </div>
+                        </div>
                     </div>
                     <div class="col-md-6">
                         <p>Creating a modpack is simple. Fill in the information here.</p>

diff --git a/resources/views/modpack/edit.blade.php b/resources/views/modpack/edit.blade.php
@@ -41,9 +41,9 @@
                                 <input type="checkbox"
                                        name="hidden"
                                        id="hidden"{{ $checked = ($modpack->hidden ? ' checked' : '') }}>
-                                <span class="help-block">Hidden modpacks will not show up in the API response for the
-                                    modpack list regardless of whether or not a client has access to the modpack.
-                                </span>
+                                <p class="help-block">Hidden modpacks will not show up in the API response for the
+                                    modpack list. However, anyone with the modpack's slug can access all of its
+                                    information.</p>
                             </div>
                         </div>
                         <div class="control-group">