Project Link: https://www.drupal.org/sandbox/jaskaran.nagra/2470461

Git Clone : git clone --branch 7.x-1.0-alpha1 http://git.drupal.org/sandbox/jaskaran.nagra/2470461.git restws_alias

PAReview: http://pareview.sh/pareview/httpgitdrupalorgsandboxjaskarannagra2470461git-7x-10-alpha1

This module allows RESTful Webservices (using restws module) to be accessible through url aliases in addition to just the direct path.

For example: If you have a node with direct path "node/1" and url alias "page/awesomepage", the restws module allows you to access json representation of the node through "node/1.json" and not "page/awesomepage.json".

Read more discussion here: https://www.drupal.org/node/1311788

This module allows you to access the node through "page/awesomepage.json" by adding new url aliases such as:
page/awesomepage.json ==> node/1.json
page/awesomepage.xml ==> node/1.xml
page/awesomepage.rdf ==> node/1.rdf

Please Note:
To generate these aliases for the nodes already existing in the system, please bulk update the aliases by going to:
Configuration -> URL aliases. Then Delete and Bulk update in succession.

Manual reviews of other projects:

  1. https://www.drupal.org/node/2469639#comment-9818817

Comments

jaskaran.nagra’s picture

Issue summary: View changes
jaskaran.nagra’s picture

Issue summary: View changes
jaskaran.nagra’s picture

Priority: Major » Normal
jaskaran.nagra’s picture

Issue summary: View changes
jaskaran.nagra’s picture

Issue summary: View changes
PA robot’s picture

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.

jaskaran.nagra’s picture

Issue summary: View changes
jaskaran.nagra’s picture

Issue summary: View changes
jaskaran.nagra’s picture

Issue summary: View changes
jaskaran.nagra’s picture

Issue summary: View changes
jaskaran.nagra’s picture

alibama’s picture

I'm using this on http://dev.bioconnector.virginia.edu and confirm that http://dev.bioconnector.virginia.edu/content/introduction-pubmed-online-... now links correctly as does http://dev.bioconnector.virginia.edu/content/introduction-pubmed-online-... - very cool - I've been looking for this functionality for almost a year now, and am very grateful for the work that jaskaran.nagra has done here :)

alibama’s picture

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

Issue summary: View changes
Issue tags: -PAreview: review bonus

Removing review bonus tag, you have not done all manual reviews, you just posted the output of an automated review tool. Make sure to read through the source code of the other projects, as requested on the review bonus page.

alibama’s picture

@klausi - my bad, while I am using this on 2 production sites, I haven't completed the review bonus work yet. thanks for the feedback

jaskaran.nagra’s picture

Oh... Thanks Klausi and alibama.

I thought I would wait for them to fix the pareview errors before manually going through the code but it makes perfect sense to remove the PA review bonus tag for now.

So, how does it work? the RTBC tag remains?

Thanks again for your help !

heddn’s picture

Once a project is moved to RTBC, then a git administrator has the chance to perform a final review. There are a lot of applications in the backlog, so one way we use to prioritize things is to use the bonus tag.

heddn’s picture

  1. This doesn't follow code standards. Inline comments should begin with only slashes, even if they are multi-line comments i.e. //
    function restws_alias_path_add_format($path, $format) {
    
      /*
       * Check if any format is added to the path.
       * Not checking just $format to avoid .json.rdf.xml type paths.
       */

However, that isn't a blocker. Thanks for your contribution, Jaskaran Nagra!

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.

jaskaran.nagra’s picture

YAY !!

Thanks heaps heddn. You are amazing!!

I have removed the multi-line comments and started a new project. Very excited :)

Will see you on the IRC. Thanks alibama and klausi for your help too.

Cheers~

jaskaran.nagra’s picture

Status: Reviewed & tested by the community » Closed (fixed)
klausi’s picture

Status: Closed (fixed) » Fixed

Let's leave it a t fixed, the issue will close automatically in 2 weeks.

Status: Fixed » Closed (fixed)

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