Skip to content
This repository has been archived by the owner on Sep 25, 2021. It is now read-only.

#484, update pinned thread titles for newly posted threads when possible #632

Closed
wants to merge 2 commits into from

Conversation

Adamantcheese
Copy link
Contributor

I have left an explanation as a comment.

@Adamantcheese
Copy link
Contributor Author

At this point I'm not going to submit any other pull requests. I think that the 8 I put in are plenty more than enough. If anyone's reading this, they can find a debug multi-release of all the features from the pulls here.

* The thread title will be updated as soon as the site has the thread listed in the thread directory
*
*/
pin.loadable.setTitle(PostHelper.getTitle(thread.op, pin.loadable));
Copy link
Contributor

@andyklimczak andyklimczak Mar 27, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure how expensive .setTitle is, but maybe we should .setTitle only if the current pin.loadable.title does not equal PostHelper.getTitle(thread.op, pin.loadable). something like:

if (!pin.loadable.getTitle().equals(PostHelper.getTitle(thread.op, pin.loadable)) {
  pin.loadable.setTitle(PostHelper.getTitle(thread.op, pin.loadable));
}

minor thing really I think, so not needed

Copy link
Contributor

@andyklimczak andyklimczak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Adamantcheese
Copy link
Contributor Author

This might also be a solution to #450, but it might not.

@Adamantcheese Adamantcheese deleted the 484 branch July 22, 2019 06:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants