Improved the Link widget's URL generation logic

Author: fulopattila122

Changelog.md

@@ -1,5 +1,10 @@
 # AppShell Changelog
 
+## Unreleased
+##### 2026-XX-YY
+
+- Changed (Improved) the link widget so that it doesn't generate the link if it's denied by `onlyIf` or `onlyIfCan` directives
+
 ## 4.17.0
 ##### 2026-04-20
 

src/Widgets/Link.php

@@ -68,10 +68,11 @@ public static function create(Theme $theme, array $options = []): Link
     public function render($data = null): string
     {
         $url = $this->url;
+        $can = $this->can($data);
         return $this->renderViewFromTheme('link', array_merge($this->options, [
-            'can' => $this->can($data),
+            'can' => $can,
             'text' => $this->text->render($data),
-            'url' => $url($data, $this),
+            'url' => $can ? $url($data, $this) : null,
         ]));
     }
 

tests/Unit/LinkWidgetTest.php

@@ -188,4 +188,22 @@ public function class_can_be_added_to_the_inner_text_wrapper()
         $html = $link->render();
         $this->assertStringContainsString('<span class="text-muted">', $html);
     }
+
+    /** @test */
+    public function it_does_not_generate_a_route_if_it_is_not_allowed()
+    {
+        $link = Link::create(new AppShellTheme(), [
+            'text' => 'View Plan',
+            'url' => [
+                'route' => 'app.this_route_does_not_exist.show',
+                'parameters' => ['$model.plan_id'],
+            ],
+            'onlyIf' => '$model.plan_id',
+        ]);
+
+        $model = new \stdClass();
+        $model->plan_id = null;
+
+        $this->assertStringNotContainsString('href="', $link->render($model));
+    }
 }