סליקה באמצעות פייקארד ודרופל

אם אכן זה המודול אז:

1) דבר ראשון תודה רבה על השיתוף, אני חושב שזה רווח לחברה ורווח לקהילה.
2) בדרך כלל כשאני רואה מודולים חדשים, או כאלו שלא קשורים אלי, אני לא חופר בהם, אבל פה אני חושב שכשירות לקהילה חשוב לי להגיד שאני אישית לא הייתי משתמש במודול הזה, ולהלן ההסבר.

כשמתעסקים עם אתרי מסחר אלקטרוני חשוב מאוד, מאוד שהקוד יהיה טוב ומאובטח. במעבר על המודול הנוכחי נתקלתי ביותר מדי מקומות חשודים. אני לא מדבר על עניין הפורמטינג הלא דרופלי -- זה ממש בקטנה, אני לא מתרגש מזה.

אבל למשל uc_allcharge_make_payment() -- שהיא בערך הפונקציה הכי חשובה:
* לא משתמשת ב FAPI, אלא ב HTML הארד קודד.
* יש שורות קוד מיותרות שאין בהן שימוש
* חסר שם איזה check_plain() אחד!
* אני בספק רב אם המודול יעבוד באתרים רב לשוניים או כאלו בלי clean urls, מאחר ואין שימוש בפונקציית url() כדי לגלות נתיבים.

לחילופין ב- uc_allcharge_response(), אין בדיקה אמיתית לגלות אם פיי-קארד אישרו את הטרנזקציה.

בקיצור, צריך לתת את הדעת על מספר עניינים. בכל אופן, אני מדגיש את סעיף אחד -- התודה על השיתוף.