We have moved to GitHub Issues
Created by Shannon Deminick 17 Mar 2016, 12:46:24 Updated by Andy Thompson 05 Jun 2018, 11:53:39Tags: Up For Grabs Community Contrib
Subtask of: U4-5419
This is a container issue, what needs to happen is we need to go through the codebase and find:
For anything that is to be removed, report it here and we'll determine if we should create sub-issues to track what is being removed.
Removed so far as part of this task
I've just submitted a pull request to get rid of:
See pull request:
Hopefully it's all good, but if there is anything else I can do on this please shout!
Oh sorry, just noticed you only wanted reports of things to be removed here, I was a bit gung-ho...
That's totally ok! Looks like you've done a stellar job :) Only reason I mentioned about reporting it first is to make sure that it's not already done or in progress by someone else.
@mangopieface I've pulled in your PR, great work! It's worth noting that anything relating to
SerializedTreeType.JsTree and (and basically anything relating to
JsTree) is obsolete and can be removed too :)
Awesome, I'll take a gander later on!
I've not taken on the JSTree yet, but tried to get rid of the following:
These are in PR https://github.com/umbraco/Umbraco-CMS/pull/1191
I'm going to put this back into Open, feel free to submit more PRs for more code removal :)
@Shandem I'm looking at removing the old SQL and MySQL installer bits that are marked as obsolete in umbraco.datalayer.
@mangopieface have you looked into these yet? (Thought as there was so much I'd offer a hand).
@nik nope, not yet I'll update here before I start removing anything
@Nik that part of the codebase is going to be VERY hard to remove, the problem is that we need to migrate over all of the usages of the old business logic code from the
umbraco.cms assembly. This is an ongoing (and very slow) process. I'll create tasks for each one of those very soon but until all of that is done there won't be much you can remove for the old sql stuff. That said, if you can remove anything from those assemblies that aren't being used or are easily migrated to new code please do!! :)
Hi, I want to join in and collaborate but as I'm new to github I would like to start very slowly and safely by making some extremely narrow changes, such as removing unneeded "using" directives, shortening object reference paths and adding/cleaning unit tests. Nothing that would make a major difference but would gain me experience in using pull requests and your process. Fixing these tiny things will make me read and learn the v7 source code before moving on to bigger tasks. Does that sound OK?
@Shandem I'd like to get involved and help clean the code. As you asked to be notified before changes are made, and as this will be my first proper PR with code, I'd like to look at the
Would you need the work done in both v7 and v8 branches?
Hi @Mundairson ! There are currently unit tests for StringExtensions found in Umbraco.Tests.Strings.StringExtensionsTests, see: https://github.com/umbraco/Umbraco-CMS/blob/temp8/src/Umbraco.Tests/Strings/StringExtensionsTests.cs
If you find there are extension methods untested please feel free to add unit tests :) I'd be happy if you want to just target v8 and if we see the same code surface area exists in v7 we can backport them if necessary.
I've submitted a PR to refactor the IsNullOrWhiteSpace() extension method.
The IsNullOrWhiteSpace() extension method's implementation can be replaced by a call to the native, static version with no loss in execution speed (there's actually a small gain on average). The main two reasons for the replacement, however, are a simple reduction in Core logic and no new string construction just to count the length. The call to Trim() could return a new string which is then thrown away. The native version steps along the string and is therefore pure, as this version now is.
This is just a PR change just to make sure I perform the PRs correctly. I added several unit tests around the change, plus the method itself is used in several other unit tests.
Priority: Up for grabs
Assignee: Shannon Deminick
Backwards Compatible: False
Due in version: 8.0.0