Chris Brown

> I wonder if the sql_cache_flush_cache 'memory' case should clear `$this->cache_array` just in case someone implements 'memory' fully in the future. Correct observation; however I'd leave that change for when...

I'd prefer to exclude the `entry_` prefix.

What's the purpose of adding a "contact name" field, when there's already first-name and last-name? You'd mentioned that *your* primary goal for this is shipping-related, mainly to add a shipping...

> what about the showcase sites? I'm not sure I follow whatever you meant to say here.

@dbltoe I think you're saying "there are some sites that do operate entirely without SSL". Yes I agree. Even in a localhost test environment some will not use SSL. That's...

The old set of defines existed primarily to allow the use of "Shared SSL Certificates", which is a largely dead concept today. Years ago often SSL was only available by...

> @barco57 wrote: > I opted for the ZCA Bootstrap template (my favorite) and OPC to get the guest checkout. However I ran into a problem, to use guest checkout...

I added that in https://github.com/zencart/zencart/pull/1516 It was born from the desire to extract repeated functionality like: ```php $tmp_value = zen_db_prepare_input($_POST['products_quantity']); $products_price = (!zen_not_null($tmp_value) || $tmp_value=='' || $tmp_value == 0) ?...

Your point is heard fully. It's still an anti-pattern which should be avoided as much as possible. A penny-off solution should be part of core. I feel like we're now...

That is correct. The thinking was: it's reasonable to parse the provided files on initial install, because it's clear that they're for install-time-only. I suppose a naming convention for upgrade-only...