Drupal Commerce DragonPay - Let's you easily use DragonPay as a Payment Gateway with your Drupal Commerce website.

DragonPay provides innovative payment solutions to help merchants accept or disburse payments online. We focus on alternative payment channels such as banks, mobile payments, ATM’s, and brick-and-mortar payment centers. By doing so, we provide our merchants convenience, better protection against fraud, and low transaction fees.

Online Payment Collection
Everybody believes e-commerce will be a major force in retail. The challenge remains in developing countries like the Philippines where the main mode of payment is still cash. With a population of over 100M, the estimated number of Filipinos with credit cards only stands at 5%. It is also estimated that only about 20% to 25% of the population at most have a bank account. How do you get these potential buyers with no credit cards to shop online?

DragonPay solves the problem by extending the online world to the physical world. Customers can buy online from e-commerce merchants and fulfill the payment through non-credit card channels. We currently support online banking of most of the major local commercial banks, physical ATM payments, over-the-counter (OTC) cash payments at banks and retail establishments (supermarkets, groceries, department stores, courier centers, remittance centers). Merchants are provided with a single, unified Application Programming Interface (API) to accept payments from any supported source.

Merchants enjoy several benefits when using DragonPay’s alternative payment collection service:

  • Broad reach to the mass market
  • No chargeback/fraud
  • Low, fixed transaction fees

Project Page: https://www.drupal.org/sandbox/jhayzhon/2369999

Git Clone: git clone --branch 7.x-1.x http://git.drupal.org/sandbox/jhayzhon/2369999.git commerce_dragonpay

PAReview: http://pareview.sh/pareview/httpgitdrupalorgsandboxjhayzhon2369999git

Comments

PA robot’s picture

Status: Active » Needs work

There are some errors reported by automated review tools, did you already check them? See http://pareview.sh/pareview/httpgitdrupalorgsandboxjhayzhon2369999git

We are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then we will take a look at your project right away :-)

Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).

I'm a robot and this is an automated message from Project Applications Scraper.

PA robot’s picture

Status: Needs work » Closed (won't fix)

Closing due to lack of activity. If you are still working on this application, you should fix all known problems and then set the status to "Needs review". (See also the project application workflow).

I'm a robot and this is an automated message from Project Applications Scraper.

mjgruta’s picture

Status: Closed (won't fix) » Needs review

Cleaned up the code.
Please Review.

ziomizar’s picture

Hi jhayzhon,

Thanks for your module

- In pareview still errors http://pareview.sh/pareview/httpgitdrupalorgsandboxjhayzhon2369999git.
you can add the pareview link in project description to help others review your project.

- theme/commerce_dragonpay.theme.css is empty remove it if is not used.

- In commerce_dragonpay.info you only need to add dependencies to commerce and remove other submodules included in commerce package like: commerce_order, commerce_payment, commerce_ui that don't have an own release.

- In configuration page can't you hide the Default currency? Is not possible change its value that is set to "PHP" (is not a currency code) better not expose this to the users.

- In configuration page you have a flag " Log Transaction status " but in the code seems that you save transictions in watchdog every time, without check if the flag is active. See line 43 of commerce_dragonpay.module

- README.txt does not follow the guidelines for in-project documentation and/or the README Template.
You miss requirements, maintainers and raccomanded modules as you include a lot of requirements with commerce package.

ziomizar’s picture

Status: Needs review » Needs work
mjgruta’s picture

Status: Needs work » Needs review

Thanks ziomizar for reviewing my module.

- Cleaned up most of the code.

- Removed css file, planning to add some styles but I guess most of the time it will be overriden by developers.

- In commerce_dragonpay.info, I'm not sure if I really need to remove the other dependencies.

- I'll leave the default currency as it is so that the developer knows that it will only accept the following currency.

- I have included the flag " Log Transaction status "

- Fixed the README.txt

Please review again thanks.

mjgruta’s picture

Issue summary: View changes
vineethaw’s picture

Status: Needs review » Needs work

Hello jhayzhon,

I enabled your module, But could not find the configuration page 'admin/commerce/config/payment-methods' as mentioned in the readme file. Also I am getting an error 'Warning: Illegal string offset 'method_id' in commerce_dragonpay_rule_get_action_settings() (line 119 of /var/www/html/newsletter/sites/all/modules/custom/commerce_dragonpay/commerce_dragonpay.module).'

mjgruta’s picture

Status: Needs work » Needs review

Hi vineethaw,

You need to enable Commerce Payment UI to see the configuration page.

I have added this on the instruction.

ajalan065’s picture

Hi jhayzhon,
I have not gone through the functionality of your module, but have seen your PA-review, and there are no such big issues.
I got a simple issue:
FILE: /var/www/drupal-7-pareview/pareview_temp/commerce_dragonpay.module
---------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
---------------------------------------------------------------------------
104 | ERROR | [x] Closing parenthesis of array declaration must be on a
| | new line

Do fix it.

In README.txt, no need to write the table of contents as it is not mandatory + your Readme is not so long so as to write table of contents.
Rest, I didnot find any problem with your .module file

mjgruta’s picture

Hi Guys,

I have cleaned up README.txt and fixed PA-review error.

Thanks.

danielhonrade’s picture

Status: Needs review » Reviewed & tested by the community
mjgruta’s picture

Any updates here? its been 6 months already. Thanks.

apaderno’s picture

Assigned: mjgruta » Unassigned
Status: Reviewed & tested by the community » Fixed

Thank you for your contribution!

I updated your account so you can promote this to a full project and also create new projects as either a sandbox or a "full" project.

Here are some recommended readings to help with excellent maintainership:

You can find lots more contributors chatting on IRC in #drupal-contribute. So, come hang out and stay involved!

Thank you, also, for your patience with the review process. Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.

Thanks go also to the dedicated reviewer(s) as well.

mjgruta’s picture

Thank you very much Alberto!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.