Stock For Product With Different Attributes (OLD)

Quote:

---

Originally posted by afx114@Jan 24 2006, 11:30 PM
Actually I'm finding out that an On/Off switch is pretty useless, because the code already handles everything normally whether or not there are products with attribute stock entered.

BigWil, what were you hoping to accomplish by making this module toggleable?

---

afx114,

Sorry I missed this post until I was reviewing the progress just now. What I intended by the toggle was to turn it off so that the core code revisions wouldn't go into effect if not turned on. Thus it wouldn't effect the stock attribute operations. If it already doesn't then thats great but a toggle would still be convenient so as to not confuse the store owner with the extra admin screens.

I swear I had some sort of problems trying to do stock operations with this installed though. I will have to go back and test again.

Any idea of when the new release will be available? I saw some cool modifications by Grayson that you might also want to include.

Cheers,

BigWil

I found a problem with my code a few posts back to display the stock quantity for each attribute next to that attribute: it breaks text attributes (because they don't have a quantity). Here's the fix.

In includes/modules/pages/product_info/main_template_vars_attributes.php, replace

Code:

---

    $sql = "select  pov.products_options_values_id,
            pov.products_options_values_name,
            pwas.quantity,
            pa.*
        from  " . TABLE_PRODUCTS_ATTRIBUTES . " pa, " . TABLE_PRODUCTS_OPTIONS_VALUES . " pov, " . TABLE_PRODUCTS_WITH_ATTRIBUTES_STOCK . " pwas
        where  pa.products_id = '" . (int)$_GET['products_id'] . "'
        and    pa.options_id = '" . (int)$products_options_names->fields['products_options_id'] . "'
        and    pa.options_values_id = pov.products_options_values_id
        and    pwas.stock_attributes = pa.products_attributes_id
        and    pov.language_id = '" . (int)$_SESSION['languages_id'] . "' " .
        $order_by;

---

with

Code:

---

    if ($products_options_names->fields['products_options_type'] != PRODUCTS_OPTIONS_TYPE_TEXT && $products_options_names->fields['products_options_type'] != PRODUCTS_OPTIONS_TYPE_FILE && $products_options_names->fields['products_options_type'] != PRODUCTS_OPTIONS_TYPE_READONLY) {
    $sql = "select  pov.products_options_values_id,
            pov.products_options_values_name,
            pwas.quantity,
            pa.*
        from  " . TABLE_PRODUCTS_ATTRIBUTES . " pa, " . TABLE_PRODUCTS_OPTIONS_VALUES . " pov, " . TABLE_PRODUCTS_WITH_ATTRIBUTES_STOCK . " pwas
        where  pa.products_id = '" . (int)$_GET['products_id'] . "'
        and    pa.options_id = '" . (int)$products_options_names->fields['products_options_id'] . "'
        and    pa.options_values_id = pov.products_options_values_id
        and    pwas.stock_attributes = pa.products_attributes_id
        and    pov.language_id = '" . (int)$_SESSION['languages_id'] . "' " .
        $order_by;
    } else {
    $sql = "select  pov.products_options_values_id,
            pov.products_options_values_name,
            pa.*
        from  " . TABLE_PRODUCTS_ATTRIBUTES . " pa, " . TABLE_PRODUCTS_OPTIONS_VALUES . " pov
        where  pa.products_id = '" . (int)$_GET['products_id'] . "'
        and    pa.options_id = '" . (int)$products_options_names->fields['products_options_id'] . "'
        and    pa.options_values_id = pov.products_options_values_id
        and    pov.language_id = '" . (int)$_SESSION['languages_id'] . "' " .
        $order_by;
    }

---

This uses the original query for text, readonly, and file type attributes, and uses the new query with quantity for dropdowns, checkboxes, and radio buttons.

Quote:

---

Originally posted by studmuffin@Feb 15 2006, 11:33 PM
guys

installed this mod, its cool thx but when an order is placed it takes the stock off the product stock rather than the attributes stock.

i see it says to edit orders.php but i cant find it, and dont know what to edit.

can anyone help a newbie pls

thx x :)

---

Go into the contribution files you downloaded, open up classes/order.php, and copy over the code changes beginning at line 645 or thereabouts. They are clearly marked as additions for this contribution. Copy over everything down through the "end of change" marker.

Quote:

---

Originally posted by studmuffin
hmm, i double checked what i did, and i copied order.php over the top of my origonal so this code should be there :/

any ideas x

thanks again

---

Did you copy classes/order.php over? The instructions refer to "classes/orders.php" but that's a typo. It may be confusing you since there's also an admin/orders.php file. (And an admin/includes/classes/order.php -- that's NOT the one you want to modify.)

Another thought: this mod will still take the number of items sold off the product stock (as that is the sum of the stock for all attributes). Are you sure it isn't also decrementing the individual attribute stock?

yeah i guessed it was a typo, its replaced the correct one - includes/classes/order.php.

if i do a test order it only updates the total stock amount not the attribs, so eg the attribs will say red = 50 blue = 50 but the total will be 99 cos 1 just got bought...... ' hope i explained that ok...

here is my order.php

http://netfinity.co.uk/order.zip

sorry to be a pain :) thx for any input...

studmuffin, I compared your order.php to mine and there are no appreciable differences between your attributes with stock code and mine. (Just a couple of extra spaces at the end of one or two lines, but I don't see how that could make a difference.)

The problem lies elsewhere....

Quote:

---

Originally posted by grayson@Feb 16 2006, 08:41 AM
I found a problem with my code a few posts back to display the stock quantity for each attribute next to that attribute: it breaks text attributes (because they don't have a quantity). Here's the fix.

In includes/modules/pages/product_info/main_template_vars_attributes.php, replace

Code:

---

## ## ## ##$sql = "select ## ##pov.products_options_values_id,
 ## ## ## ## ## ## ## ## ## ## ## ## ##pov.products_options_values_name,
 ## ## ## ## ## ## ## ## ## ## ## ## ##pwas.quantity,
 ## ## ## ## ## ## ## ## ## ## ## ## ##pa.*
 ## ## ## ## ## ## ## ##from ## ## ##" . TABLE_PRODUCTS_ATTRIBUTES . " pa, " . TABLE_PRODUCTS_OPTIONS_VALUES . " pov, " . TABLE_PRODUCTS_WITH_ATTRIBUTES_STOCK . " pwas
 ## ## ## ## ## ## ## ##where ## ## pa.products_id = '" . (int)$_GET['products_id'] . "'
 ## ## ## ## ## ## ## ##and ## ## ## pa.options_id = '" . (int)$products_options_names->fields['products_options_id'] . "'
 ## ## ## ## ## ## ## ##and ## ## ## pa.options_values_id = pov.products_options_values_id
 ## ## ## ## ## ## ## ##and ## ## ## pwas.stock_attributes = pa.products_attributes_id
 ## ## ## ## ## ## ## ##and ## ## ## pov.language_id = '" . (int)$_SESSION['languages_id'] . "' " .
 ## ## ## ## ## ## ## ##$order_by;

---

with

Code:

---

## ## ## if ($products_options_names->fields['products_options_type'] != PRODUCTS_OPTIONS_TYPE_TEXT && $products_options_names->fields['products_options_type'] != PRODUCTS_OPTIONS_TYPE_FILE ##&& $products_options_names->fields['products_options_type'] != PRODUCTS_OPTIONS_TYPE_READONLY) {
 ## ## ## ##$sql = "select ## ##pov.products_options_values_id,
 ## ## ## ## ## ## ## ## ## ## ## ## ##pov.products_options_values_name,
 ## ## ## ## ## ## ## ## ## ## ## ## ##pwas.quantity,
 ## ## ## ## ## ## ## ## ## ## ## ## ##pa.*
 ## ## ## ## ## ## ## ##from ## ## ##" . TABLE_PRODUCTS_ATTRIBUTES . " pa, " . TABLE_PRODUCTS_OPTIONS_VALUES . " pov, " . TABLE_PRODUCTS_WITH_ATTRIBUTES_STOCK . " pwas
 ## ## ## ## ## ## ## ##where ## ## pa.products_id = '" . (int)$_GET['products_id'] . "'
 ## ## ## ## ## ## ## ##and ## ## ## pa.options_id = '" . (int)$products_options_names->fields['products_options_id'] . "'
 ## ## ## ## ## ## ## ##and ## ## ## pa.options_values_id = pov.products_options_values_id
 ## ## ## ## ## ## ## ##and ## ## ## pwas.stock_attributes = pa.products_attributes_id
 ## ## ## ## ## ## ## ##and ## ## ## pov.language_id = '" . (int)$_SESSION['languages_id'] . "' " .
 ## ## ## ## ## ## ## ##$order_by;
 ## ## ## ##} else {
 ## ## ## ##$sql = "select ## ##pov.products_options_values_id,
 ## ## ## ## ## ## ## ## ## ## ## ## ##pov.products_options_values_name,
 ## ## ## ## ## ## ## ## ## ## ## ## ##pa.*
 ## ## ## ## ## ## ## ##from ## ## ##" . TABLE_PRODUCTS_ATTRIBUTES . " pa, " . TABLE_PRODUCTS_OPTIONS_VALUES . " pov
 ## ## ## ## ## ## ## ##where ## ## pa.products_id = '" . (int)$_GET['products_id'] . "'
 ## ## ## ## ## ## ## ##and ## ## ## pa.options_id = '" . (int)$products_options_names->fields['products_options_id'] . "'
 ## ## ## ## ## ## ## ##and ## ## ## pa.options_values_id = pov.products_options_values_id
 ## ## ## ## ## ## ## ##and ## ## ## pov.language_id = '" . (int)$_SESSION['languages_id'] . "' " .
 ## ## ## ## ## ## ## ##$order_by;
 ## ## ## ##}

---

This uses the original query for text, readonly, and file type attributes, and uses the new query with quantity for dropdowns, checkboxes, and radio buttons.

---

so would this typo fix allow for the stock number to be shown in the dropdown menu?

Yahaira - no, it won't fix the display of stock for dropdown menu attributes. It fixes a problem with text and file attributes, which don't have any associated quantity, and therefore were not being selected by the SQL query I used.

hmmmz

any ideas....i'm at a loss :(

x ::frust