Stock by Attribute v4.0 addon for v1.3.5-1.3.9

[mc12345678]

FWIW, I've updated the code to support the single dropdown option being used across single and multiple attribute options including keeping the customid. I may have found on SBA 1.5.3 for ZC 1.5.1 that in the sba_sequenced_dropdowns that there might have been an extra displayed } in the javascript. I've removed it from the uploaded file so it may resolve an issues seen previously, it may cause it. :) Let me know if this caused an issue. May need to run through the install again on a clean system to see any negative effects.

Also, almost have a general use of READONLY attributes incorporated for say sequenced dropdowns so that their info will still be displayed and in the sequence desired, just not interfering with the remaining operation. :) I say almost, though there is still a bit of work to do and some general testing.

[tronage]

Quick Question:
I'm still trying to resolve the issue with the Winchester_Black template and the 2nd attribute only showing one value. In an effort to fix it, I'm comparing a fresh download of the template with my server files. And I've found two files:
Includes/templates/winchester_black/templates/
tpl_modules_main_product_image.php
tpl_shopping_cart_default.php

That are VERY different (not so much different as more each has parts the other doesn't). Since I've uploaded SBA after winchester_black, if there was an overwrite it would be the SBA file.

So my question is the above files right now I'm using the SBA versions? And maybe I need to be using the Winchester_black? or merge the two? Let me know your thoughts.

[picaflor-azul]

Quick Question:
I'm still trying to resolve the issue with the Winchester_Black template and the 2nd attribute only showing one value. In an effort to fix it, I'm comparing a fresh download of the template with my server files. And I've found two files:
Includes/templates/winchester_black/templates/
tpl_modules_main_product_image.php
tpl_shopping_cart_default.php

That are VERY different (not so much different as more each has parts the other doesn't). Since I've uploaded SBA after winchester_black, if there was an overwrite it would be the SBA file.

So my question is the above files right now I'm using the SBA versions? And maybe I need to be using the Winchester_black? or merge the two? Let me know your thoughts.

If the files in the plugin and the template are the same then you need to merge them.

Thanks,

Anne

[tronage]

UPDATE:

Anne has been a huge help, however her most recent post points back to SBA being issue with my 2nd attribute only showing one value:
https://www.zen-cart.com/showthread....71#post1284771

If you go back one page in her thread, she posted a link to a "working" version (and I tested and verified it worked) But when she changed her settings to how I have mine, and set the stock to zero it broke it. And when using the default template (as the link currently shows here: http://weblyweb.com/sba/index.php?ma...roducts_id=180) it is broke in identically the same way as I'm experiencing. And she said this was a fresh install of everything.

As a "test" I did not change what options I had set for dynamic dropdowns. I picked an item with two attributes, and added stock to all the combinations. sync'ed the quantities, and reloaded the live site. Issue still remained. So I'm guessing it has to be something with the options that I've got set under dynamic dropdowns? Currently set as:

Enable Dynamic Dropdowns	1	 
Product Info Single Attribute Display Plugin	multiple_dropdowns	 
Show Out of Stock Attributes	True	 
Product Info Multiple Attribute Display Plugin	sba_sequenced_dropdowns	 
Mark Out of Stock Attributes	None	 
Display Out of Stock Message Line	False	 
Display Javascript Popup for Out-of-Stock Selection	False	 
Prevent Adding Out of Stock to Cart	False	 
SBA Number of Records to Displayed	25

I'm still using the previous "fix" version... I've not installed the "latest update" you mention above regarding the single dropdown, readonly, etc.

Any ideas?

[mc12345678]

If the files in the plugin and the template are the same then you need to merge them.

Thanks,

Anne

They are each modified from a vanilla install and therefore the changes made by/for each should be evident. Usually such changes/differences are easily integrated and do not fight with each other so should be rather straight forward.

[tronage]

If the files in the plugin and the template are the same then you need to merge them.

Thanks,

Anne

AH you rock Anne! And I'm sure when I flipped to the default template it did not break... I didn't "install" it into the default template. I just turned on the default under tools-template selection... so it was a "false positive". So when properly installed it always breaks with my settings. maybe somewhere in these files the issue lays?

[mc12345678]

UPDATE:

Anne has been a huge help, however her most recent post points back to SBA being issue with my 2nd attribute only showing one value:
https://www.zen-cart.com/showthread....71#post1284771

If you go back one page in her thread, she posted a link to a "working" version (and I tested and verified it worked) But when she changed her settings to how I have mine, and set the stock to zero it broke it. And when using the default template (as the link currently shows here: http://weblyweb.com/sba/index.php?ma...roducts_id=180) it is broke in identically the same way as I'm experiencing. And she said this was a fresh install of everything.

As a "test" I did not change what options I had set for dynamic dropdowns. I picked an item with two attributes, and added stock to all the combinations. sync'ed the quantities, and reloaded the live site. Issue still remained. So I'm guessing it has to be something with the options that I've got set under dynamic dropdowns? Currently set as:

Enable Dynamic Dropdowns	1	 
Product Info Single Attribute Display Plugin	multiple_dropdowns	 
Show Out of Stock Attributes	True	 
Product Info Multiple Attribute Display Plugin	sba_sequenced_dropdowns	 
Mark Out of Stock Attributes	None	 
Display Out of Stock Message Line	False	 
Display Javascript Popup for Out-of-Stock Selection	False	 
Prevent Adding Out of Stock to Cart	False	 
SBA Number of Records to Displayed	25

I'm still using the previous "fix" version... I've not installed the "latest update" you mention above regarding the single dropdown, readonly, etc.

Any ideas?

I think the version previously loaded has an extra curly bracket in the javascript display of the sba_sequenced_dropdown.php class file. That extra curly bracket was supposed to be commented out in the earlier version (not an issue in SBA 1.5.4) but in comparing the two versions did not get corrected before.

Short of it, fixed in the above download link. This is also why requested to test the various conditions to identify the "cause" it was the DD file for sba_sequenced_dropdowns that I loaded for SBA 1.5.3 to support ZC 1.5.1.

[mc12345678]

Found it.

In the condition provided, the function stkmsg is removed from the page; however, calls to that function were not removed... Updated a new copy of the includes/classes/pad_sba_sequenced_dropdowns.php file.

At line: 247 add:

if ($this->out_of_stock_msgline == 'True') {

Then at new line 249 add:

}

Then at new line 262 add:

if ($this->out_of_stock_msgline == 'True') {

and again now at new line 264 add:

}

This will remove the error that is caused by trying to call a non-existent function that is between each pair of new code.

so 247 through 249 will look something like:

if ($this->out_of_stock_msgline == 'True') {
  $out.=" stkmsg(frm);\n";
}

[mc12345678]

As usual, Anne certainly has gone above and beyond to provide support.

While the above code changes will solve the immediate issue, on the surface I don't consider it an ideal solution. My basis for that statement is the extra PHP necessary to prevent the javascript error that appeared and the need to "remember" that anywhere the stkmsg call is made instead of one time within the javascript section itself. That is further to consider that the less javascript presented to a guest, the shorter the load and operation time. So, it may actually be a faster/better solution, but seems like a near mess on the code side.

Basically have code that is writing code is a way to think about it. (PHP writing javascript.) Would be nice to keep it simple enough to allow others to update/modify/etc., but complex enough to give the necessary/desired result(s).

Just expressing some thoughts and thankful for the testing/reporting.

[tronage]

Salvation at last!
All seems to be working now. Just need to turn on some side boxes that Anne had me turn off, but I don't see them interfering. Your changers worked, at least for now if/when a better solution is created as you mention. Thanks so much!!!!!

Also a quick question: the drop downs for the sequenced drops are appearing in a box with a white background, and the template is primarily a black background. Could you point me in the right direction to which file or area I should be looking at to change this background to black and the text to white?
Thanks!