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

update code to use standard Zen Cart functionallity #7

Closed
wants to merge 2 commits into from
Closed

update code to use standard Zen Cart functionallity #7

wants to merge 2 commits into from

Conversation

Zen4All
Copy link

@Zen4All Zen4All commented Feb 17, 2016

Review on Reviewable

@@ -213,20 +213,20 @@
/* @var $epdlanguage_query type array */
//$epdlanguage_query = $db->Execute("SELECT languages_id, name FROM ".TABLE_LANGUAGES." WHERE code = '".DEFAULT_LANGUAGE."'");
if (!defined(DEFAULT_LANGUAGE)) {
$epdlanguage_query = ep_4_query("SELECT languages_id, code FROM " . TABLE_LANGUAGES . " ORDER BY languages_id LIMIT 1");
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This style of query offers in house error management and does not crash to resolve the issue by use of the ZC /cache or /logs directory or some other "outside" location, but instead provide a debug file that is readily available "on screen" for review.

@Zen4All
Copy link
Author

Zen4All commented Feb 17, 2016

Ok, I understand your point of view, so I'll leave the rest of the code alone. Though in my opinion it over complicates the code 😉

@Zen4All Zen4All closed this Feb 17, 2016
@mc12345678
Copy link
Owner

What we do for backwards compatibility... There WILL come a time when the "option" won't be available to use mysql_ type codes. I almost changed to ZC style queries when upgrading, but then I remembered the times I made an error in an import file and was thankful for the error logging introduced by chadd...

Yes there are ZC plugins that can support, but didn't want to take this "backwards" or force a different style of usage. Besides that, as a benefit of how the errors are handled, processing continues even if one "row" has an error in it rather than fatally stopping.

@mc12345678
Copy link
Owner

Also, don't get me wrong there may be other code improvements to be made and I appreciate the previous fix. Certainly would welcome a dutch or other translation if you are able. Particularly am interested in an italian translation seeing as the italian group won't allow me to have an account with. Them to request the same from there.

@Zen4All
Copy link
Author

Zen4All commented Feb 18, 2016

I'll see what I can do about the Dutch translation. For my Italian is as bad as my French and Spanish 😆 . Maybe you could ask 100asa(https://www.zen-cart.com/member.php?31874-100asa) he is italian

mc12345678 pushed a commit that referenced this pull request Apr 25, 2016
To return files to the status that mostly was done at least twice before.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants