Closed (fixed)
Project:
Weight
Version:
8.x-3.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
4 Jul 2017 at 17:57 UTC
Updated:
1 Jan 2019 at 15:59 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
vprocessor commentedPatch attached but I am not sure about that it's correct way to get language from views row
Comment #3
vprocessor commentedPatch updated
Comment #4
andypostI guess it no go to save entity here + loop
Comment #5
VaanDeFanel commentedI confirm that the patch 3 works on Drupal 8.3.3 with module weight version 8.x-3.1-alpha1. I tested with 2 languages with views.
I hope it will be commited to the next release :)
Comment #6
ziomizar commentedThis will work just for node entity, it should works for all the entity types.
We need a more stable way to retrieve the language and then use a more easy syntax to implement this, like:
Comment #7
fall_0ut commentedComment #9
ziomizar commentedThanks @fall_0ut,
Now just the testing are missing
Comment #10
ziomizar commentedCreated a follow up on #3016674: Create tests for multi language support
Comment #11
ReBa commentedI have another problem with multilingual weight.
Reproduction:
Where I get the error:
This is probably because the weight field has been added later.
To avoid this problem, I've added a patch which solves the issue whereafter the normal functionality provided is executed properly.
I created a patch working from the current dev branch (today) so the patch applied in #7 should be ignored as it's already merged into the dev branch.
Comment #12
andypostNice catch
Comment #14
ziomizar commented@ReBa following your steps I dont get any errors managing the view.
I can reproduce your error with these:
In any case your patch make sense, thx!
Comment #16
ziomizar commentedComment #17
ziomizar commentedI found another bug just for coincidence:
Comment #18
ziomizar commentedremoving debugging
Comment #19
ziomizar commentedComment #21
ziomizar commented