diff --git a/app/Models/PlatformModel.php b/app/Models/PlatformModel.php index 29c7259fc095727a761097ce81efe2655f1f6ef4..74242821dfe4e21a10b59418169082f388b8c67f 100644 --- a/app/Models/PlatformModel.php +++ b/app/Models/PlatformModel.php @@ -145,6 +145,8 @@ class PlatformModel extends Model /** * @param mixed[] $podcastsPlatformsData + * + * @return int|false Number of rows inserted or FALSE on failure */ public function savePodcastPlatforms( int $podcastId, @@ -165,6 +167,11 @@ class PlatformModel extends Model $this->db->query($deleteJoinQuery, [$podcastId, $platformType]); + if ($podcastsPlatformsData === []) { + // no rows inserted + return 0; + } + return $this->db ->table('podcasts_platforms') ->insertBatch($podcastsPlatformsData); diff --git a/modules/Admin/Controllers/PodcastPlatformController.php b/modules/Admin/Controllers/PodcastPlatformController.php index 6193642086420f8b4833997b1b81993be034fc26..cfc76aa2213401b5504b7b8151cc4036167300ce 100644 --- a/modules/Admin/Controllers/PodcastPlatformController.php +++ b/modules/Admin/Controllers/PodcastPlatformController.php @@ -70,7 +70,7 @@ class PodcastPlatformController extends BaseController $this->request->getPost('platforms') as $platformSlug => $podcastPlatform ) { - $podcastPlatformUrl = $podcastPlatform['url']; + $podcastPlatformUrl = trim((string) $podcastPlatform['url']); if ($podcastPlatformUrl === null) { continue; } @@ -79,11 +79,12 @@ class PodcastPlatformController extends BaseController continue; } + $podcastPlatformAccountId = trim((string) $podcastPlatform['account_id']); $podcastsPlatformsData[] = [ 'platform_slug' => $platformSlug, 'podcast_id' => $this->podcast->id, 'link_url' => $podcastPlatformUrl, - 'account_id' => $podcastPlatform['account_id'] === '' ? null : $podcastPlatform['account_id'], + 'account_id' => $podcastPlatformAccountId === '' ? null : $podcastPlatformAccountId, 'is_visible' => array_key_exists('visible', $podcastPlatform) && $podcastPlatform['visible'] === 'yes',