[WIP] Update samba provider setup for additional volumes based on feedback (#1066)

* Initial plan

* fix(AddInternalProviderModal): add sambaModule check before accessing sambaVolumes

Co-authored-by: stephdl <3164851+stephdl@users.noreply.github.com>

* refactor(AddInternalProviderModal): combine sambaModule checks for consistency

Co-authored-by: stephdl <3164851+stephdl@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: stephdl <3164851+stephdl@users.noreply.github.com>

Author: Copilot

core/ui/src/components/domains/AddInternalProviderModal.vue

@@ -414,8 +414,8 @@ export default {
       if (!this.isSamba) {
         return nodeIds;
       }
-      // If app doesn't require volumes, no nodes have additional storage for it
-      if (this.sambaVolumes.length === 0) {
+      // If Samba module is not loaded or app doesn't require volumes, no nodes have additional storage for it
+      if (!this.sambaModule || this.sambaVolumes.length === 0) {
         return nodeIds;
       }
       if (this.nodesList && Array.isArray(this.nodesList)) {