wrote:
Hi Gord
Thanks again for your help.
For the sake of simplicity , I'll keep two versions of my wb - one with
the limitation and one without. I'll do any development work on one and
only add the routines below once it's ready to go. I'll save it then in
a different name , so I always keep the development one unrestricted for
my own purposes.
I assume , in any case , that I can have the macro always in place in
the wb , and it will only be activated when I put the code into the
Thisworkbook module. In that way , I can do development work and only
place the code in the Thisworkbook module as the final thing when it's
ready to go.
Not sure what you mean by protecting the project under VBA Project
Properties.
I take your point that it's not foolproof security by any means , and
can be circumvented , but I do think it will be enough for my purposes.
(As a thought though - once the 30 day trial is over , and the wb is no
longer accessible , what is to stop the user simply reinstalling the
original file over the top of the existing one and having another 30
days?)
^_^
Colin
To get rid of the hidden name "Expiration Date" you could run this macro.
Sub foo()
For Each Name In ThisWorkbook.Names
Name.Visible = True
Next
End Sub
Now go to Insert>Name>Define and delete the name which you will see refers to
the expiry date.
Save the workbook which is now ready for the name to be re-created when you
next open it.
If user is sophisticated enough he will be able to run a similar macro and
change the date from 90 days to 9000 days but if he is that savvy then he will
have defeated your original code anyway.
If by "locked out" you mean you have already made the workbook read-only,
disabling macros will not change the read-only property.
Just save as a copy of the original after deleting the name then disable macros
if you want to do more editing of code. You may have to do this several times
before you final copy is ready.
When ready for distribution make sure you have deleted the name and protected
the project under VBA Project Properties.
Gord
On Sat, 1 Dec 2007 22:47:18 +0000, Colin Hayes
<
[email protected]>
wrote:
Hi Gord
OK Thanks for that. I've put it all in place and all seems well.
I do note from the article on cpearson.com that he says :
'This procedure, TimeBombWithDefinedName, uses a hidden defined name to
store the expiration date. The first time the workbook is opened, that
defined name will not exist and will be created by the code. **(Be sure
that you delete this name when you are done with your own development
work on the workbook.)'**
I'm not clear as to how I would go about deleting the hidden defined
name , or where I would find it.
Or should I just replace all references to 'Expiration Date' with some
other term and let it start again?
I assume also that if I were to lock myself out during development , I
could re-open the wb by disabling macros in excel. Would that work?
Thanks Gord for your help.
Thisworkbook module is accessed under the Excel Icon left of "File" on the
Menu
Bar after Right-click and "View Code".
On third thought, the msgbox should p[robably be wrapped inside the "If"
statement
ThisWorkbook.ChangeFileAccess xlReadOnly
MsgBox "This workbook has become readonly"
End If
End Sub
The stored value is in each workbook and is not common to all.
The stored value does not get overwritten after the first opening of the
workbook.
The name "Expiration Date" holding the start date is created and if it exists,
it is not re-created.
Keep reading Chip's instructions to get an idea of how all this works.
Gord
On Sat, 1 Dec 2007 14:08:33 +0000, Colin Hayes
<
[email protected]>
wrote:
Hi Gord
OK Thanks for that.
Is the 'Workbook_Open event procedure in the ThisWorkbook code module'
under the first tab on the worksheet , or under the Excel symbol top
left of the screen? I always confuse those. I'll put the code to run the
macro there.
Also , if I have the code in several different workbooks , will the
routine not over-write it's stored values , or is each record kept
separately for each wb when the code is in place?
Thanks again Gord
On second thought, place your msgbox after the readonly save has taken
place
if
you feel you need a message.
ThisWorkbook.ChangeFileAccess xlReadOnly
End If
MsgBox "This workbook has become readonly"
End Sub
Make sure you read all of Chip's page including the caveats about security
weaknesses in Excel and how to make sure that users will enable macros
when
they
open the workbook.
Gord
On Fri, 30 Nov 2007 21:31:15 -0800, Gord Dibben
<gorddibbATshawDOTca>
wrote:
Colin
Scroll down to the bottom of the page at Chip's site and he provides
Workbook_Open code to start the macro.
As far as a message goes, I guess you could add that here.
If CDate(Now) >= CDate(ExpirationDate) Then
If NameExists = False Then
MsgBox "This workbook will now become read-only"
ThisWorkbook.Save
End If
ThisWorkbook.ChangeFileAccess xlReadOnly
End If
Why wait until tomorrow....change your system date.
Gord
On Sat, 1 Dec 2007 02:19:24 +0000, Colin Hayes
<
[email protected]>
wrote:
Hi Gord
OK Thanks for your help. I've inserted the module into my wb with this
as the code :
Private Const C_NUM_DAYS_UNTIL_EXPIRATION = 1
Sub TimeBombMakeReadOnly()
''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
' TimeBombMakeReadOnly
' This procedure uses a defined name to store the expiration
' date and if the workbook has expired, makes the workbook
' read-only.
''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
Dim ExpirationDate As String
Dim NameExists As Boolean
On Error Resume Next
ExpirationDate = Mid(ThisWorkbook.Names("ExpirationDate").Value, 2)
If Err.Number <> 0 Then
'''''''''''''''''''''''''''''''''''''''''''
' Name doesn't exist. Create it.
'''''''''''''''''''''''''''''''''''''''''''
ExpirationDate = CStr(DateSerial(Year(Now), _
Month(Now), Day(Now) + C_NUM_DAYS_UNTIL_EXPIRATION ))
ThisWorkbook.Names.Add Name:="ExpirationDate", _
RefersTo:=Format(ExpirationDate, "short date"), _
Visible:=False
NameExists = False
Else
NameExists = True
End If
''''''''''''''''''''''''''''''''''''''''''''''''''''''
' If the today is past the expiration date, make the
' workbook read only. We need to Save the workbook
' to keep the newly created name intact.
''''''''''''''''''''''''''''''''''''''''''''''''''''''
If CDate(Now) >= CDate(ExpirationDate) Then
If NameExists = False Then
ThisWorkbook.Save
End If
ThisWorkbook.ChangeFileAccess xlReadOnly
End If
End Sub
i hope this is how it's meant to be. I haven't changed anything ,
although I made it one day so that I can see if it works by tomorrow.
I have a small question :
I really need this routine to start counting down as soon as the wb is
opened for the first time , on any machine. Does the macro have to be
run to set it working on each machine? If so , it sort of defeats the
object...!
Also , could you suggest some code to bring up a message box to
show
the
wb has expired?
Thanks
Colin
Dibben
See in-line responses..............
On Fri, 30 Nov 2007 13:35:38 +0000, Colin Hayes
<
[email protected]>
wrote:
Hi
OK Thanks. I thought I'd try the one below.
I have a couple of questions though , if you could advise ;
The workbook it's protecting is called 'Receipts' - do I need to put
this anywhere in the code?
No. Thisworkbook is the workbook you are running the Sub on.
If I use this on more than one wb , do I need to change the name of
the
file it stores , so as not to overwrite it.
No changes necessary as far as I can tell.....Thisworkbook refers to
whatever
workbook the code is in.
I assume the file it is
storing is 'Expiration date'. It's not clear to me where it stores the
file....
Expiration Date is a named range which holds the date to expire. The
code
creates this named range.
The line...........Private Const C_NUM_DAYS_UNTIL_EXPIRATION =
90
must
be
placed above the line..........Sub TimeBombMakeReadOnly()
Also , I'm not sure where I should be placing the code in the
workbook.
Under the tab at the bottom , or under the Excel symbol at top-left of
the sheet...?
Alt + F11 to open VB Editor
CTRL + r to open Project Explorer if not visible.
Right-click on your workbook and Insert>Module.
Paste code into that module with the edits above.
Save the workbook then run the macro which sets the time to expire
as
90
days
from when you run the macro.
Have you downloaded the sample workbook from Chip's site to see
how
he
has
done
it?
Gord
Sub TimeBombMakeReadOnly()
''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
' TimeBombMakeReadOnly
' This procedure uses a defined name to store the expiration
' date and if the workbook has expired, makes the workbook
' read-only.
''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
Dim ExpirationDate As String
Dim NameExists As Boolean
On Error Resume Next
ExpirationDate = Mid(ThisWorkbook.Names("ExpirationDate").Val ue,
2)
If Err.Number <> 0 Then
'''''''''''''''''''''''''''''''''''''''''''
' Name doesn't exist. Create it.
'''''''''''''''''''''''''''''''''''''''''''
ExpirationDate = CStr(DateSerial(Year(Now), _
Month(Now), Day(Now) + C_NUM_DAYS_UNTIL_EXPIRATI ON
))
ThisWorkbook.Names.Add Name:="ExpirationDate", _
RefersTo:=Format(ExpirationDate, "short date"), _
Visible:=False
NameExists = False
Else
NameExists = True
End If
''''''''''''''''''''''''''''''''''''''''''''''''''''''
' If the today is past the expiration date, make the
' workbook read only. We need to Save the workbook
' to keep the newly created name intact.
''''''''''''''''''''''''''''''''''''''''''''''''''''''
If CDate(Now) >= CDate(ExpirationDate) Then
If NameExists = False Then
ThisWorkbook.Save
End If
ThisWorkbook.ChangeFileAccess xlReadOnly
End If
End Sub
Thanks again.