New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Misc refactoring #445
Misc refactoring #445
Conversation
Signed-off-by: Norv <a.w.norv@gmail.com>
Signed-off-by: Norv <a.w.norv@gmail.com>
… topicsList() (parallel to getBoardsList()) Signed-off-by: Norv <a.w.norv@gmail.com>
Signed-off-by: Norv <a.w.norv@gmail.com>
…r message. Not sure if this is so useful.Just need to clear the view of the action_post() code.. Signed-off-by: Norv <a.w.norv@gmail.com>
…x anyway? MessageIndex doesn't even use it) Signed-off-by: Norv <a.w.norv@gmail.com>
…mall function, which can probably be improved. Commented out log_topics part of markBoardsRead(), experimentally.. because I don't see why that update should happen when you mark boards as read... Put it back if it has consequences. Signed-off-by: Norv <a.w.norv@gmail.com>
Signed-off-by: Norv <a.w.norv@gmail.com>
Signed-off-by: Norv <a.w.norv@gmail.com>
{ | ||
// Limit them to $modSettings['preview_characters'] characters | ||
$row['first_body'] = strip_tags(strtr(parse_bbc($row['first_body'], $row['first_smileys'], $row['id_first_msg']), array('<br />' => ' '))); | ||
if ($smcFunc['strlen']($row['first_body']) > $modSettings['preview_characters']) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably you just missed this one, I'd prefer Util::strlen() instead of $smcFunc['strlen'], for the sake of consistency..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed norv@23dfd97
/methinks another batch of all replacements for utility functions is waiting for us. :)
Signed-off-by: Norv <a.w.norv@gmail.com>
Refactor poll information retrieval in Display controller. Attempt slightly consistency in subs function names.. Signed-off-by: Norv <a.w.norv@gmail.com>
Signed-off-by: Norv <a.w.norv@gmail.com>
Signed-off-by: Norv <a.w.norv@gmail.com>
Signed-off-by: Norv <a.w.norv@gmail.com>
…er $resetTopics() to switch it on or off if needed. Signed-off-by: Norv <a.w.norv@gmail.com>
Signed-off-by: Norv <a.w.norv@gmail.com>
Signed-off-by: Norv <a.w.norv@gmail.com>
…th the existing, i.e. messagePointer() and next/previous following topicPointer() and related. Signed-off-by: Norv <a.w.norv@gmail.com>
A batch of refactoring, from more parts of the code. It needs pretty badly another pair of eyes, 'cause mine are don't see anything anymore.. 🔮