Problem/Motivation

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

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork advanced_pwa-3461174

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

ady1503 created an issue. See original summary.

gmaximus’s picture

gmaximus commented

Pls confirm your version of Drupal.

What did you do when this error was given?

ady1503’s picture

ady1503 commented

Versión de Drupal
10.3.1

PHP
Versión
8.1.29

Path: /system/files/advanced_pwa/icons/android-chrome-512x512.png. Symfony\Component\HttpKernel\Exception\AccessDeniedHttpException: in Drupal\system\FileDownloadController->download() (line 82 of /home/***/public_html/***/web/core/modules/system/src/FileDownloadController.php).

Drupal\Core\File\Exception\DirectoryNotReadyException: The specified file 'temporary://mstile-144x144.png' could not be copied because the destination directory 'public://images' is not properly configured. This may be caused by a problem with file or directory permissions. in Drupal\Core\File\FileSystem->prepareDestination() (line 457 of /home/***/public_html/***/web/core/lib/Drupal/Core/File/FileSystem.php).

Error: Call to a member function getCompleteBaseUrl() on null in Drupal\advanced_pwa\Controller\AdvancedpwaController->advancedpwaGetManifest() (line 190 of /home/***/public_html/***/web/modules/contrib/advanced_pwa/src/Controller/AdvancedpwaController.php).

Warning: Undefined property: Drupal\advanced_pwa\Controller\AdvancedpwaController::$requestContext en Drupal\advanced_pwa\Controller\AdvancedpwaController->advancedpwaGetManifest() (línea 190 de /home/***/public_html/***/web/modules/contrib/advanced_pwa/src/Controller/AdvancedpwaController.php)

gmaximus’s picture

gmaximus commented

Thanks for that.

I believe this is related to another issue about using the private file system. The above error suggests that the public:// directory is not set on the website?

I plan to spend 2-3 days on this project starting tomorrow. I think I know what needs tweaking to resolve this.

gMaximus opened merge request !14

  • gMaximus committed 2342b1d4 on 8.x-1.x 
    Issue #3461174 by gMaximus, ady1503: New install and this error
gmaximus’s picture

gmaximus commented

I believe part of the errors you gave me was related to the private file system. It is now hardcoded to public. So all is well as long as public is configured. Please test the latest dev and let me know if things work for you now.

gmaximus’s picture

gmaximus commented
Status: Active » Fixed
Related issues: +#3461175: Private system and multisite.

The file system scheme was fixed in the related issue.

gmaximus’s picture

gmaximus commented

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.