online icon indicating copy to clipboard operation
online copied to clipboard

Cannot protect / unprotect sheet

Open avatar1024 opened this issue 3 years ago • 16 comments

Since updating to CODE 6.4.0, it is no longer possible to protect or unprotect sheets on a spreadsheet. The option is missing both on right click on the sheet name and from the menu.

avatar1024 avatar Nov 06 '20 15:11 avatar1024

are you sure that un/protect are available in earlier versions of Online? If so, pls tell me where I have to look - I don't see it. thnx

cornouws avatar Nov 06 '20 21:11 cornouws

@cornouws Thank you for the reply. I am pretty sure since most spreadsheet are work with are protected but since I cannot go back to earlier version it's true that I cannot verify. I would be surprised if I had just noticed it know since I have been using CODE for a while know, but it is a possibility. Are you able to try on 4.2 or 4.0?

avatar1024 avatar Nov 06 '20 21:11 avatar1024

I will add it to the menubar, should work there.

Andreas-Kainz avatar Nov 06 '20 21:11 Andreas-Kainz

@Andreas-Kainz Thanks a lot! Is it available somewhere currently that I have missed?

avatar1024 avatar Nov 06 '20 22:11 avatar1024

didn't saw it, but I know the codebase not that long. however I sync the COOL menubar with the LibO one and protect is available so it should be available in Cool too.

Andreas-Kainz avatar Nov 06 '20 22:11 Andreas-Kainz

@Andreas-Kainz Ok thanks! I look forward to it. CODE has been shaping up really nicely and I'm trying to get more and more people on it! Can you update this post when you know in which release those change will be made?

avatar1024 avatar Nov 06 '20 22:11 avatar1024

One thing to be aware of ... we need to make sure any new dialogs we throw up are async enabled. What does that mean ? that means that the dialog doesn't chew up a stack frame by calling 'Execute' in the code, since this can cause problems for collaborative editing (that are not immediately apparent). So - worth getting code review on any new .uno commands we expose =) someone needs to take a peek at the implementation(s). Of course - if it was exposed before, or is present in Notebookbar but not in menus (or vv.) then someone already had a look I think; so great to sync. those two.

Thanks !

mmeeks avatar Nov 06 '20 22:11 mmeeks

I'm not 100% sure what you mean @mmeeks. What I did is, that I test before add an .uno command (dialog) to the menubar. In addition I'm sure that the dialogues I added to the menubar are also available in the notebookbar implementation (now standard layout). I didn't add commands which want be useful or didn't work. anyhow it can alway happen that something get wrong. I'm not a big fan that all the menubar's are located within one .js file. In LibO there are separate files for writer, calc, impress, ... in online everything is in one (big) Control.Menubar.js file.

Andreas-Kainz avatar Nov 06 '20 22:11 Andreas-Kainz

@cornouws Are you able to try on 4.2 or 4.0?

Yes. I have access to a server running 4.2 - So I carefully checked before asking you ;) But the good news is, that apparently working online and offline for you feel similarly great, that you see it as one experience ;)

cornouws avatar Nov 09 '20 14:11 cornouws

Hello, this got closed but as far as I can tell @Andreas-Kainz commit was not incorporated since I still cannot see any option (ether in right click on sheet name nor on the menu) to protect / unprotect sheets. Should it be reopened?

avatar1024 avatar Nov 29 '21 16:11 avatar1024

Sure, let's reopen. And just noting that I don't think protecting sheets was ever available in Collabora Online.

thebearon avatar Nov 30 '21 03:11 thebearon

Sure, let's reopen. And just noting that I don't think protecting sheets was ever available in Collabora Online.

Yes you are right, it was not available, my mistake. So maybe this should be a feature request rather than a bug?

avatar1024 avatar Dec 01 '21 13:12 avatar1024

Can someone change the tag to feature request on this?

avatar1024 avatar Apr 16 '22 09:04 avatar1024

Hi, everybody.

We also need this feature.

We need to protect some columns and cell in the general Calc documents

andrsharov avatar May 26 '22 10:05 andrsharov

Should this be marked as feature request or enhancement instead of a bug?? And do we know if there is any plan to add this function?

avatar1024 avatar Jul 29 '22 08:07 avatar1024

Hi, @avatar1024 .

Yes this is not a bug, this is feature request . Please mark it as feature request.

We need some functionality like Google Sheets , like this https://support.google.com/a/users/answer/9308980?hl=en

andrsharov avatar Jul 29 '22 11:07 andrsharov

@thebearon or @gitgt could you re-check what's the status of this? If it's still valid or if it was implemented. I have replaced the bug tag with enhancement

pedropintosilva avatar Feb 21 '23 09:02 pedropintosilva

Valid enhancement seen in multiple channels.

timur-g avatar Feb 21 '23 10:02 timur-g

The changes have landed. You can protect / unprotect a sheet, and do the same for the cells. There is a visual indicator as well.

In 23.05.9.1 or the one after.

hfiguiere avatar Feb 14 '24 14:02 hfiguiere