Warning: foreach() argument must be of type array|object, string given in Drupal\advanced_pwa\Form\ManifestConfigurationForm->submitForm()

Created on 28 November 2024, 6 months ago

Problem/Motivation

After installing the module andsetup first configuration, if we do not add screenshots, we catch the following error:

Warning: foreach() argument must be of type array|object, string given in Drupal\advanced_pwa\Form\ManifestConfigurationForm->submitForm() (line 446 of /var/www/html/drupal/modules/contrib/advanced_pwa/src/Form/ManifestConfigurationForm.php)

Steps to reproduce

1. Install module.
2. Go to `/admin/config/system/advanced-pwa`.
3. Fill required fields and save configuration.
4. Get an error.

Proposed resolution

Change validation of `uploaded_screenshots` from `->hasValue()`, to `->getValue()` and `empty()`.

🐛 Bug report

Status

Active

Version

1.71

Component

Code

Created by

🇵🇱Poland michal_liszka

Live updates comments and jobs are added and updated live.

Merge Requests

!15Warning: foreach() argument must be of type array|object, string given in Drupal\advanced_pwa\Form\ManifestConfigurationForm->submitForm()
Open
🇵🇱Poland michal_liszka
updated 6 months ago

Comments & Activities

Issue created by @michal_liszka
Comment 6 months ago →
🇵🇱Poland michal_liszka
michal_liszka → changed the visibility of the branch 8.x-1.x to hidden.
Merge request !15fix: foreach() argument must be of type array|object, string given. → (Open) created by michal_liszka
Status changed to RTBC 3 days ago7:15am 13 May 2025
Comment 3 days ago →
🇮🇳India arunkumark Coimbatore
Verified the patch applied successfully, and the warning issue was removed after the patch. Attaching the screenshot of the manual testing.

Applying patch:

Before Patch

After Patch

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024