Lauren
Supporter
Languages:
English (English )
Timezone:
America/New_York (GMT-05:00)
|
Thank you for testing further, I have reported that the issue is still happening on your site and our 2nd tier will continue looking into this.
|
Lauren
Supporter
Languages:
English (English )
Timezone:
America/New_York (GMT-05:00)
|
I think we finally figured out the correct settings. The Publications post type must be set to Translatable Only (not fallback to default) and those ACF fields set correctly as previously mentioned. With these two settings changed, I am not able to reproduce it in the posts that you shared links of that were not previously working:
hidden link
hidden link
hidden link
Let me know if you still see it disabled anywhere. Thanks!
|
johnS-9
|
Hi Lauren,
Sorry but I disagree that this is a solution, it would fundamentally change how content is currently being shown on the site.
I would like the Publications post type to be in "fallback to the default language" mode. If I change it to "translatable only" then the other languages on my client's site will be missing a lot of content that was there before.
The reason I believe this is a bug is because "Translate or fallback to default" mode means there *might* be a translation. If there's not a translation then show the default language item. This behaviour is what we want, this has been a feature of WPML for a long time.
So surely it is a bug that ACFML is disabling fields in "Translate or fallback to default mode". It is a perfectly valid scenario for a translation to exist and for those fields to be editable.
For example:
English - Original language.
French - Translation exists, I want to be able to translate the fields in to French.
Spanish - Translation does not exist, so show the original language version (English).
Many thanks,
|
Lauren
Supporter
Languages:
English (English )
Timezone:
America/New_York (GMT-05:00)
|
I can understand why you would prefer to use Fallback to default. Our developers have intentionally set this field to disable with that option, but have found a workaround. I can't say for sure if this is something that will change permanently, that will be up to our development team. In the meantime, you can use this workaround:
In acfml/classes/class-wpml-acf-relationship-field.php line 14:
Set:
$has_element_with_display_translated = true;
To:
$has_element_with_display_translated = false;
Please let me know if this resolves the issue.
|
johnS-9
|
Hi Lauren,
Thanks for the code change, that's helpful. I'm a bit apprehensive about applying patches to plugins that might receive an update and remove the patch.
I've added a new JS file to the admin area from my theme with this inside which should hopefully continue to work even if the plugin's updated. I'll see how this goes and revert to the PHP change if needed.
```js
$('.wpml_acf_annotation').prev().prop('disabled', false);
$('.wpml_acf_annotation.relationship').prev().find('input').prop('disabled', false);
$('.wpml_acf_annotation.taxonomy').prev().find('input').prop('disabled', false);
```
I would love if this change made it's way into an ACFML update.
Thanks for your help with this matter.
|