Skip to content

Commit

Permalink
Merge pull request 'Use name field for post titles instead of `summ…
Browse files Browse the repository at this point in the history
…ary` (ref #1220)' (#173) from apub-post-name into main

Reviewed-on: https://yerbamate.ml/LemmyNet/lemmy/pulls/173
  • Loading branch information
dessalines committed Feb 22, 2021
2 parents 7dc3ff4 + 8d5e9f8 commit 2cbd158
Showing 1 changed file with 8 additions and 4 deletions.
12 changes: 8 additions & 4 deletions crates/apub/src/objects/post.rs
Original file line number Diff line number Diff line change
Expand Up @@ -58,8 +58,9 @@ impl ToApub for Post {
// https://git.asonix.dog/Aardwolf/activitystreams/issues/5
.set_many_contexts(lemmy_context()?)
.set_id(self.ap_id.to_owned().into_inner())
// Use summary field to be consistent with mastodon content warning.
// https://mastodon.xyz/@Louisa/103987265222901387.json
.set_name(self.name.to_owned())
// `summary` field for compatibility with lemmy v0.9.9 and older,
// TODO: remove this after some time
.set_summary(self.name.to_owned())
.set_published(convert_datetime(self.published))
.set_many_tos(vec![community.actor_id.into_inner(), public()])
Expand Down Expand Up @@ -174,8 +175,11 @@ impl FromApubToForm<PageExt> for PostForm {

let name = page
.inner
.summary()
.as_ref()
.name()
.map(|s| s.map(|s2| s2.to_owned()))
// The following is for compatibility with lemmy v0.9.9 and older
// TODO: remove it after some time (along with the map above)
.or_else(|| page.inner.summary().map(|s| s.to_owned()))
.context(location_info!())?
.as_single_xsd_string()
.context(location_info!())?
Expand Down

0 comments on commit 2cbd158

Please sign in to comment.