Merge pull request #134 from lilyinstarlight/fix/empty-gecos
Set full_name_opt to None when GECOS data is empty
This commit is contained in:
commit
8e19b3f791
3 changed files with 4 additions and 2 deletions
|
|
@ -109,6 +109,7 @@ impl GreeterProxy {
|
|||
full_name_opt: user
|
||||
.gecos
|
||||
.as_ref()
|
||||
.filter(|s| !s.is_empty())
|
||||
.map(|gecos| gecos.split(',').next().unwrap_or_default().to_string()),
|
||||
icon_opt,
|
||||
theme_opt: None,
|
||||
|
|
|
|||
|
|
@ -105,6 +105,7 @@ fn user_data_fallback() -> Vec<UserData> {
|
|||
name: user.name,
|
||||
full_name_opt: user
|
||||
.gecos
|
||||
.filter(|s| !s.is_empty())
|
||||
.map(|gecos| gecos.split(',').next().unwrap_or_default().to_string()),
|
||||
icon_opt,
|
||||
theme_opt: None,
|
||||
|
|
|
|||
|
|
@ -642,9 +642,9 @@ impl cosmic::Application for App {
|
|||
}
|
||||
None => {}
|
||||
}
|
||||
match &self.flags.current_user.gecos {
|
||||
match self.flags.current_user.gecos.as_ref().filter(|s| !s.is_empty()) {
|
||||
Some(gecos) => {
|
||||
let full_name = gecos.split(",").next().unwrap_or("");
|
||||
let full_name = gecos.split(",").next().unwrap_or_default();
|
||||
column = column.push(
|
||||
widget::container(widget::text::title4(full_name))
|
||||
.width(Length::Fill)
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue