- Oct 18, 2012
-
-
Moshe Kaplan authored
-
Moshe Kaplan authored
2. Removing all static variable (using cache variable_get())
-
- Oct 17, 2012
-
-
Moshe Kaplan authored
-
Moshe Kaplan authored
1. removing the dSeroCache 2. Adding comments
-
- Oct 14, 2012
-
-
Moshe Kaplan authored
-
- Oct 13, 2012
-
-
Moshe Kaplan authored
-
Moshe Kaplan authored
-
- Oct 07, 2012
-
-
Moshe Kaplan authored
-
Moshe Kaplan authored
-
Moshe Kaplan authored
-
Moshe Kaplan authored
-
Moshe Kaplan authored
1. You added functions inside the .info file. Those functions should go inside .module or .install files 2. When you define a class the file should be referenced on the .info file 3. On line 154 of .module the title should be inside the t() function 4. You are executing code directly on the module file, from line 234 to the end of the file. I've worked with some modules until now and I've never seen this. I'm quite sure you can find hooks to execute that code at the right moment. 5. line 236. drupal_set_html_head is a Drupal 6 function. Your module is Drupal 7, do you need to test if that function exists ?
-
- Oct 06, 2012
-
-
Moshe Kaplan authored
-
Moshe Kaplan authored
1. fixing dsero_anti_adblock_for_google_adsense_install() + enable() is aninstallation hook and must be declared in an install file 2. Matching unique names
-
http://ventral.orgMoshe Kaplan authored
-
http://ventral.orgMoshe Kaplan authored
-
Moshe Kaplan authored
2. Modifying file names 3. Implementing code review suggestions
-
- Oct 05, 2012
-
-
Moshe Kaplan authored
-