Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge #36

Merged
merged 51 commits into from Aug 25, 2014
Merged

Merge #36

merged 51 commits into from Aug 25, 2014

Conversation

acidvertigo
Copy link
Owner

No description provided.

acidvertigo and others added 30 commits August 10, 2014 19:37
When $value is an object then we have to check if it has object vars because according to PHP manual "Objects with no properties are no longer considered empty."  http://us1.php.net/manual/en/function.empty.php

So is_null($value) when $value is an object it always return false. So the appropriate way to check is using get_object_vars which will return newProp, but the protected and private members will not be returned.
added isset($SID) to tep_href_link so not to provide: Notice: Undefined variable: SID in...

I know notices are not shown is osCommerce because of :

error_reporting(E_ALL & ~E_NOTICE);

but that does not mean that we don't have to check for those.
All images are now responsive to 100% width of their container.  Can be turned off by setting parameter to false.
Most images need to be aligned center.
Bootstrapped
Change the tep_get_all_get_params function
tep_not_null update using get_object_vars
Issue #600 obsolete file probably born from an anxious young v3 version
added gloabl $SID to tep_href_link so not to produce: Notice: Undefined variable: SID in...
Include tep_parse_input_field_data into tep_output string_function
added isset($SID) to tep_href_link
added gloabl $SID to tep_href_link
Bug Fix - Account Notifications
Set the cookie domain when checking for SSL connection
haraldpdl and others added 21 commits August 23, 2014 15:42
Remove count query in product_info.php and check the total resultset fro...
Use basename instead of strrpos
Move the product model breadcrumb from application top to product_info.php removing a redundant query
avoids +1 query in application top
If  $cPath == 0 in url like this: http://localhost/oscom/product_info.php?cPath=0&products_id=15 bm_categories gives an error
Use basename instead of substr strrpos
Move the product model breadcrumb from application top
Casting int instead to call intval function
Send header "404 not found" when products cannot be found
acidvertigo added a commit that referenced this pull request Aug 25, 2014
@acidvertigo acidvertigo merged commit 9721566 into acidvertigo:patch-4 Aug 25, 2014
acidvertigo pushed a commit that referenced this pull request Mar 25, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants