In a lot of cases, bits of code are duplicated - probably unnecessarily. One example, is T98550/T100024. There are other examples out there, and we should seek these out and reduce them where possible.
Social Tools is a huge mess of dependencies anyway, but with duplicated code it becomes a whole lot worse.
Birthday formatting could should probably be totally rewritten, likely to use whatever methods the core Language class offers. It's awful, hacky and somewhat duplicated (between UserProfile and SpecialUpdateProfile classes). See rESPRcf0c2f84af6585a6d49354861e61242cb8de031a
PictureGame, PollNY and QuizGame all include a stripped-down version of Lokesh Dhakar's lightbox JS code. These three are almost identical; compared to the PollNY version, both PictureGame and QuizGame versions of the lightbox script include a jQuery( document ).ready( LightBox.init ); call at the bottom of the file to fire the lightbox once DOM is ready, and additionally the QuizGame version contains two additional opacity definitions in the setText function. Either way, this is a glorious nightmare.
Given that all of these extensions depend on the architecture provided by SocialProfile (the wAvatar class, point calculation via the UserStatsTrack class, etc.), it could be possible to create a folder like /extensions/SocialProfile/common/js/ and dump a "master copy" of the lightbox script there. This feels icky in the sense that if and when not everyone has PictureGame/PollNY/QuizGame installed, that script is just there, doing nothing but bloating up the total directory size. Then again disk space these days is pretty cheap and we probably don't need to worry about a few kilobytes...
On a very related matter, that's not all the code duplication as far as those three extensions go. Each extension's images directory is almost identical.
PictureGame's images directory:
PollNY's images directory:
QuizGame's images directory:
So all three extensions contain the same icons, overlay and AJAX loader images; the only differences are in the vote-bar images and their naming:
SocialProfile has a directory for common/shared (as in shared between SocialProfile "components", not sure if external social tools which still depend on SP use these) images, /extensions/SocialProfile/images/. It already contains ajax-loader-white.gif, pencilIcon.gif (under the file name editIcon.gif) and voteIcon.gif
And now for some duplicated things written in PHP:
Change 227714 had a related patch set uploaded (by Jack Phoenix):
Version 1.7.2: responsiveness, a tad bit less code duplication
Change 227714 merged by Jack Phoenix:
Version 1.7.2: responsiveness, a tad bit less code duplication
Change 231619 had a related patch set uploaded (by Jack Phoenix):
Add some common/shared assets to be used by PictureGame, PollNY & QuizGame
Change 231620 had a related patch set uploaded (by Jack Phoenix):
Version 3.2: less code duplication
Change 231623 had a related patch set uploaded (by Jack Phoenix):
Version 3.2: less code duplication
Change 231627 had a related patch set uploaded (by Jack Phoenix):
Version 3.3: less code duplication
Change 231619 merged by Jack Phoenix:
Add some common/shared assets to be used by PictureGame, PollNY & QuizGame
Change 275197 had a related patch set uploaded (by Jack Phoenix):
[WIP/UNTESTED?] Version 4.2.0: comments of the day cleaned up & unconditionally enabled
Change 275197 abandoned by Jack Phoenix:
[WIP/UNTESTED?] Version 4.2.0: comments of the day cleaned up & unconditionally enabled
Reason:
Will submit again as a new changeset.
Change 275198 had a related patch set uploaded (by Jack Phoenix):
[WIP/UNTESTED?] Version 4.2.0: comments of the day cleaned up & unconditionally enabled
Change 275198 merged by Jack Phoenix:
Comments of the day cleaned up & unconditionally enabled
Change 331082 had a related patch set uploaded (by Jack Phoenix):
Finally make the "comments of the day" feature work again
Change 331082 merged by jenkins-bot:
Finally make the "comments of the day" feature work again