Closed (duplicate)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
26 Feb 2014 at 15:04 UTC
Updated:
9 Sep 2018 at 22:08 UTC
Jump to comment: Most recent
Comments
Comment #1
robin.ingelbrecht commentedComment #2
robin.ingelbrecht commentedComment #3
contactvishaljain commentedHi robin.ingelbrecht,
PAReview is reporting issue on the code. Please see http://pareview.sh/pareview/httpgitdrupalorgsandboxrobiningelbrecht22056....
Comment #4
robin.ingelbrecht commentedI solved the issues.
http://pareview.sh/pareview/httpgitdrupalorgsandboxrobiningelbrecht2205641git
Comment #5
robin.ingelbrecht commentedI solved the issues
http://pareview.sh/pareview/httpgitdrupalorgsandboxrobiningelbrecht2205641git
Comment #6
robin.ingelbrecht commentedComment #7
robin.ingelbrecht commentedComment #8
robin.ingelbrecht commentedComment #9
robin.ingelbrecht commentedProject Review:
https://drupal.org/comment/8523709#comment-8523709
Comment #10
PA robot commentedWe 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 #11
gobinathm@vishal_yug, please perform full manual review.
@robin.ingelbrecht suggestion on the application request.
1. Please update your project description to include an non-maintainer version of GIT repo clone command.
2. Project applications with Review bonus takes priority. So please complete manual reviews & include those url's here. Once done, please include appropriate tags in the application.
Note: the above comments are not based on project review. Those are simple suggestion on project application.
Comment #12
robin.ingelbrecht commentedUpdated project description to include an non-maintainer version of GIT repo clone command.
Comment #13
mehul.shah commentedHi Robin,
1. In validation anchor_menu_items_validate
You are validation only 1st part of link path instead you have to check the whole path after .
2. Condition for 2 anchor tags is not handled?
Ex. //abc
3. Condition for anchor tag at other position i.e. except 1st position is not handled
Ex. abc/ OR abc/xyz/
Comment #14
robin.ingelbrecht commentedComment #15
robin.ingelbrecht commentedHi mehul.shah,
1. In validation anchor_menu_items_validate
I validate everything except for the last element because that is the anchor element. For example: admin/structure/blocks#anchor => admin/structure/blocks gets validated
2. Condition for 2 anchor tags is not handled?
You can't use 2 anchor tags on one url?
3. Condition for anchor tag at other position i.e. except 1st position is not handled
The module expects
<anchor>/admin/structure/blocks/firstblock as input and it will format the link as admin/structure/block#firstblock.So the last element is always the anchor element, the first is a placeholder so that drupal knows it's an anchor link and everything between is the link itself.
Comment #16
robin.ingelbrecht commentedupdated description :
Integrates with scroll_to_destination_anchors
Comment #17
simon.westyn commentedNo objections, everything looks good and clean!
Works perfectly!
Comment #18
pingwin4eg@simon.westyn
You may not set status to RTBC without reviewing the application.
@robin.ingelbrecht
Can you explain please (on the project's page) what is so extra your module does that Drupal core does not provide by default? "#" sign can be used normally in menu items.
Comment #19
slydevil commentedRobin, I hate to tell you this, but there is another module that will allow you to do this very thing.
1.2 Ensure your project is not a duplication
Check out the Menu Token module: https://www.drupal.org/project/menu_token
Not only does it allow you to add anchors, but it also allows query strings and tokens.
Comment #20
pingwin4egPostponed, according to last 2 comments. Please, add a description of how your project differs from others.
Comment #21
robin.ingelbrecht commentedThis project is an duplicate of https://www.drupal.org/project/menu_token.
You can remove it...
Comment #22
PA robot commentedProject 1: https://www.drupal.org/node/2346321
Project 2: https://www.drupal.org/node/2205693
As successful completion of the project application process results in the applicant being granted the 'Create Full Projects' permission, there is no need to take multiple applications through the process. Once the first application has been successfully approved, then the applicant can promote other projects without review. Because of this, posting multiple applications is not necessary, and results in additional workload for reviewers ... which in turn results in longer wait times for everyone in the queue. With this in mind, your secondary applications have been marked as 'closed(duplicate)', with only one application left open (chosen at random).
If you prefer that we proceed through this review process with a different application than the one which was left open, then feel free to close the 'open' application as a duplicate, and re-open one of the project applications which had been closed.
I'm a robot and this is an automated message from Project Applications Scraper.
Comment #23
avpaderno