Expose the fields from biblio to search_api

This module provides functionality for exposing the fields in biblio to search_api for indexing.

It has been tested using search_api_solr, but presumably other backend interfaces will also support this.

Once enabled, no configuration is needed. Any biblio fields you wish to index can be selected from the 'admin/config/search/search_api/index/INDEXNAME/fields' page.

Although most fields are indexed verbatim, the module exposes the 'biblio_authors', 'biblio_secondary_authors', 'biblio_tertiary_authors', 'biblio_subsidiary_authors' and 'biblio_corp_authors' fields as comma separated lists of the authors.

Important Links

  • Git: git clone --branch 7.x-1.x http://git.drupal.org/sandbox/JacobSanford/2150745.git biblio_search_api
  • Repository Viewer
  • Git: git.drupal.org/sandbox/JacobSanford/2150745.git
  • Project page: here
  • pareview results: here
    • CommentFileSizeAuthor
    #6 biblio.png5.75 KBdsim

    Comments

    PA robot’s picture

    PA robot commented

    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.

    tallosoft’s picture

    tallosoft commented

    Hello Jacob,

    I read through your project page, and I like the description of the project.

    I have not installed the module, but I ran PAREVIEW against it, and the following error is presented, and should be corrected:

    FILE: /var/www/drupal-7-pareview/pareview_temp/biblio_search_api.module
    --------------------------------------------------------------------------------
    FOUND 1 ERROR(S) AFFECTING 1 LINE(S)
    --------------------------------------------------------------------------------
    49 | ERROR | Data type of return value is missing
    --------------------------------------------------------------------------------

    aroq’s picture

    aroq commented

    Hi,

    line #54:
    foreach ($item->biblio_contributors as $biblio_contributor) {
    $author_names[] = $biblio_contributor['name'];
    }

    Probably you'll need to make sure that $item->biblio_contributors variable exists before you iterate through it.

    aroq’s picture

    aroq commented
    Status: Needs review » Needs work
    jacobsanford’s picture

    jacobsanford
    Primary language English
    Location Fredericton 🇨🇦
    commented
    Status: Needs work » Needs review

    @tallosoft, @aroq:

    Thanks for the review! I have corrected both suggestions and resubmitted the PAREVIEW.

    dsim’s picture

    dsim commented
    StatusFileSize
    new biblio.png5.75 KB

    Hi,

    I think you've missed a minor change in changelog.txt file after pareview check. Please find the attachment.

    Thanks

    jacobsanford’s picture

    jacobsanford
    Primary language English
    Location Fredericton 🇨🇦
    commented

    Yikes! Thanks for the heads up. Corrected.

    rachel_norfolk’s picture

    rachel_norfolk
    Primary language English
    Location UK
    commented

    I just happen to be working on a site using biblio and search_api. This module does allow me to add biblio fields to search_api and causes no issues I can see.

    Nice one!

    izus’s picture

    izus commented
    Status: Needs review » Needs work

    hi,
    here is my review
    return ($author_names);
    could it just be return $author_names; ?

    Also, i see how this module works for search_api integration, but i find it more general that that !
    For me, the dependency to search_api could just be moved as all what the module does is defining some properties which is quit independent from search_api.

    I also wonder if a little patch to biblio module wouldn't be a better idea that a new module

    PA robot’s picture

    PA robot commented
    Status: Needs work » Closed (won't fix)

    Closing due to lack of activity. Feel free to reopen if you are still working on this application (see also the project application workflow).

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