?
avatar photodude
photodude
14 Jul 2016

Steps to reproduce the issue

with content versioning on,

  1. create a custom HTML module (save/close)
  2. edit custom HTML module (save close)

the pervious version should be maintained in the content history, but no version history is maintained

Expected result

Version history should by maintained for custom HTML modules

Actual result

no version history is maintained

Additional comments

custom HTML modules are very much like articles, and some site owners use custom HTML modules like micro content used through out a site. These same site owners may either desire to keep a content history or are required to keep such a history for corporate or regalatory reasons.

avatar photodude photodude - open - 14 Jul 2016
avatar brianteeman
brianteeman - comment - 15 Jul 2016

I dont remember why but this request was denied previously #7019 (comment)


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11136.

avatar photodude
photodude - comment - 15 Jul 2016

@brianteeman could the reason be somewhere on Gilp? (I'm not currently on Glip)

If we can't find a Justification for why this request was previously denied, I suggest we move it to a RFC.

avatar brianteeman
brianteeman - comment - 15 Jul 2016

On 15 Jul 2016 8:53 p.m., "Walt Sorensen" notifications@github.com wrote:

@brianteeman could the reason be somewhere on Gilp?

Unfortunately not it was a decision made at the cleanup sprint

avatar chrisdavenport
chrisdavenport - comment - 15 Jul 2016

@photodude I don't recall the reason either (and I was there) but I doubt it was a final "no way, ever" kind of thing, so if you would like to think through the technical and architectural issues and produce an RFC it will certainly be considered.

avatar mbabker
mbabker - comment - 15 Jul 2016

For one you'd basically have to version all modules or hack the UCM data
structure in a way that it can work with non-component type data. That's a
pretty big wall to get over when dealing with it.

On Friday, July 15, 2016, Chris Davenport notifications@github.com wrote:

@photodude https://github.com/photodude I don't recall the reason
either (and I was there) but I doubt it was a final "no way, ever" kind of
thing, so if you would like to think through the technical and
architectural issues and produce an RFC it will certainly be considered.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#11136 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAWfoSB_TKHXRSltu5NbLJwWd77QsbL2ks5qV-_igaJpZM4JM6XC
.

avatar photodude
photodude - comment - 15 Jul 2016

Thanks @chrisdavenport, I'll consider writing up some technical and architectural solutions.

@mbabker I agree that UCM structure is a big wall to deal with. IMO Modules should be versionable by type, and only if the module is set up to support versioning.

For the time being my efforts are going to be focused on the PHPCS 2.x Coding Standards and possibly the HHVM testing for some time. I'll definitely come back to this at some point but it's lower on my consideration list.

avatar brianteeman
brianteeman - comment - 16 Jul 2016

I remember now that the reasoning was to do with the issues with UCM and with the roadmap for page builder which would possibly make this need redundant


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11136.

avatar photodude
photodude - comment - 16 Jul 2016

page builder is on the Joomla 4 roadmap right?

I still think this maybe worth pursuing, pending the implementation of page builder.

avatar brianteeman
brianteeman - comment - 20 Jul 2016

If you want to go ahead and write some code then fine I'm certainly not going to stop you. I am going to close this now and if/when you create some code that will be a new PR


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11136.

avatar brianteeman brianteeman - change - 20 Jul 2016
Category Modules
avatar brianteeman brianteeman - change - 20 Jul 2016
Status New Closed
Closed_Date 0000-00-00 00:00:00 2016-07-20 17:51:53
Closed_By brianteeman
avatar brianteeman brianteeman - close - 20 Jul 2016
avatar brianteeman brianteeman - close - 20 Jul 2016
avatar brianteeman brianteeman - change - 21 Jul 2016
Labels Added: ?

Add a Comment

Login with GitHub to post a comment