-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
leaflet maps do not load in categories edit admin pages #1
Comments
Thanks for reporting this! We'll look at pushing a fix. |
Hi, any hints how to fix this, where to look for the bug? I could do a PR if you guide me. |
Hi Piotr,
I've been out of town, but will be back on Monday and will take a look
then. Sorry for the delay.
…--
Michael
On Aug 24, 2017 4:23 AM, "Piotr Szymański" <notifications@github.com> wrote:
Hi, any hints how to fix this, where to look for the bug? I could do a PR
if you guide me.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<#1 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AABAwMXSXXUo15EhZrHuj9-iwwfKofUaks5sbUD_gaJpZM4Ozhfq>
.
|
i'm using wp geometa 0.3.4 and acf 4.4.11 the map editor renders correctly on posts with these settings |
Can I just force put the javascript link ref in the admin page somewhere to get this working asap as a workaround? |
Thanks for the screenshots, I was able to reproduce the issue and think I have it fixed. Please let me know if it doesn't work for you now. The 0.0.5 beta release is here: If you'd like to see what changed, it was in leaflet-php and the changes are here: I missed the fact that since ACF 4 loads the fields via an admin-ajax call, the header scripts and styles were never dequeued in leaflet-php. This change dequeues them the first time a map is printed. This is a beta release and I have not tested it as thoroughly as I will before the actual release. As such, there may be bugs, so please test before deploying. |
I'm no longer able to reproduce the error here, but I thought of what should be a more robust way to ensure that the required libraries get loaded. The problem is that there's no map on the page when the page loads, so the usual Please uninstall the version of geometa-acf that you have installed and try the version attached here: Thanks, |
Unfortunately the map is still not there with acf 0.0.5beta2 and newest acf4 and newest wordpress. |
Can you try disabling all plugins except ACF and Geometa for ACF and see if the map appears then? Perhaps there's an unexpected conflict with another plugin. If the map does appear with other plugins disabled, tracking down which plugin Geometa for ACF conflicts with would be the next step. Thanks, |
Can you go to the WP-GeoMeta plugin page, to the System Status tab, and let
me know which install of WP-GeoMeta-Lib says *(In Use)* next to it?
[image: Inline image 1]
Then, can you disable WP-GeoMeta and see if the maps work then?
Thanks,
Michael Moore
…On Wed, Sep 27, 2017 at 11:19 AM, Piotr Szymański ***@***.***> wrote:
I've disabled all plugins left just geometa for acf, wp-geometa and acf.
still not working.
[image: image]
<https://user-images.githubusercontent.com/291663/30924918-6d0370fa-a3b0-11e7-93de-3b87d85302c0.png>
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<#1 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AABAwO4Kc05gnFr1Ey3eeuserBZnoPtYks5smnWsgaJpZM4Ozhfq>
.
|
Plugin Name | WP-GeoMeta Version disabling wp-geometa causes javascript errors in missing properties of objects and the map does not load |
it also says
|
(but all of them provide 0.3.3) |
Hi,
when the geometa acf field is assigned to both a post and category taxonomy, the version 0.0.4 does not load leaflet JS files on category edit page.
The text was updated successfully, but these errors were encountered: