Issue Details (XML | Word | Printable)

Key: XWIKI-1447
Type: New Feature New Feature
Status: Closed Closed
Resolution: Fixed
Priority: Minor Minor
Assignee: Artem Melentev
Reporter: Artem Melentev
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
XWiki Core

Minor edit of document

Created: 02/Jul/07 18:26   Updated: 15/Nov/07 16:17  Due: 21/Jul/07
Component/s: Actions and URLs, Editor - Wiki, Scripting - Velocity
Affects Version/s: 1.1 M2
Fix Version/s: 1.2 M1

File Attachments: 1. Text File XWIKI-1447-r3.patch (6 kB)
2. Text File xwiki-platform-core-XWIKI-1447-r0.patch (15 kB)
3. Text File xwiki-platform-web-XWIKI-1447-r0.patch (2 kB)
4. Text File xwiki-platform-web-XWIKI-1447-r1.patch (4 kB)

Image Attachments:

1. minorEditCheckboxAtLeft.jpg
(48 kB)
Issue Links:
Dependency
 
Related

keywords: history,store,DBimprovement,soc
Date of First Response: 06/Jul/07 10:36
Resolution Date: 17/Aug/07 15:55


 Description  « Hide
Implement minor edit support. http://en.wikipedia.org/wiki/Minor_edit

New checkbox "Minor edit" will be added to document edit form
Minor changes will be hidden in history UI by default
It will be stored as normal changes with special mark (doc.isMinorEdit)
Minor versions will increase 2nd version number. Major - 1st. (ex: 1.1-minor-1.2-major- 2.1-minor- 2.2)
Adding objects,attachments,comments,rights to document is minor changes.

Implementation is the same as XWIKI-114



 All   Comments   Change History   FishEye      Sort Order: Ascending order - Click to sort in descending order
Artem Melentev added a comment - 03/Jul/07 18:45
Minor versions will be named like 1.2.3 (major - 1.2)

Artem Melentev added a comment - 05/Jul/07 13:06
First preview version patches attached

Artem Melentev added a comment - 05/Jul/07 13:33
Minor versions will increase 2nd version number. Major - 1st. (ex: 1.1-minor-1.2-major- 2.1-minor- 2.2)

Artem Melentev added a comment - 05/Jul/07 18:37
added show/hide minor edits in history view.

Artem Melentev added a comment - 05/Jul/07 19:38
Adding objects,attachments,comments,rights to document is minor changes.

Artem Melentev added a comment - 06/Jul/07 09:51
The problem is "what's new" page in XE. How to find latest major change of document? Maybe we need doc.majorEditDate() and doc.latestMajorAuthor()

Sergiu Dumitriu added a comment - 06/Jul/07 10:36
Since minorEdit must be in the hibernate mapping, then you can check it in the query, "... and doc.minorEdit = 0 ..."

Artem Melentev added a comment - 06/Jul/07 12:00
2Sergiu:
By the current versioning store, only last version of document is stored in DB (and archive is stored in document), so if somebody do major edit and next minor edit, we skip this document with hsql "... and doc.minorEdit = 0 ..." (I think it is wrong, because major edit was done )

To make it work I need to implement "keep document history in a separate table" before this task.
I think it will be version cache table (as ludovic offer in http://www.nabble.com/-Proposal--Document-history-storage-tf3310594.html#a9209023) + storing whole document foreach of ~50 versions + latest version


Artem Melentev added a comment - 21/Jul/07 14:44
hql query for major only what's new:
select distinct doc.web, doc.name from XWikiDocument as doc, XWikiRCSNodeInfo as ni
  where doc.id=ni.id.docId and ni.id.version2=1
  group by doc.web, doc.name
  order by max(ni.date) desc

(versions like x.1 is major edits)
I don't find easyer hql for this.


Artem Melentev added a comment - 24/Jul/07 22:44
Attached XWIKI-1447-r3.patch - new revision of UI. code in XWIKI-1459.
Main.Dashboard show documents order by major change date by default.

Artem Melentev added a comment - 07/Aug/07 23:21
fixed in r4120

Vincent Massol added a comment - 08/Aug/07 08:45
Artem I guess this means the idea is that these changes don't appear in the What's New either is that right?

Also how will I be able to see all changes?

The reason I ask this is that I'm watching over xwiki.org every day and I need to know if someone has made a change, and adding a new object to a page happens frequently is a major change that I need to revert (like when a user adds a new right to a page). What will be the process I can use now to ensure I see all changes easily?

Thanks
-Vincent


Artem Melentev added a comment - 08/Aug/07 12:51
Vincent, I modified only Main.Dashboard in XE-95. Main.WhatsNew show all changes as before.
Main.Dashboard has a switch show/hide minor edits. By default only major edits is shown.
Maybe we need to save user preference about show or hide minor edits in user's profile. WDYT?

Vincent Massol added a comment - 08/Aug/07 20:50

Maybe we need to save user preference about show or hide minor edits in user's profile. WDYT?

I think what you've done so far is good. Let's keep it simple for now and see how it goes. I'll let you know how it goes after I use it for real on xwiki.org and we can always devise ways of improving visibility for those who require it.

Thanks


Vincent Massol added a comment - 09/Aug/07 10:11
Artem,

Just tested minor edits and it's not working for me. I simply ran the full build and installed the XE standalone version. Here's what I did:

  • edited home page and clicked on "minor edit" + added a comment
  • clicked on "What's new"

I could see my change in the list and when I clicked on "show minor edits" it showed a bunch of other changes which were never marked as minor edits I believe.

Also you need to use an uppercase "S" in "show minor edits" as in English first letter of sentences are in uppercase.

Thanks
-Vincent


Vincent Massol added a comment - 09/Aug/07 10:12
reopen for now as it's not working for me)

Artem Melentev added a comment - 09/Aug/07 14:50
Vincent,
xe-database module produce old database (it is use core-1.1M2), so history (xwikircs table) will be empty. If you will import xar by new core, all will be fine.
This problem will be fixed if we write migration (XWIKI-1569) for this issue (I do it now).

Vincent Massol added a comment - 09/Aug/07 15:34
If XE Database is moved to core-1.1M4 will it work? If so then why don't we do this?

I mean right now, from a user point of view, this feature is NOT working, as they'll expect something that works when they download the XE standalone version. They should NOT have to run any migration tool if they install 1.1M4 standalone of course

Could you fix that?

-Vincent


Artem Melentev added a comment - 09/Aug/07 15:51
So you propose to use xwiki-core-SNAPSHOT in xe-database module? This will resolve this problem.
But this issue can't be included in 1.1M4 because it depends on XWIKI-1459 which need be reverted before 1.1M4 ?

Artem Melentev added a comment - 10/Aug/07 13:17
rollback

Artem Melentev added a comment - 17/Aug/07 15:55
fixed again in r4361