[Last Call] Learn how to a build a cloud-first strategyRegister Now

x
  • Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 320
  • Last Modified:

Re-protecting a spreadsheet Macro


I have a worksheet that I have protected.  I periodically give users the password in order for them to make changes.  When they click on one of the buttons on the worksheet, I want the spreadsheet to password enable the spreadsheet again.

Sub Reactivate_Password()
    ActiveSheet.Protect Password:="password"
End Sub

I have this macro embedded in the button that takes a user back to another worksheet and that's where I wanted to call this macro.

I thought this to be a simple one.

Thank you in advance.

B.
0
Bright01
Asked:
Bright01
  • 5
  • 4
  • 2
2 Solutions
 
TinTombStoneCommented:
Sub Button_Code

'Stuff here

Reactivate_Password
End Sub
0
 
Bright01Author Commented:
TinTombStone,

I think that's what I've got.

Sub ReturntoQuestionaire_Click()
'This closes the Existing sheet and returns the user to the Questionaire
Sheet42.Visible = xlSheetHidden
Reactivate_Password
Sheet1.Activate
End Sub

It doesn't work however.  What am I missing?

B.
0
 
dlmilleCommented:
Where is this module located?  The ReturnToQuestionnaire really should be in a public module, as well as the Reactivate_Password code, and the button_click code.  All should be public.  If you have them embedded in a sheet's codepage, in different places (a sheet's codepage is visible only to itself, but it can reference public routines).  Perhaps that's the problem.

What line do you get the error?

Here's a sample of your code in a workbook to play this out.  Let me know if it works for you.  See where I placed the code, and see if you did something similar.

Hope this helps!

Dave
resetPassword-r1.xlsm
0
Technology Partners: We Want Your Opinion!

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

 
Bright01Author Commented:
Dave,

Greetings!  The Reactivate_Password sub is in a module.  

Sub Reactivate_Password()
    ActiveSheet.Protect Password:="pass"
'    ActiveSheet.EnableSelection = xlUnlockedCells
End Sub

However, the sub is called from a worksheet sub.

Here's what's in the Worksheet.

Sub ReturntoQuestionaire_Click()
'This closes the Maturity by Category and returns the user to the Questionaire
Sheet42.Visible = xlSheetHidden
Reactivate_Password
Sheet1.Activate
End Sub
0
 
dlmilleCommented:
Try putting the word PUBLIC before the word sub on the reactivate:

Public Sub Reactivate_Password()

Also, its good practice to use CALL in front of procedure calls, so change the line in the ReturnToQuestionaire_Click() to:

Call Reactivate_Password.

I also suggest moving this all to one public module, as in the example I posted.  You don't need mouse click routines in worksheet code pages.  While it may work, it is not good practice to put more code than just event management in worksheet code pages.  Results may not always be guaranteed.

Again, however, I ask, "What line did you get the error?".  Or are you getting any errors?  What are you experiencing?

Dave
0
 
dlmilleCommented:
Also, can you clear your workbook of any confidential data and just upload it?  I can assist more readily, then.

Dave
0
 
Bright01Author Commented:
Dave,

Much thanks for stepping in here.

The spreadsheet works great except when I press the "Return to Questionaire" button, it doesn't relock the sheet.  The code in the specific worksheet is;

Sub ReturntoQuestionaire_Click()
Sheet42.Visible = xlSheetHidden
Call Reactivate_Password
Sheet1.Activate
End Sub

This button returns the user to the Questionaire.

The Module that has the subroutine looks like this;

Public Sub Reactivate_Password()
    ActiveSheet.Protect Password:="pass"
'    ActiveSheet.EnableSelection = xlUnlockedCells
End Sub

I added Pulblic as you suggested and also added "call" to the routine in the spreadsheet.  What's odd is that when I use the button/sub on the worksheet, it does not activate the password or protect.

B.

0
 
dlmilleCommented:
can you upload?
0
 
Bright01Author Commented:
Dave,

Here you go.  It's a mocked up version but I'm certain you will see what I'm trying to do immediately.  I'm simply trying to relock the worksheet (target) when you return to the other (source) worksheet.

Thank you,

B.
Relocking-Spreadsheet.xlsm
0
 
TinTombStoneCommented:

In Sub ReturntoSource_Click()


swap the lines

Sheet2.Visible...
and
Call...


This should work

Sub ReturntoSource_Click()
'This closes the Target and returns to Source
Call Reactivate_Password
Sheet2.Visible = xlSheetHidden
Sheet1.Activate
End Sub
0
 
Bright01Author Commented:
Thank you TinTombStone for providing the code.  It does the job.

Thank you Dave for the comments and lesson on what I should be doing with a global macro.

Great teamwork you guys,

B.
0

Featured Post

Concerto Cloud for Software Providers & ISVs

Can Concerto Cloud Services help you focus on evolving your application offerings, while delivering the best cloud experience to your customers? From DevOps to revenue models and customer support, the answer is yes!

Learn how Concerto can help you.

  • 5
  • 4
  • 2
Tackle projects and never again get stuck behind a technical roadblock.
Join Now