Bug 113928

Summary: Read-only checkbox/toggle UNO command
Product: LibreOffice Reporter: Yousuf Philips (jay) (retired) <philipz85>
Component: LibreOfficeAssignee: Gülşah Köse <gulsah.1004>
Status: RESOLVED FIXED    
Severity: enhancement CC: gulsah.1004, raykowj, xiscofauli
Priority: medium    
Version: 6.0.0.0.alpha1+   
Hardware: All   
OS: All   
Whiteboard: target:6.1.0
Crash report or crash signature: Regression By:
Bug Depends on:    
Bug Blocks: 86899, 108882    

Description Yousuf Philips (jay) (retired) 2017-11-19 08:16:11 UTC
It would be identical to .uno:EditDoc, but would have the opposite toggle behaviour - would be checked/toggle when a document is not editable.
Comment 1 Yousuf Philips (jay) (retired) 2017-11-19 08:17:07 UTC
Gulsah, Jim: either of you want to take this on?
Comment 2 Jim Raykowski 2017-11-19 10:09:17 UTC
Currently working on background tab pages. Also in the progress of resubmitting the paragraph Alignment and Indents & Spacing tabs preview window fixes. Need some guidance on how to go about that [1]. In addition I have some requests for reviews to get to. Gulsah, if you are available for this one, go for it.

[1] https://gerrit.libreoffice.org/#/c/43678/
Comment 3 Gülşah Köse 2017-11-20 13:55:23 UTC
Hi Yousuf, it is hard to understand what that bug report wants. Could you give more detail about it. I have time, maybe i can do it.
Comment 4 Gülşah Köse 2017-11-20 15:28:39 UTC
OK, get it now,
Comment 5 Yousuf Philips (jay) (retired) 2017-11-20 17:39:46 UTC
(In reply to Gülşah Köse from comment #3)
> Hi Yousuf, it is hard to understand what that bug report wants. Could you
> give more detail about it. I have time, maybe i can do it.

Hi Gulsah, so .uno:EditDoc is toggled on when a document is editable and toggled off when its in read only mode. This bug is to create a 'Read Only' command which does the opposite, meaning it is toggled on when a document is in read only mode and toggled off when the document is editable.
Comment 6 Gülşah Köse 2017-11-20 20:50:09 UTC
Done. Needs code review https://gerrit.libreoffice.org/#/c/45005/
Comment 7 Commit Notification 2017-12-17 20:49:51 UTC
Gulsah Kose committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=5afa76782d192d66711e02400987d6153ef9e5a7

tdf#113928 Add uno:ReadOnlyDoc command.

It will be available in 6.1.0.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 8 Commit Notification 2017-12-18 07:39:08 UTC
Maxim Monastirsky committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=44ac1579f31c4ad7d026f7e2736ad2124e40295e

tdf#113928 More handling of SID_READONLYDOC

It will be available in 6.1.0.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 9 Maxim Monastirsky 2017-12-23 20:48:22 UTC
Seems fixed.