Dropshipping

[lesliekirk]

Re: Dropshipping

I did a view source on the code and it's displaying the the

<input name="ShippingMethod" value="flatrate:Drop Shipping" type="radio"> Drop Shipping ($10.00)<br>

after the

<!-- // test all the non-PG methods and display them -->

which would mean (to me) that it's not "obeying" the price group or because there is a shipping method equal to flatrate:Drop Shipping it's going to display it.

[lesliekirk]

Re: Dropshipping

I thought it was working okay when logged in, but it's not. It still display all the shipping methods, except now the price group rate is listed before the methods that shouldn't be displayed.

<!-- // assign method if in PG -->
<input name="ShippingMethod" value="flatrate:Drop Shipping" type="radio"> Drop Shipping ($10.00)<br>
<!-- // test all the non-PG methods and display them -->
<input name="ShippingMethod" value="baseunit:1 - Domestic Routine First Class Mail" checked="checked" type="radio"> 1 - Domestic Routine First Class Mail - U.S. &amp; Puerto Rico Only ($6.00)<br>
<input name="ShippingMethod" value="baseunit:2 - Domestic Priority - U.S. &amp; Puerto Rico Only" type="radio"> 2 - Domestic Priority - U.S. &amp; Puerto Rico Only ($12.00)<br>
<input name="ShippingMethod" value="flatrate:Drop Shipping" type="radio"> Drop Shipping ($10.00)<br>

[wcw]

Re: Dropshipping

Change the OR to AND in this line.
<mvt:if expr="(l.settings:shippingmethod:code NE 'flatrate:Wholesale Shipping') OR (l.settings:shippingmethod:code NE 'flatrate:Drop Shipping')">

[lesliekirk]

Re: Dropshipping

I tried changing the OR to an AND - no change.

I then tried this

<!-- // see if customer is in PG -->
<mvt:item name="toolkit" param="pgroup|pcount" />
<mvt:if expr="g.pcount GT 0">
<mvt:foreach iterator="customer_pgroup" array="customer_pgroups">
<mvt:if expr="l.settings:customer_pgroup:name EQ 'Wholesale'">
<mvt:item name="toolkit" param="sassign|whole|1" />
</mvt:if>
<mvt:if expr="l.settings:customer_pgroup:name EQ 'Dropship'">
<mvt:item name="toolkit" param="sassign|drop|1" />
</mvt:if>
</mvt:foreach>
</mvt:if>

<!-- // assign method if in PG -->
<mvt:if expr="g.whole EQ 1">
<input name="ShippingMethod" value="flatrate:Wholesale Shipping" type="radio"> Wholesale Shipping ($18.00)<br>
<mvt:elseif expr="g.drop EQ 1">
<input name="ShippingMethod" value="flatrate:Drop Shipping" type="radio"> Drop Shipping ($10.00)<br>

<mvt:else>
<!-- // test all the non-PG methods and display them -->
<mvt:foreach array="shippingmethods" iterator="method">

<mvt:if expr="ISNULL g.ShippingMethod AND pos1 EQ 1">
<input type="radio" name="ShippingMethod" value="&mvte:method:module;:&mvte:method:code;" checked="checked" /> &mvt:method:name; (&mvt:method:formatted_price;)<br />
<mvt:elseif expr="g.ShippingMethod EQ (l.settings:method:module $ : $ l.settings:method:code)">
<input type="radio" name="ShippingMethod" value="&mvte:method:module;:&mvte:method:code;" checked="checked" /> &mvt:method:name; (&mvt:method:formatted_price;)<br />
<mvt:else>
<input type="radio" name="ShippingMethod" value="&mvte:method:module;:&mvte:method:code;" /> &mvt:method:name; (&mvt:method:formatted_price;)<br />
</mvt:if>

</mvt:foreach>
                    
            </mvt:if>

It hides the other methods when logged in to the Price Group and works. But when logged out - the flatrate:Drop Shipping still displays as one of the shipping methods available to everyone.

[Bruce - PhosphorMedia]

Re: Dropshipping

you'd have to wrap the for each loop to NOT show the Drop Ship method.

[lesliekirk]

Re: Dropshipping

More along these lines?

<!-- // see if customer is in PG -->
<mvt:item name="toolkit" param="pgroup|pcount" />
<mvt:if expr="g.pcount GT 0">
<mvt:foreach iterator="customer_pgroup" array="customer_pgroups">
<mvt:if expr="l.settings:customer_pgroup:name EQ 'Wholesale'">
<mvt:item name="toolkit" param="sassign|whole|1" />
</mvt:if>
<mvt:if expr="l.settings:customer_pgroup:name EQ 'Dropship'">
<mvt:item name="toolkit" param="sassign|drop|1" />
</mvt:if>
</mvt:foreach>
</mvt:if>

<!-- // assign method if in PG -->
<mvt:if expr="g.whole EQ 1">
<input name="ShippingMethod" value="flatrate:Wholesale Shipping" type="radio"> Wholesale Shipping ($18.00)<br>
<mvt:elseif expr="g.drop EQ 1">
<input name="ShippingMethod" value="flatrate:Drop Shipping" type="radio"> Drop Shipping ($10.00)<br>

<mvt:elseif expr="(l.settings:shippingmethod:code NE 'flatrate:Wholesale Shipping') OR (l.settings:shippingmethod:code NE 'flatrate:Drop Shipping')">
<!-- // test all the non-PG methods and display them -->
<mvt:foreach array="shippingmethods" iterator="method">

<mvt:if expr="ISNULL g.ShippingMethod AND pos1 EQ 1">
<input type="radio" name="ShippingMethod" value="&mvte:method:module;:&mvte:method:code;" checked="checked" /> &mvt:method:name; (&mvt:method:formatted_price;)<br />
<mvt:elseif expr="g.ShippingMethod EQ (l.settings:method:module $ : $ l.settings:method:code)">
<input type="radio" name="ShippingMethod" value="&mvte:method:module;:&mvte:method:code;" checked="checked" /> &mvt:method:name; (&mvt:method:formatted_price;)<br />
<mvt:else>
<input type="radio" name="ShippingMethod" value="&mvte:method:module;:&mvte:method:code;" /> &mvt:method:name; (&mvt:method:formatted_price;)<br />
</mvt:if>

</mvt:foreach>
                    
            </mvt:if>

I "think" I tried this already but if it's what you are suggesting Bruce, I will gladly try it again

[wcw]

Re: Dropshipping

You were supposed to change OR to AND

[Bruce - PhosphorMedia]

Re: Dropshipping

I think this:

<mvt:elseif expr="(l.settings:shippingmethod:code NE 'flatrate:Wholesale Shipping') OR (l.settings:shippingmethod:code NE 'flatrate:Drop Shipping')">

Should be this:

<mvt:elseif expr="(l.settings:customer_pgroup:name NE 'Wholesale' OR l.settings:customer_pgroup:name EQ 'Dropship')">

You want to test WHO the customer IS or ISN'T before running the Methods loop. INSIDE the method loop, you need to make sure WholeSale Shipping and FlatRate Shipping options are blocked.

(If you are only using flat rate methods for the "special" customers, you can use one rule (l.settings:method:module EQ 'flatrate')

[lesliekirk]

Re: Dropshipping

You were supposed to change OR to AND

That didn't help - still displays the Drop Shipping rate when not logged into a Price Group.

[wcw]

Re: Dropshipping

Change it back to OR
On the next line where you had deleted the <mvt:else> put that line back in.

[lesliekirk]

Re: Dropshipping

I think this:

<mvt:elseif expr="(l.settings:shippingmethod:code NE 'flatrate:Wholesale Shipping') OR (l.settings:shippingmethod:code NE 'flatrate:Drop Shipping')">

Should be this:

<mvt:elseif expr="(l.settings:customer_pgroup:name NE 'Wholesale' OR l.settings:customer_pgroup:name EQ 'Dropship')">

You want to test WHO the customer IS or ISN'T before running the Methods loop. INSIDE the method loop, you need to make sure WholeSale Shipping and FlatRate Shipping options are blocked.

(If you are only using flat rate methods for the "special" customers, you can use one rule (l.settings:method:module EQ 'flatrate')

My ignorant question 4,726 of the day, isn't that what this code is doing?

<!-- // assign method if in PG -->
<mvt:if expr="g.whole EQ 1">
<input name="ShippingMethod" value="flatrate:Wholesale Shipping" type="radio"> Wholesale Shipping ($18.00)<br>
<mvt:elseif expr="g.drop EQ 1">
<input name="ShippingMethod" value="flatrate:Drop Shipping" type="radio"> Drop Shipping ($10.00)<br>

So if I change it as Bruce suggest it would become:

<!-- // assign method if in PG -->
<mvt:if expr="g.whole EQ 1">
<input name="ShippingMethod" value="flatrate:Wholesale Shipping" type="radio"> Wholesale Shipping ($18.00)<br>
<mvt:elseif expr="g.drop EQ 1">
<input name="ShippingMethod" value="flatrate:Drop Shipping" type="radio"> Drop Shipping ($10.00)<br>

<mvt:elseif expr="(l.settings:customer_pgroup:name NE 'Wholesale' OR l.settings:customer_pgroup:name EQ 'Dropship')">

<!-- // test all the non-PG methods and display them -->
<mvt:foreach array="shippingmethods" iterator="method">

[lesliekirk]

Re: Dropshipping

Just a follow up - I tried the previous snippet I "thought out loud" on

<!-- // assign method if in PG --> <mvt:if expr="g.whole EQ 1"> <input name="ShippingMethod" value="flatrate:Wholesale Shipping" type="radio"> Wholesale Shipping ($18.00)<br> <mvt:elseif expr="g.drop EQ 1"> <input name="ShippingMethod" value="flatrate:Drop Shipping" type="radio"> Drop Shipping ($10.00)<br>  <mvt:elseif expr="(l.settings:customer_pgroup:name NE 'Wholesale' OR l.settings:customer_pgroup:name EQ 'Dropship')">  <!-- // test all the non-PG methods and display them --> <mvt:foreach array="shippingmethods" iterator="method">

The Drop Shipping rate is still displaying for a non-Price Group customer.

[wcw]

Re: Dropshipping

If your conditional is wrong in the foreach it will show. So to figure out why it is showing, you need to comment out one section at a time to see how it is getting there; ie the part before the foreach, then try commenting out the foreach section.

[lesliekirk]

Re: Dropshipping

Think I'm going to have to start over. I never really removed the <mvt:else> I made it mvt:elseif so now I'm not sure where you are suggesting that I re-add the <mvt:else>

[wcw]

Re: Dropshipping

<mvt:foreach array="shippingmethods" iterator="method">
<mvt:if expr="(l.settings:shippingmethod:code NE 'flatrate:Wholesale Shipping') AND (l.settings:shippingmethod:code NE 'flatrate:Drop Shipping')">

this would not list wholesale or drop

</mvt:if>