Printable View
Yeah, I ran the install/update script before posting my question. I'm glad I didn't miss anything obvious. I'll keep digging and perhaps try your tests but I hate to start all over again with a fresh zc install.Quote:
Originally Posted by mc12345678
Thank you for all that you do for us!
Dave
Did you try running it a second time? That's really what the above testing is/was about in my opinion. Another thought is had you logged out and then logged back in (refreshing session information) to see if the data became available?Quote:
Originally Posted by Dave224
In that testing I used the "fresh" install perspective because of the information provided in the two previous posts that had indicated the problem. Seemed for them that running the install twice took care of the issue. They were set on it being a browser issue to which I disagree. The testing is all about identifying which situation is true. If there's a "need" to run it twice, then I need to figure out what operation is not reaching the database at the right time with the right information the first time around. Now that I think about it, I wouldn't be surprised if there is some database caching issue where a select query is run one time before an install step and then again after the install step and the same data is returned which is that the action taken didn't happen.
Ok, I may have found the issue considering I only found one recent code change that may have caused installation not to complete. I would have expected some sort of message to be provided about it (in the list of content on the install page), but not sure that gets reviewed for what was done/not done...
I've uploaded a patch that I think corrects the installation issue: Correct installation · mc12345678/Stock_By_Attributes_Combined@45d01a9 (github.com)
This change appears to be the only change that may have had an effect on operation and may be the reason that a portion of the installation did not finish. Still would be nice to know basically what the last output line was on the install screen.
I found one problem with my install...I had missed installing a few new files. Got that fixed. Ran install again. I now have the SBA menu item in admin under catalog. But I get a fatal error of unknown column popt.products_options_track_stock when looking at product info page for SBA product.
I put debug statements in stock_by_attr_install.php and discovered that $action=null when first starting the install process, which brings up the info screen and script selection box which was expected. But when selecting a script and clicking run script, nothing happens except a return to admin home. No $action variable is set, leading me to believe the installer script(s) were not run.
There is nothing SBA-related displayed at the bottom of the install page or admin home page. I ran installer twice, same result. I installed SBA all over again with the patch (March 18 version). Same results.
I'm using Firefox on Mac. Tried Chrome on Mac...same results.
Dave
zc157c, SBA 1.5.4 (March 18 GITHUB version), php 7.3.21
So as far as the fileset on github, that field 'products_options_track_stock' is only in three files and really is not used by the code at least from a settings perspective. The three files are:Quote:
Originally Posted by Dave224
admin/stock_by_attr_install.php
includes/classes/pad_base.php
includes/classes/pad_sba_sequenced_dropdowns.php
So yes, if the database install is not successful, then accessing a catalog side product information page of a product tracked by SBA will begin to generate issues. There is no catalog side "detection" to prevent this. There shouldn't really be, because if the admin installation is successful, then the information expected on the catalog side will be there for use.
As to the issue with information following the "action", the dropdown is within a form that uses $_GET to provide information. Therefore selecting an item from the dropdown list and pressing Run Script, assigns the selected item to the URI parameter selectSBAinstall, so it and then '$action' will be set to whatever was selected.
I can not seem to reproduce the problem described above with the patch applied.
Performed test 1 as you recommended in the referenced post. Same results as before. There is no output from the install script. All that happens is the admin home page comes up after clicking "run script". No change on second run of script. No difference when running the check files script. The SBA menu item is not placed in the admin/catalog menu.
I installed a clean (unmodified) zc version 1.5.7c, installing the test data. I loaded all the SBA files in the March 18 version from Github that were in the admin and includes folders and the files in the 1.5.7 folder. The YOUR TEMPLATE files were placed in the template_default folder and the classic green template selected. I did not install the Dutch or French language files. No merging of the five files that were different from the base zc files was done; the SBA files were written over the 1.5.7c files.
I logged in to admin and added stock_by_attr_istall.php to the url and hit return. The help/select script page comes up as expected (Stock By Attribute (SBA) installation script SBA Version 1.5.4 for Zen Cart Version 1.5.7c). I selected the Full/Upgrade DB install script and clicked run script.
The url in the browser admin home page url window following running the script was my_domain/admin/index.php?cmd=home&selectSBAinstall=installAll&getSBAinstallPage=Run+Script if that helps.
I'm running with Mac OS 10.15.7 (Catalina) with Firefox (86.0.1) on a local development MAMP Pro 6.0.1 server running php 7.3.21, Apache 2.4.46, and MySql 5.7.30.
Hope this helps. What configuration are you using that won't reproduce the problem?
Dave
I think there may be something wrong with the github distribution. Admin/includes contains folders and files that are or should be in the store-side includes folder. There are files in admin that are not shown in the list of new admin files or new core files shown in the install help page for zc 157c.
Dave
Please provide specifics. Github location, files that are perceived to be incorrectly placed. What led to this conclusion.Quote:
Originally Posted by Dave224
The following files are included in the SBA zc157c distribution but not listed on the SBA install help page:
admin/includes/auto_loaders/config.eo_sba.php
admin/includes/auto_loaders/config.products_with_attributes_stock.php
admin/includes/auto_loaders/config.zc154_compatibility.php
admin/includes/classes/observers/class.products_with_attributes_stock.php
admin/functions/extra_functions/products_with_attributes_stock.php
admin/includes/functions/extra_functions/reg_ddsba.php
admin/includes/init_includes/init_eo_sba.php
admin/includes/init_includes/init_sba_copy_to_confirm_messagestack.php
admin/includes/init_includes/init_sba_copy_to_confirm/php
admin/includes/init_includes/init_zc154_compatibility.php
admin/includes/languages/english/edit_orders_sba.php
admin/includes/languages/english/extra_definiotions/reg_ddsba.php
admin/includes/languages/english/images/buttons/button_sba_link.gif
admin/includes/languages/english/modules/product_sba.php
includes/classes/class.products_with_attributes_class_stock.php
includes/classes/observers/auto.advanced_search_categories_custom_id.php
includes/classes/pad_base.php
includes/classes/pad_multiple_dropdowns.php
includes/classes/pad_sba_sequenced_dropdowns.php
includes/classes/pad_sba_single_dropdown.php
includes/classes/pad_sequenced_dropdowns.php
includes/classes/pad_single_dropdown.php
includes/classes/pad_single_radioset.php
includes/extra_configures/pwas_non_stock.php
includes/extra_datafiles/dynamicdropdownfor sba.php
includes/languages/english/extra_definitions/dynamic_dropdowns_sba.php
includes/templates/YOUR_TEMPLATE/templates/tpl_modules_attributes.php
includes/templates/YOUR_TEMPLATE/templates/tpl_main_product_image.php
The following files are shown on the install help page as changed or updated core files but are not included in the zc 1.5.7c SBA distribution (Are they listed for other zc versions?):
admin/options_name_manager.php
admin/options_values_manager.php
admin/includes/functions/general.php
admin/includes/classes/order.php
includes/classes/order.php
includes/functions/functions_lookups.php
includes/functions/extra_functions/products_with_attributes.php
includes/modules/pages/checkout_shipping/header_php.php
includes/modules/pages/shopping_cart/header_php.php
The following override files are listed in the SBA install help page but not included in the SBA zc157c distribution:
includes/modules/YOUR_TEMPLATE/attributes.php
The distribution is Stock_By_Attributes_Combined-master.zip located at https://github.com/mc12345678/Stock_...ads/master.zip with date of March 18, 2021. Files used are in the admin, includes and 1_5_7 folders.
My intention is to try and get to the bottom of my SBA install difficulties. My thought was that there may be extra or missing files in the distribution that are messing up the install after studying the install help page. I realize I'm grasping at straws.
Dave
So, the incomplete list of files on the install screen... that definitely isn't a factor of the problem with the install. That's a problem with updating the list of files in that area to match the reality of the code as cross referenced with the ZC version in use (assuming that the reported version actually matches the fileset as distributed for that version, etc...).Quote:
Originally Posted by Dave224
I do know this. In ZC 1.5.7, the admin side of ZC changed to using a "cmd" parameter to represent the page that is being used/accessed. I don't currently have the background of why it was implemented, what problems it resolved, etc... but it has become a part of the core code.
There are a few pieces throughout that exist to support it and keep it working the way intended. One of which that helps is for things like installations to actually happen basically through natural navigation of the admin. Either through the modules->plugins manager, or by being logged in and accessing say a configuration menu...
Well, the way SBA was written for installation, was to not cause any changes to the store *until* the install was accessed. When the install is accessed, the configuration menu is supposed to be generated. Sort of, not here until you see it situation. Yes, there are files associated with that part of the installation process which is why anywhere I have discussed installation has been to load all of the files in the includes and admin folders to their respective location (a few are template related because they have been the same for all associated ZC versions and have not (yet) deserved being somewhere separate/copied multiple times). Then, where a specific ZC version is considered, to use the files only from that version (some have thought they had to "build" up to the current version of something). In some cases, because of improvements to the ZC core code, less has needed to be modified about that code to support the features offered.
So, anyways, one of the "factors" of the new cmd parameter, is a sort of chicken/egg condition of the parameter already being in the browser uri and being set to the current page... I actually maybe describing that slightly incorrect as it is a bit of a brain twister of what needs to be set to what, before/after this/that in order for operation to continue as likely expected/desired.
If I may suggest, if the configuration option exists for SBA after any of these tests that one navigate to it from somewhere in the store and then attempt the install script. I need to disable a few things on my test store that might be preventing this problem to see if I have to add/incorporate some work around for ZC 1.5.7. I had previously reported an issue with zen_draw_form when using a 'get' type form, though don't recall if it was resolved or remains open. It was reported in github and offered a sort of work around but its something that should at some point be corrected.
I will say this though too. Not all files that are included with the plugin are "mandated" for operation. Some offer an improved convenience of the drop down restrictions, but the central aspect of the software works just fine without it or with some variation. There is at least one file that is listed/present that really is only there because it is now an empty file where previous versions had the file there...
I still need to look into/at operation or the various ways that this issue could be generated to either provide approriate guidance to support the install process or to modify the install script to prevent the problem even if it is to apply a work around.
I think I'll go away for a while to give you time to find the solution. I'm afraid I don't understand everything you posted, but please let me know if you need more information from me.
Thank you for all you do for zen cart!
Dave