bug: FTP remote type missing from profile form remote_storage dropdown #171

New Issue

2026-06-29T14:20:35Z

jmiller commented

2026-06-29 14:20:35 +00:00

Summary

The remote_storage dropdown in forms/profile.xml lists none, sftp, google_drive, and s3 — but ftp is missing. Meanwhile, a full ftp fieldset with 7 fields exists (host, port, username, password, path, passive, ssl), all with showon="remote_storage:ftp". Since ftp is never selectable, these fields can never be shown in the legacy profile form.

The new multi-remote system (#__mokosuitebackup_remotes) and RemoteTable::check() both accept ftp as a valid type. So the FTP fields are orphaned in the profile form but functional via the remotes CRUD modal.

Locations

forms/profile.xml line 220-227 — dropdown missing <option value="ftp">COM_MOKOJOOMBACKUP_REMOTE_FTP</option>
forms/profile.xml lines 411-477 — full FTP fieldset that can never be shown
src/Table/RemoteTable.php line 33 — accepts ftp as valid type

What to do

Either:

Option A: Add ftp to the remote_storage dropdown so legacy profile-level FTP config works
Option B: Remove the orphaned FTP fieldset entirely, since FTP is now handled via the multi-remote system

Why

Users cannot configure FTP via the profile form despite all the fields being defined. This is either a missing option or dead code that should be cleaned up.

## Summary The `remote_storage` dropdown in `forms/profile.xml` lists `none`, `sftp`, `google_drive`, and `s3` — but **`ftp` is missing**. Meanwhile, a full `ftp` fieldset with 7 fields exists (host, port, username, password, path, passive, ssl), all with `showon="remote_storage:ftp"`. Since `ftp` is never selectable, these fields can never be shown in the legacy profile form. The new multi-remote system (`#__mokosuitebackup_remotes`) and `RemoteTable::check()` both accept `ftp` as a valid type. So the FTP fields are orphaned in the profile form but functional via the remotes CRUD modal. ## Locations - `forms/profile.xml` line 220-227 — dropdown missing `<option value="ftp">COM_MOKOJOOMBACKUP_REMOTE_FTP</option>` - `forms/profile.xml` lines 411-477 — full FTP fieldset that can never be shown - `src/Table/RemoteTable.php` line 33 — accepts `ftp` as valid type ## What to do Either: - [ ] **Option A**: Add `ftp` to the `remote_storage` dropdown so legacy profile-level FTP config works - [ ] **Option B**: Remove the orphaned FTP fieldset entirely, since FTP is now handled via the multi-remote system ## Why Users cannot configure FTP via the profile form despite all the fields being defined. This is either a missing option or dead code that should be cleaned up.

jmiller added the component: remote label 2026-06-29 14:20:35 +00:00

jmiller commented

2026-06-29 14:20:38 +00:00

Branch created: feature/171-bug-ftp-remote-type-missing-from-profile

git fetch origin
git checkout feature/171-bug-ftp-remote-type-missing-from-profile

Branch created: [`feature/171-bug-ftp-remote-type-missing-from-profile`](https://git.mokoconsulting.tech/MokoConsulting/MokoSuiteBackup/src/branch/feature/171-bug-ftp-remote-type-missing-from-profile) ```bash git fetch origin git checkout feature/171-bug-ftp-remote-type-missing-from-profile ```

Sign in to join this conversation.

Priority Medium

Type Feature

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: MokoConsulting/MokoSuiteBackup#171