fix collectstatic command when using ManifestStaticFilesStorage

[anneFly]

Description:

Since version 0.5.0. our project doesn't build anymore because it fails during collectstatic.
This PR fixes it for me, but I'm not sure of any other side effects this change could have. So, if you decide this PR is not the right way to fix the issue, I'd appreciate a different solution because we can't upgrade to 0.5.0 atm.

How to reproduce:

With a fresh django project that has django-map-widgets version 0.5.0 installed, add this to the settings:

STATICFILES_STORAGE = "django.contrib.staticfiles.storage.ManifestStaticFilesStorage"

Then try to run python manage.py collectstatic.
You will get an error like this:

ValueError: The file 'mapwidgets/css/magnific-popup.css.map' could not be found with <django.contrib.staticfiles.storage.ManifestStaticFilesStorage object at 0x7fc7fb525e20>.

Possible future improvements:

It might be worth having a CI check that runs collectstatic with different STATICFILES_STORAGE settings. (If you'd like a contribution, I might be able to give it a shot but not anytime soon.)

[erdem]

Hi @anneFly

Thank you for your PR.

I integrated the latest version of the package into a fresh Django project just a few days ago and didn't encounter any failures during the collectstatic execution.

if you upgraded 0.4.2 version to 0.5.0,you may better to run collectstatic with clear argument.

python manage.py collectstatic --clear

It might be didn't fail in my setup because STATICFILES_STORAGE setting difference in my project. I will try to reproduce this issue.

It might be worth having a CI check that runs collectstatic with different STATICFILES_STORAGE settings. (If you'd like a contribution, I might be able to give it a shot but not anytime soon.)

noted.

Thanks

[vnagendra]

This is the same issue discussed here also codyfletcher/django-jazzmin-fork#1 and I can confirm that we are seeing this as well.

[erdem]

Thanks @anneFly and @vnagendra, I have just published the new release.

pip install django-map-widgets==0.5.1

[vnagendra]

Thank you for the timely merge/release! I was poking around to revert my changes to go back to the previous version when I saw this! Great job maintaining the project...