Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
This module, based on PHP-IBAN library, implements a validation plugin for Field validation module.
The project is available since 2013 at https://www.drupal.org/sandbox/encelado/1936168 and has already been subject to some contributions by the community.
To further contribute:
git clone --branch 7.x-1.x https://git.drupal.org/sandbox/encelado/1936168.git iban_validator
cd iban_validator
PAreview:
http://pareview.sh/pareview/httpgitdrupalorgsandboxencelado1936168git
Comments
Comment #2
PA robot CreditAttribution: PA robot commentedThere are some errors reported by automated review tools, did you already check them? See http://pareview.sh/pareview/httpgitdrupalorgsandboxencelado1936168git
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.
Comment #3
encelado CreditAttribution: encelado commentedNon-compliant comments are going to be fixed in the next commit.
Comment #4
fishfree CreditAttribution: fishfree as a volunteer commentedSome paraview.sh warnings can be solved easily. For example:
Inline comments must end in full-stops, exclamation marks, or question marks.
The closing brace for the class must have an empty line before it.
Manual review:
You'd better change PHP-IBAN library URL from googlecode to github, both in iban.module file and field_validation_iban_validator.inc file, although it redirects automatically.
Comment #5
encelado CreditAttribution: encelado commentedDocumentation just fixed.
Comment #6
m.bdour CreditAttribution: m.bdour as a volunteer commentedAutomated Review
Issues reported by PAReview.sh: http://pareview.sh/pareview/httpgitdrupalorgsandboxencelado1936168git
Manual Review
The module was installed and tested and it worked as it shall do in a perfect way.
This review uses the Project Application Review Template.
As for this issue summery it needs to have the following information in separate lines:
Comment #7
klausiThe class name and README style alone is surely not an application blocker, anything else that you found or should this be RTBC instead?
Comment #8
m.bdour CreditAttribution: m.bdour as a volunteer commented@klausi
There is no issue reported about the README file, this issue already follows guidelines for in-project documentation.
The only issue that needs a higher reviewer role to check is the duplicate issue. There is already a module for IBAN field (https://www.drupal.org/project/iban_field).
The module was installed, tested and worked perfectly.
Comment #9
m.bdour CreditAttribution: m.bdour as a volunteer commentedSorry I changed the issue status by mistake.
Comment #10
encelado CreditAttribution: encelado commentedPlease consider that by extending a generic and widely used validation module with a very specialized external library my module takes a completely different approach, with clear benefits and no duplication of effort.
Comment #11
gisleWe don't block applications over duplication. However, it is considered polite to inform users about alternatives. Please take a look at the project page template and consider adding the section "Similar projects and how they are different".
There are still some style/naming issues reported by PAreview, but nothing that blocks promotion.
At 116 lines (including comments and blanks) and 5 functions, it just passes the bar for length and complexity.
There are IMHO no more blockers. Moving to RTBC. Note that promotion will not happen until a git administrator has given this a second set of eyeballs.
Comment #12
encelado CreditAttribution: encelado commentedOn the occasion of the first commit on the 8.x-1.x branch I updated the project page with the references to similar projects.
Thank you all for the time spent on reviewing this project.
Comment #13
MiSc CreditAttribution: MiSc as a volunteer commentedThanks for your contribution, encelado!
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!
Thanks, 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 to the dedicated reviewer(s) as well.