Skip to content
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

Objectification #7846

Conversation

Sesquipedalian
Copy link
Member

Refactors SMF to use the object oriented code paradigm.

Signed-off-by: Jon Stovell <[email protected]>
Signed-off-by: Jon Stovell <[email protected]>
Signed-off-by: Jon Stovell <[email protected]>
Signed-off-by: Jon Stovell <[email protected]>
This eliminates a whole bunch of redundant code and variables.

Signed-off-by: Jon Stovell <[email protected]>
Done in a separate commit to preserve history.
Signed-off-by: Jon Stovell <[email protected]>
Signed-off-by: Jon Stovell <[email protected]>
Signed-off-by: Jon Stovell <[email protected]>
Signed-off-by: Jon Stovell <[email protected]>
Signed-off-by: Jon Stovell <[email protected]>
Signed-off-by: Jon Stovell <[email protected]>
Signed-off-by: Jon Stovell <[email protected]>
With this commit, Sources/Load.php ceases to exist because everything it contained has been moved elsewhere.
Signed-off-by: Jon Stovell <[email protected]>
Signed-off-by: Jon Stovell <[email protected]>
Signed-off-by: Jon Stovell <[email protected]>
Signed-off-by: Jon Stovell <[email protected]>
Signed-off-by: Jon Stovell <[email protected]>
Signed-off-by: Jon Stovell <[email protected]>
Also adds SMF\Unicode\Utf8String::extractWords(), which does a much better job of finding words in non-ASCII strings than the old code did. There is still room for improvement, but that will have to wait until later.

Signed-off-by: Jon Stovell <[email protected]>
Also fixes truncate_array(), since it didn't actually work. Also removes useless $deep parameter.

Signed-off-by: Jon Stovell <[email protected]>
Like Load.php, Subs.php has been removed rather than turned into a backward compatibility stub file because every existing mod would assume that it had already been included and wouldn't try to include it themselves. The class_exists() calls in Subs-Compat.php are all we need to do now to maintain backward compatibility for those mods.

Signed-off-by: Jon Stovell <[email protected]>
@Sesquipedalian
Copy link
Member Author

Bleh. I chose the wrong target branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant