Disable COD Payment

[mdo82]

Thanks for your detailed explanation.
I am indeed begun wrong with this topic.
Now I had my information in the first post to fit, but it will be even more confusing.
Maybe I should start a new topic for clarity?

I will briefly explain what I want to do anything to summarize.
I want to remove the moneyorder option for a customer.

In the beginning of the topic I started about cod, but I had the wrong translation for myself.
The change I would therefore run the file moneyorder.php

If I put the following code in my file:

      if (strstr($_SESSION['customers_id'],'3739')){
	$this->enabled = false;
      }

The option for selecting "moneyorder" is not showing at all if i put (!strstr in the code.
Without the !strstr the option is showing for everyone.
Including for customer 3739.

I hope that now have more clearly put down.
Problem for me also is the translation.

[RodG]

Maybe I should start a new topic for clarity?

No, it's probably best to leave it here,

If I put the following code in my file:

      if (strstr($_SESSION['customers_id'],'3739')){
    $this->enabled = false;
      }

The option for selecting "moneyorder" is not showing at all if i put (!strstr in the code.
Without the !strstr the option is showing for everyone.
Including for customer 3739.

This is telling me that the 'if' check isn't functioning as expected.

Try this:
Find the line of code that reads:

  $this->enabled = ((MODULE_PAYMENT_MONEYORDER_STATUS == 'True') ? true : false);

Then add the following code immediately after it

  if(($_SESSION['customer_id']) == "3739")  $this->enabled = false ;

I've confirmed/verified that this works. There is no need to use 'strstr' because you are looking for an exact match. If you do use the 'strstr' ) and assuming it works) it would *also* match customerID's '123739' and '3739123' as well as all other customers with '3739' as *part* (substring) of their Id, which *isn't* what you want.

Cheers
RodG

[mdo82]

No, it's probably best to leave it here,



This is telling me that the 'if' check isn't functioning as expected.

Try this:
Find the line of code that reads:

  $this->enabled = ((MODULE_PAYMENT_MONEYORDER_STATUS == 'True') ? true : false);

Then add the following code immediately after it

  if(($_SESSION['customer_id']) == "3739")  $this->enabled = false ;

I've confirmed/verified that this works. There is no need to use 'strstr' because you are looking for an exact match. If you do use the 'strstr' ) and assuming it works) it would *also* match customerID's '123739' and '3739123' as well as all other customers with '3739' as *part* (substring) of their Id, which *isn't* what you want.

Cheers
RodG

Thank you for your clear explanation.
It now works as I want

Thank you!