Skip to content

updated image file path with drupal service to fetch the correct path

Jincy Jithin requested to merge issue/pwa-3371404:pwa_3371404-manifest-not-served-with-D10 into 2.x

Updated ManifestConfigurationForm to fetch the uploaded icon form the correct folder, (/web/sites/default/files/pwa). This fixed the following issues:

  • "image uploaded is actually there it is but it doesn't show the current image"
  • "pop up doesnt show the one that let the user know that they can make the web page a desktop or phone app"
  • "In drupal 10.1.1 it is not possible to add the logo of the app, it gives an error and the PWA module cannot be used."
  • "Adding the icon gives this error. (https://www.drupal.org/project/pwa/issues/3373426)"

\Drupal::service('file_url_generator')->generateString("") used to replace $wrapper = $this->streamWrapperManager->getViaScheme( $this->config('system.file')->get('default_scheme') ); $file_path = $wrapper->realpath() . '/pwa/' . $file->getFilename(); updated to show the right image path with the code $file_path = $wrapper->realpath() . '/' . $file->getFilename();

Merge request reports