Integration of Linkit module with Internal nodes module, Which allows you to select the nodes from Available node and you can set the url as your 301 redirect URL so, you don't need to go back and forth to check for node and nid. you can type the title and find the node that will give you full url.

Sandbox Path:
https://www.drupal.org/sandbox/developermitesh/2316333

Git clone

git clone --branch 7.x-1.x http://git.drupal.org/sandbox/developermitesh/2316333.git internal_nodes_linkit

Comments

miteshmap’s picture

Title: Internal Nodes Linkit » [D7] Internal Nodes Linkit
PA robot’s picture

Status: Needs review » Needs work

Git clone command for the sandbox is missing in the issue summary, please add it.

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.

miteshmap’s picture

Issue summary: View changes
miteshmap’s picture

Status: Needs work » Needs review
miteshmap’s picture

Issue summary: View changes
PA robot’s picture

Status: Needs review » Needs work

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

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

miteshmap’s picture

Status: Needs work » Needs review

Automated Review

Here is the result url, no error found.!
http://pareview.sh/pareview/httpgitdrupalorgsandboxdevelopermitesh231633...

miteshmap’s picture

Issue summary: View changes
th_tushar’s picture

Status: Needs review » Needs work

Hi,

I have manually reviewed your code, it seems that you are setting a drupal variable but not deleting them during un-installation. So, make sure you delete them else may create a garbage variables in the drupal system.

Changing the status to "Needs Work", you can revert it back after making the changes.

miteshmap’s picture

Status: Needs work » Needs review

Added hook_uninstall to delete variables.

Michael Hodge Jr’s picture

Status: Needs review » Reviewed & tested by the community

Automated Review

Parreview.sh found some issues, see: http://pareview.sh/pareview/httpgitdrupalorgsandboxdevelopermitesh231633... The big thing is you need a newline at the end of the .install file. Otherwise you are good to go.

Manual Review

I went through your code and didn't see any glaring issues that needed to be fixed. I'm going to set this to RTBC.

miteshmap’s picture

Updated .install file added a newline at the end of the file.

kscheirer’s picture

Status: Reviewed & tested by the community » Fixed

The code is short, but shows a good understanding of the Drupal API. Additionally checked through some other projects by the author (jquery_date_range) and found good code there too. No other issues found.

Thanks for your contribution, developermitesh!

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.

miteshmap’s picture

Thanks very much for your updates. The project is now moved to - https://www.drupal.org/project/internal_nodes_linkit
Looking forward to make jquery_date_range a full project. I will make updates on it and solve the pending issues soon and apply for the full project.

kscheirer’s picture

Status: Closed (duplicate) » Fixed

Sorry, the bot got confused. @developermitesh you already have "git vetted user" status, and can publish your own projects. Closing the other application instead.

Status: Fixed » Closed (fixed)

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