-
Notifications
You must be signed in to change notification settings - Fork 7
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
Configuration works on new entries, but not on existing entries #54
Comments
Aha, interesting, I suspect this is to do with that |
Thanks for the quick response @joshangell. I just shot an email over way. Let me know if there's any additional info that'd be helpful to pass along. |
OK, a note on the exact reproduction of this:
Almost certainly to do with the URL matching for loading configs. |
- Qualified the Loader count() calls for opcode speed improvements - Fixed an issue with the configurations not being loaded on existing entries for non-default sites ([#54](#54)) - Fixed an issue where the global block types settings page would still show deleted global configs if there was a global set with a contextual config for that field * tag '3.3.6': Changelog for 3.3.6
@brettburwell done! Give it a whirl :) |
You're a champ. Thanks for so quickly digging into that one. We'll give it a look today! |
No worries! |
Howdy @joshangell. Just wanted to confirm that the fix looks good on this end. Thanks again! |
Brilliant! Thanks for letting me know :)
On Mon, 8 Apr 2019 at 16:37, Brett Burwell ***@***.***> wrote:
Howdy @joshangell <https://github.com/joshangell>. Just wanted to confirm
that the fix looks good on this end. Thanks again!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#54 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAYq-_E65-nu8f4RIm7fvGPW6BvNI67Tks5ve2IxgaJpZM4cWGhA>
.
--
-Josh
angell.io - specialist Craft CMS development and consultancy
|
We're seeing an issue where our Spoon configuration works properly on new entries, but isn't being applied when viewing existing entries.
While trying to debug the issue I noticed something unexpected happen that might provide a clue about what's going on.
As a test, I uninstalled and reinstalled Spoon and tried rebuilding the configuration from scratch (to see if that might resolve the issue). First, I did a quick check by adding a few blocks into a few groups to see if things were working as expecting.
In my initial test all seemed well. That configuration was working on new and existing entries.
Then I went back and updated the configuration (to more closely match the setup we have in our production environment). When I did this I immediately bumped into the same bug. Although I noticed that new entries were using the updated configuration, but existing entries were somehow referencing the original configuration (from the first test, where I added a few blocks to a few groups) [screen shots of both configurations are pasted below].
So from what I can gather, it looks like Spoon is somehow not referencing the updated field configuration consistently.
Below is a bit more detail. Please let me know if I can pass along any additional information that might be helpful.
Some background details on the site + configuration
Manage blocks on a per-site basis
.Double-instantiating a menu button on an element (garnish.js:220)
message 4 times when viewing an existing entry. I see the same message, but only twice when creating a new entry.Screen shot of the updated configuration (taken when first creating the test entry)
Screen shot of test configuration (taken after saving the test entry)
The text was updated successfully, but these errors were encountered: