Call up Macro's Not working

[Lew_SJH]

Hi there,

Currently have two codes I'm trying to get to work together

And

And trying to get them to run together using

But upon opening the file it doesnt work and I receive the Compile Error, having had a look around the fix seems to be to rename the module, but even renaming them something really random doesnt seem to work and keeps throwing this error up,

Does anyone have any solutions?

Thanks!

[Domski]

You can only have one set of workbook event code in a workbook at a time. You need to combine all your workbook open events into one etc.

Dom

[Lew_SJH]

Thanks Dom,

this is my first attempt at VBA so im not the greatest, how do i go about combining it all?

I've tried inserting both formulas into "thisworkbook" but keep getting the ambiguous compile error message?

Thanks

Lewis

[Domski]

Quite an ambitious first attempt

As an example you've got your two workbook open events:

Please Login or Register  to view this content.

Which should be combined to:

Please Login or Register  to view this content.

You would need to do the same with your BeforeClose event codes depending on the order you wish the code to execute.

As to the location of your code all workbook/worksheet event codes should go on the relevant ThisWorkbook or Sheet code pages. Pretty much everything else should go in a regular module. Some info here that explains it.

Dom

[Lew_SJH]

Quite an ambitious first attempt

As an example you've got your two workbook open events:

Please Login or Register  to view this content.

Which should be combined to:

Please Login or Register  to view this content.

You would need to do the same with your BeforeClose event codes depending on the order you wish the code to execute.

As to the location of your code all workbook/worksheet event codes should go on the relevant ThisWorkbook or Sheet code pages. Pretty much everything else should go in a regular module. Some info here that explains it.

Dom

Brilliant thanks for that, will give it a go now, thanks for the help (so far I've got it working exactly how I want it to but Macro's are having to be enabled at all times which isnt the best) so will be good to get the force macro working aswell

One other question, My director (the one who asked me to try and make this work) will need access to all the sheets, is there a simple way of re-jigging the "case" structure to allow more than one sSName?

I've seen the example PASSWORDFORM4 file but think I would struggle adapting that code to my code giving this is my first attempt!

Thanks again Dom

[Domski]

Sorry, which code are you trying to change to reflect that?

Dom

[Lew_SJH]

Sorry its in the User Form Section

As you can see I have everyone set up to view their own respective sheets and the main "sales board" sheet, but the user named Rod is required to view all of the sheets, if that makes sense?

[Domski]

One method you might want to consider is using the user's logon ID to unhide the sheets rather than a name or password.

If you name the worksheets with the user's logon ID's a simple sub like this would do the trick:

Please Login or Register  to view this content.

For your existing code you could just set a case for your boss and call your ShowAllSheets macro.

Dom

[Lew_SJH]

thanks!

will test and report back