CVS edit link for quazardous

I have coded an "Extended search page" module on Top of Search API (http://drupal.org/project/search_api).

The goal is to provide a page like the default "Find content" but on top of scalable technology for sites with huge amounts of nodes.

So this module support solr and mysql through Search API and uses entity metadata to discover objects fields.

it features :

- non fulltext index can be used as filters (like in the content page) in the search page form

- you can choose a form widget for each filter

- thoses widgets are shown in the search page and managed by URL (vs session for SEO)

- suported fields : boolean, date with date_range, node_reference (with mono term select)

- widgets support can be extended by a hook

next will be a "Use as default content manager" and a Field/CCK widget integration for entity_reference (ie a lighbox node browser)

Comments

quazardous’s picture

Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new14.85 KB
brianV’s picture

Status: Needs review » Needs work

Hello, and thank you for submitting a CVS application.

Before your module or theme is reviewed, I would like to present this checklist which covers 90% of the types of errors that reviewers are forced to reject an application for. Please make sure the items on the checklist are all taken care of prior to setting your application back to 'Needs review'.

  1. Please run your module through the Coder module code review. This will pick up a lot of coding standards, function misuse, security, and internationalization issues. See http://drupal.org/project/coder
  2. Please make sure all non-binary files have an @file documentation block as per http://drupal.org/node/1354#files
  3. Please make sure that your code commenting complies with the Drupal Doxygen guidelines as shown at http://drupal.org/node/1354 . This ensures that the API module and external sites like drupalcontrib.com can properly parse your code's documentation.
  4. Please make sure that the proper CVS $Id$ tags are added to all files as per http://drupal.org/handbook/cvs/faq. This requirement will be going away once we move from CVS to Git next year.
  5. Please make sure all your text strings are wrapped in t(). This makes them translateable. See http://api.drupal.org/api/drupal/includes--common.inc/function/t/6
  6. Remove any commented out code.
  7. Make sure your module or theme is ready to be released in it's submitted form. Modules that don't work, or aren't ready to be used are immediately rejected.
  8. Please don't include a 'version' line in your .info file. This is always added to the .info file by the Drupal.org CVS packaging script.
  9. Please remove all copyright statements from your code. Once you accept a patch from another user, your copyright statement is no longer valid anyways.
  10. Please remove any mention of code license - all code submitted to Drupal's CVS is automatically licensed under the GPLv2.
  11. Third party libraries are not allowed into CVS, except in certain conditions. Please see the 3rd party libraries policy at http://drupal.org/node/422996 .

Once you have double checked all the above items, please set your application status to 'needs review'. If you made any changes to the code, please make sure to upload the updated module as well.

quazardous’s picture

Status: Needs work » Needs review
StatusFileSize
new14.16 KB

pass through coder : 227 -> 0 normal warning

quazardous’s picture

StatusFileSize
new24.24 KB

new features... in case u didn't start the review !

there is 1 warning left but it comes from a copied/reworked function from node module ;p

quazardous’s picture

StatusFileSize
new32.94 KB

new sub module ext_search_node_ref : autocomplete over Search API

avpaderno’s picture

Issue tags: +Module review
quazardous’s picture

hi,

I've created the project and committed to CVS

http://drupal.org/project/ext_search_page

best regards

zzolo’s picture

Component: Miscellaneous » miscellaneous
Status: Needs review » Postponed

Hi. Please read all the following and the links provided as this is very important information about your CVS Application:

Drupal.org has moved from CVS to Git! This is a very significant change for the Drupal community and for your application. Please read the following documentation on how this affects and benefits you and the application process:
Migrating from CVS Applications to (Git) Full Project Applications

  • The status of this application will be put to "postponed" and by following the instructions in the above link, you will be able to reopen it.
  • Or if your application has been "needs work" for more than 5 weeks, your application will be marked as "closed (won't fix)". You can still reopen it, by reading the instructions above.
avpaderno’s picture

Issue summary: View changes
Status: Postponed » Closed (won't fix)

As per previous comment, I am setting this issue as Won't fix.
Since now users can create full projects, applications have now a different purpose and they are handled on a different issue queue. See Apply for permission to opt into security advisory coverage for more information.