DrupalGap

Could you please check and solve the code style issues found by PAReview? (You can use this tool for re-checking your code.)

Good question - I've just checked a post in the support section about the same problem. Basically the bot means that there should be no "extra" blank lines at the end of the file. (Maybe the best way to check this if you're confused is to open the file in a hex editor and look for 0A values. At the end of each file, you should see only one; for example in the README I found two.)

should we need to install Drupalgap from the Android Market .
I am trying to connect the drupal site with an Android emulator all ready Phonegap have been installed

can any one can send me the proper connectivity link of android mobile App with drupal site .

how to enable drupaalgap if i don't have saved in

site/all/modules/ drupalgap

HI Tyler.frankenstein,

Could you tell me how to enable DrupalGap module in my drupal site? Where can I download it?

i have followed the link http://www.tylerfrankenstein.com/drupalgap/module-installation

Is that necessary to enable session authentication?

i have done mostly everything in that link but problem in troubleshooting

i could not Enable 'Session authentication' but , services are installed
but i am getting like this:
Session authentication
Services has no settings available.

i have a doubt in giving the name:

Path to endpoint :...................... can be any thing or should be DrupalGap

Name: my_drupal_services /* i have given any name or should be the my_drupalsite name */
The unique ID for this endpoint. /* give me example */

Server: REST

Path to endpoint: my_services /* it should be like this or i can give any name like DrupalGap */

Debug mode enabled: optional

Session authentication: checked

There is no DrupalGap module because the file u r ask me to enable is DrupalGap dont have installation file is not there & Readme file is empty on there in Drupalgap app folder .
please see this link
http://drupal.org/node/1423558

I downloaded, installed and tested the module for Drupal 7. Everything appeared to be functional and working in any of my tests, basically I couldn't get the module to break, regardless of input. Much of the pain in getting things working was getting eclipse setup, but this was due to poor documentation on the android site. Once my android development environment (which isn't required for this module), setup, setting up drupalgap was a breeze. The only reason you would need this environment would be if you wanted to extend the mobile application side, but wouldn't be required on the web server side.

I could see this application being extremely useful as more things start moving towards mobile, especially considering the move towards mobile for drupal 8, this module can help backfill some of the legacy systems by providing an interface to talk to say drupal 6 or 7 which maybe not as mobile friendly as drupal 8.

This module seems very far along just for where it is now. I was able to post content, login, edit content, post comments, etc. From a developer or content contributor it seems very easy to extend, so if you say wanted a mobile app to approve comments, doing so with this module would be extremely easy. One of the nice things the developer did as well was built this around the services module which makes it even easier to extend with other modules.

All updates appeared to be atomic, in that if you updated the permissions on the webserver, say this person no longer is able to post content, then they will no longer be able to post from the phone.

Note: Also verified code meets Drupal Coding Style Standards:
http://ventral.org/pareview/httpgitdrupalorgsandboxsignalpoint1414540git

Added note about iOS.

Review of the 7.x-1.x branch:

• Drupal Code Sniffer has found some issues with your code (please check the Drupal coding standards). See attachment.

This automated report was generated with PAReview.sh, your friendly project application review script. You can also use the online version to check your project. Get a review bonus and we will come back to your application sooner.

manual review:

• The drupalcs error in the Javascript file indicates that you should put the contents of the if statement on a separate line.
• _drupalgap_resource_content_types_list(): do not concatenate variables directly into DB queries, use place holders instead. This can cause SQL injection vulnerabilities. See http://drupal.org/writing-secure-code
• drupalgap_install(): do you really have to create your endpoint here? The Services documentation says "Endpoints are exportable and can be packaged as features (with or without the features module).". So there must be a hook that should hold your default endpoint.
• t($documentation_msg): always pass literal strings to t() so that translations tools can pick the string up from the source code. You should avoid passing variables to t() where possible.
• drupalgap_init(): why do you need hook_init()? This is called on literally every page request, are you sure that you need your javascript on every page?
• drupalgap_views_api(): Views API version 2 does not exist in Drupal 7.
• drupalgap_status(): do not break a translatable sentence like that. Use a placeholder instead and wrap that in one t() call so that it makes sense for translators.

Removing review bonus tag, you can add it again if you have done another 3 reviews of other projects.

manual review:

• drupalgap_status(): why do you need to add that jquery snippet? You should make use Drupal.behaviors in you JS file, "Any function defined as a property of Drupal.behaviors will get called when the DOM has loaded." http://drupal.org/node/304258 see also http://drupal.org/node/756722

Otherwise this looks RTBC to me. Removing review bonus tag, if there are still changes required you can add it again if you have done another 3 reviews of other projects.

Assigning to tim.plunkett as he said he might have time to look at some RTBC issues.

Forgot to set it to RTBC.

Here's a final clean-up patch, please make these changes before promoting.

That said, the code looks clean, and I think it's more than ready for promotion.
Use your new powers wisely, and if you can, please continue to help reviewing other project applications.

thankyou you for you drupalgap module tyler.frankenstein

with you help we have created a phone app using phonegap .
added save and load module in drupalgap
and created new module.

so that the user data's from mobile can be stored in database and
that dates can bee viewed on the site for which we have created new module
to display with the help of views

for an example in the app they are 8 fields if you give values of first
two fields which should be must if the data are stored once it will get that
the date's of remaining fields and display it back to the form.

The screen short i cant upload because if 8.8 MB Space is required.

Removed link to project home page now that the link is entered in a field on the sandbox project node.