No Guests : Customer Login required like Admin Login

[Nigel]

Comparing Zipurman's video and ecartz' code, the error on line 3 seems to be the inclusion of open and close parenthesise at the end of the class declaration.
Removing them allows it to work straight away.

Code: Select all

class hook_shop_siteWide_loginRequired {

not

Code: Select all

class hook_shop_siteWide_loginRequired() {

Is this the correct solution or should I do something else ?

[zipurman]

That is correct. Class syntax should not include (). Good catch.

[Nigel]

I've been away from the software for a while but started again this weekend.
I lost my mother to a severe stroke and it takes your mind to a different place and changes your perspective of what's important and your priorities.

The login required seems to work well but the What's new side bar shows the price of the item when not logged in.
Can anyone suggest what I should do please ?

[zipurman]

I've been away from the software for a while but started again this weekend.
I lost my mother to a severe stroke and it takes your mind to a different place and changes your perspective of what's important and your priorities.

The login required seems to work well but the What's new side bar shows the price of the item when not logged in.
Can anyone suggest what I should do please ?

So sorry to hear about your mother. My condolences.

You could edit the includes/modules/boxes/bm_whats_new.php

You could change add the following to the function execute() {

Code: Select all

function execute() {

  if (($GLOBALS['customer'] ?? null) instanceof customer) {

** leave all the code in here ***

   }
  
}

[Nigel]

I've been away from the software for a while but started again this weekend.
I lost my mother to a severe stroke and it takes your mind to a different place and changes your perspective of what's important and your priorities.

The login required seems to work well but the What's new side bar shows the price of the item when not logged in.
Can anyone suggest what I should do please ?

So sorry to hear about your mother. My condolences.

You could edit the includes/modules/boxes/bm_whats_new.php

You could change add the following to the function execute() {

Code: Select all

function execute() {

  if (($GLOBALS['customer'] ?? null) instanceof customer) {

** leave all the code in here ***

   }
  
}

Sorry for the delay in response.

Thanks re. my mum.

Should this be added as a user template by creating modules/boxes/bm... ?
Furthermore, is your if statement surrounding everything in the execute function ?

[zipurman]

Yes, you would EDIT the file includes/modules/boxes/bm_whats_new.php

Then wrap the entire execute logic (everything inside the function) with the if statement.

[Nigel]

Yes, you would EDIT the file includes/modules/boxes/bm_whats_new.php

Then wrap the entire execute logic (everything inside the function) with the if statement.

Thanks.
Second attempt worked fine.