DNNCommunity / DNNCommunity/Dnn.CommunityForums

ENH: Move topic properties to separate table

Closed
#749 0 comments 0 reactions 1 assignee Claimed by @johnhenley View on GitHub
enhancement performance tasks technical debt
Dominant language
C#
Stars
18
Forks
23
Avg merge
19h 28m
Merged PRs (30d)
20

Description

Topic property values are stored in xml in "topicdata" column. Create a "topic properties" table and move each property value to separate row.

Context:

> @johnhenley it;s a long time ago. IIRC one thing was that you cannot remove a custom property once you created it

Hmm. That would make sense since property values are stored as an xml document in the topic table :(. So to remove a property would require unpacking and repacking every topic that has a property being deleted.

As I've moved through the modernization of the module, moving properties to separate data structures/tables has been something I've thought about doing. I can see properties being a very useful feature if they are more obvious and accessible.

_Originally posted by @johnhenley in https://github.com/DNNCommunity/Dnn.CommunityForums/issues/725#issuecomment-2049579696_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.