I18n loot_scroller correct name

assets/voxygen/i18n/en/hud/chat.ftl

@@ -81,7 +81,13 @@ hud-chat-chat_tab_hover_tooltip = Right click for settings
 
 ## HUD Pickup message
 
-hud-loot-pickup-msg = { $amount ->
+hud-loot-pickup-msg = { $is_you ->
-   [one] { $actor } picked up { $item }
+    [true] You picked up { $amount ->
-   *[other] { $actor } picked up {$amount}x {$item}
+        [1] { $item }
+        *[other] {$amount}x {$item}
+    }
+    *[false] { $actor } picked up { $amount ->
+        [1] { $item }
+        *[other] { $amount }x { $item }
+   }
 }

assets/voxygen/i18n/ru/hud/chat.ftl

@@ -53,10 +53,25 @@ hud-chat-default_death_msg = { "[" }{ $name }] умер
 
 hud-chat-all = Все
 hud-chat-chat_tab_hover_tooltip = ПКМ для настроек
-hud-loot-pickup-msg =
+hud-loot-pickup-msg = { $is_you ->
-    { $actor } подобрал { $amount ->
+    [true] Вы подобрали { $amount ->
-        [one] { $item }
+        [1] { $item }
-       *[other] { $amount }x { $item }
+        *[other] {$amount}x {$item}
     }
+    *[false] { $gender ->
+        [she] { $actor } подобрала { $amount ->
+            [1] { $item }
+            *[other] { $amount }x { $item }
+        }
+        [he] { $actor } подобрал { $amount ->
+            [1] { $item }
+            *[other] { $amount }x { $item }
+        }
+        *[other] { $actor } подняло { $amount ->
+            [1] { $item }
+            *[other] { $amount }x { $item }
+        }
+    }
+}
 hud-chat-goodbye = До свидания!
 hud-chat-connection_lost = Соединение потеряно. Выход через { $time } секунд.

assets/voxygen/i18n/uk/hud/chat.ftl

@@ -163,33 +163,12 @@ hud-chat-chat_tab_hover_tooltip = Правий клік для налаштув
 
 ## HUD Pickup message
 
-# hud-loot-pickup-msg = {$actor} підняли { $amount ->
-#    [1] { $item }
-#    *[other] {$amount}x {$item}
-# }
-
-# hud-loot-pickup-msg = { $user_gender ->
-#     [she] { $actor } підняла { $amount ->
-#         [1] { $item }
-#         *[other] { $amount }x { $item }
-#     }
-#     [he] { $actor } підняв { $amount ->
-#         [1] { $item }
-#         *[other] { $amount }x { $item }
-#     }
-#     *[other] { $actor } підняло { $amount ->
-#         [1] { $item }
-#         *[other] { $amount }x { $item }
-#     }
-# }
-
 hud-loot-pickup-msg = { $is_you ->
     [true] Ви підняли { $amount ->
         [1] { $item }
         *[other] {$amount}x {$item}
     }
-
+    *[false] { $gender ->
-    *[false] { $user_gender ->
         [she] { $actor } підняла { $amount ->
             [1] { $item }
             *[other] { $amount }x { $item }

client/src/lib.rs

@@ -96,7 +96,7 @@ const PING_ROLLING_AVERAGE_SECS: usize = 10;
 #[derive(Debug)]
 pub enum Event {
     Chat(comp::ChatMsg),
-    GroupInventoryUpdate(comp::Item, String, Uid),
+    GroupInventoryUpdate(comp::Item, Uid),
     InviteComplete {
         target: Uid,
         answer: InviteAnswer,
@@ -2657,8 +2657,8 @@ impl Client {
                     kind,
                 });
             },
-            ServerGeneral::GroupInventoryUpdate(item, taker, uid) => {
+            ServerGeneral::GroupInventoryUpdate(item, uid) => {
-                frontend_events.push(Event::GroupInventoryUpdate(item, taker, uid));
+                frontend_events.push(Event::GroupInventoryUpdate(item, uid));
             },
             // Cleanup for when the client goes back to the `presence = None`
             ServerGeneral::ExitInGameSuccess => {

common/net/src/msg/server.rs

@@ -162,7 +162,7 @@ pub enum ServerGeneral {
     /// currently pending
     InvitePending(Uid),
     /// Update the HUD of the clients in the group
-    GroupInventoryUpdate(comp::Item, String, Uid),
+    GroupInventoryUpdate(comp::Item, Uid),
     /// Note: this could potentially include all the failure cases such as
     /// inviting yourself in which case the `InvitePending` message could be
     /// removed and the client could consider their invite pending until
@@ -330,7 +330,7 @@ impl ServerMsg {
                         | ServerGeneral::InviteComplete { .. }
                         | ServerGeneral::ExitInGameSuccess
                         | ServerGeneral::InventoryUpdate(_, _)
-                        | ServerGeneral::GroupInventoryUpdate(_, _, _)
+                        | ServerGeneral::GroupInventoryUpdate(_, _)
                         | ServerGeneral::TerrainChunkUpdate { .. }
                         | ServerGeneral::TerrainBlockUpdates(_)
                         | ServerGeneral::SetViewDistance(_)

server/src/client.rs

@@ -183,7 +183,7 @@ impl Client {
                     | ServerGeneral::InviteComplete { .. }
                     | ServerGeneral::ExitInGameSuccess
                     | ServerGeneral::InventoryUpdate(_, _)
-                    | ServerGeneral::GroupInventoryUpdate(_, _, _)
+                    | ServerGeneral::GroupInventoryUpdate(_, _)
                     | ServerGeneral::SetViewDistance(_)
                     | ServerGeneral::Outcomes(_)
                     | ServerGeneral::Knockback(_)

server/src/events/inventory_manip.rs

@@ -269,7 +269,6 @@ impl ServerEvent for InventoryManipEvent {
                                     &data.uids,
                                     &data.groups,
                                     &data.alignments,
-                                    &data.stats,
                                     &data.entities,
                                     &data.ability_map,
                                     &data.msm,
@@ -322,7 +321,6 @@ impl ServerEvent for InventoryManipEvent {
                                                     &data.uids,
                                                     &data.groups,
                                                     &data.alignments,
-                                                    &data.stats,
                                                     &data.entities,
                                                     &data.ability_map,
                                                     &data.msm,
@@ -1081,7 +1079,6 @@ fn announce_loot_to_group(
     uids: &ReadStorage<Uid>,
     groups: &ReadStorage<comp::Group>,
     alignments: &ReadStorage<comp::Alignment>,
-    stats: &ReadStorage<comp::Stats>,
     entities: &Entities,
     ability_map: &AbilityMap,
     msm: &MaterialStatManifest,
@@ -1090,14 +1087,12 @@ fn announce_loot_to_group(
         members(*group_id, groups, entities, alignments, uids)
             .filter(|(member_e, _)| member_e != &entity)
             .for_each(|(e, _)| {
-                clients.get(e).and_then(|c| {
+                clients.get(e).map(|c| {
-                    stats.get(entity).map(|stats| {
+                    c.send_fallible(ServerGeneral::GroupInventoryUpdate(
-                        c.send_fallible(ServerGeneral::GroupInventoryUpdate(
+                        item.duplicate(ability_map, msm),
-                            item.duplicate(ability_map, msm),
+                        *uid,
-                            stats.name.to_string(),
+                    ));
-                            *uid,
+                    Some(())
-                        ))
-                    })
                 });
             });
     }

voxygen/src/hud/loot_scroller.rs

@@ -10,6 +10,7 @@ use common::{
     comp::inventory::item::{Item, ItemDesc, ItemI18n, MaterialStatManifest, Quality},
     uid::Uid,
 };
+use common_net::sync::WorldSyncExt;
 use conrod_core::{
     color,
     position::Dimension,
@@ -354,50 +355,40 @@ impl<'a> Widget for LootScroller<'a> {
                 )
                 .set(state.ids.message_icons[i], ui);
 
-
+                let target_name = match self.client.player_list().get(taken_by) {
-    
+                    Some(info) => info.player_alias.clone(),
-                // let info = self.client.player_list().get(&taken_by.unwrap()).unwrap();
+                    None => match self.client.state().ecs().entity_from_uid(*taken_by) {
-                // info.character.unwrap().gender;
+                        Some(entity) => {
-
+                            let stats = self.client.state().read_storage::<common::comp::Stats>();
-                // let you_uid = self.client.uid().unwrap();
+                            stats
-                
+                                .get(entity)
-                // // Based on voxygen/i18n-helpers/src/lib.rs gender_str function
+                                .map_or(format!("<entity {}>", *taken_by), |e| e.name.to_owned())
-                // let gender_str = |uid: &Uid| {
+                        },
-                //     if let Some(pi) = info.character.as_ref().unwrap().gender{
+                        None => format!("<uid {}>", *taken_by),
-                //         match pi {
+                    },
-                //             common::comp::Gender::Feminine=> "she".to_owned(),
-                //             common::comp::Gender::Masculine=> "he".to_owned(),
-                //         }
-                //     } else {
-                //         "??".to_owned()
-                //     }
-                // };
-
-
-                // pub fn personalize_alias(&self, uid: Uid, alias: String) -> String {
-                //     let client_uid = self.uid().expect("Client doesn't have a Uid!!!");
-
-                let (user_gender, actor, is_you) = match self.client.player_list().get(taken_by) {
-                    Some(player_info) => match player_info.character.as_ref() {
-                        Some(character_info) => (match character_info.gender {
-                            Some(common::comp::Gender::Feminine)=> "she".to_string(),
-                            Some(common::comp::Gender::Masculine)=> "he".to_string(),
-                            None => "??".to_string(),
-                        }, character_info.name.to_string(), self.client.uid().expect("Client doesn't have a Uid!!!") == *taken_by),
-                        None => ("??".to_string(), "Someone".to_string(), false),
-                    }
-                    None => ("??".to_string(), "Som!!".to_string(), false),
                 };
 
-                // client_uid == uid;
+                let (user_gender, is_you) = match self.client.player_list().get(taken_by) {
-                // info.character.as_ref().unwrap().name.to_string()
+                    Some(player_info) => match player_info.character.as_ref() {
+                        Some(character_info) => (
+                            match character_info.gender {
+                                Some(common::comp::Gender::Feminine) => "she".to_string(),
+                                Some(common::comp::Gender::Masculine) => "he".to_string(),
+                                None => "??".to_string(),
+                            },
+                            self.client.uid().expect("Client doesn't have a Uid!!!") == *taken_by,
+                        ),
+                        None => ("??".to_string(), false),
+                    },
+                    None => ("??".to_string(), false),
+                };
 
                 let label = self.localized_strings.get_msg_ctx(
                     "hud-loot-pickup-msg",
                     &i18n::fluent_args! {
-                          "user_gender" => user_gender,
                           "is_you" => is_you.to_string(),
-                          "actor" => actor,
+                          "gender" => user_gender,
+                          "actor" => target_name,
                           "amount" => amount,
                           "item" => {
                               let (name, _) =

voxygen/src/session/mod.rs

@@ -289,7 +289,7 @@ impl SessionState {
                 client::Event::Chat(m) => {
                     self.hud.new_message(m);
                 },
-                client::Event::GroupInventoryUpdate(item, taker, uid) => {
+                client::Event::GroupInventoryUpdate(item, uid) => {
                     self.hud.new_loot_message(LootMessage {
                         amount: item.amount(),
                         item,