[D7]allowed-value-list

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

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.

Hi,
On your install file , I did not understand why are you using hook_disable instead of hook_uninsatll. In most of the cases, hook_uninstall should be used to delete the variables that are created.

Hi drupalbabaji,

Here is my review

1 Basic application checks.

in the .info file Sparxsys Solution is not a package you should use Fields instead.
the title of this issue should be [D7]allowed-value-list ...

2 repository checks.

ok

3 Security .

ok

4 Licensing .

ok

5 Documentation.

you should add a live example because to see how a user can edit the allowed value field is not obvious... I initially got 403 when trying to access the field settings page. when you say "can navigate to PATH" you should write something like avl/field_name/edit instead of PATH
the Wish List Features is clear... but from my point of view greatly limit the use of the module...

6 Coding standards.

pareview doesn't output nothing

7 API

instead of using a separate configuration page to grant permissions this could be done directly on the field setting page by implementing hook_field_instance_settings_form

typo in .admin.inc 155 Permssion edited
+ a lot of others, in comments as well

you should manage to get a fully featured module because to be able to only add new values is not very useful. your module should manage edit and delete items as well, it's a pity.

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.