U4-11556 - V8 - Remove metablogConfig.config

Created by Lee Kelleher 06 Aug 2018, 09:08:07 Updated by Sebastiaan Janssen 26 Aug 2018, 15:58:26

Tags: Up For Grabs PR

Since CookComputing.XmlRpcV2 assembly was removed in #U4-10398, (due to reworking of the User section), the "metablogConfig.config" can be removed.

Searching through the codebase, there are no code references that use it: https://github.com/umbraco/Umbraco-CMS/search?q=metablogConfig

Comments

Sebastiaan Janssen 06 Aug 2018, 09:46:40

Can probably remove it in v7 as well then! 👍


Lee Kelleher 08 Aug 2018, 08:52:42

@sebastiaan My only concern with removing it for v7, is the SystemFiles.MetablogConfig string. https://github.com/umbraco/Umbraco-CMS/blob/dev-v7/src/Umbraco.Core/IO/SystemFiles.cs#L40-L46

Would removing that be considered a breaking-change?


Lee Kelleher 08 Aug 2018, 08:58:29

...or we could add an Obsolete attribute, same as the AccessXml entry? (for v7 that is)


Lee Kelleher 08 Aug 2018, 09:20:45

PR submitted: https://github.com/umbraco/Umbraco-CMS/pull/2841


Sebastiaan Janssen 08 Aug 2018, 09:47:14

Oh yes, sorry, didn't realize but I guess it could still be used in v7? I actually don't really know what this even does? Can it still be used for anything practical? 🤔


Lee Kelleher 08 Aug 2018, 10:00:34

It was used for "Content Channels" in the old User section. More control over hooking up to XML-RPC/MetaWeblog APIs/clients, such as Windows Live Writer, (even Word). But the rest of that code/logic got thrown out with the new User section rewrite in v7.7 anyway.


Shannon Deminick 09 Aug 2018, 04:20:05

yeah, it's all pretty much dead even in v7. for v8 we should remove all the stuff entirely


Priority: Normal

Type: Task

State: Fixed

Assignee:

Difficulty: Normal

Category:

Backwards Compatible: False

Fix Submitted:

Affected versions:

Due in version: 7.13.0

Sprint:

Story Points:

Cycle: