This module defines a field type. It should expose the corresponding entity metadata type so that the field can be indexed in Search API, etc.

This is brought to you by the effort to leverage Search API on Drupal.org.

CommentFileSizeAuthor
#9 machine_name-search-api-1832328-9.patch457 byteskscheirer
#7 machine_name-search-api-1832328-7.patch458 byteskscheirer
#3 machine_name-search-api-1832328-3.patch458 byteskscheirer

Comments

damien tournoud’s picture

damien tournoud commented

This is the extend of the required patch:

--- a/machine_name.module	2012-06-29 20:16:50 +0000
+++ b/machine_name.module	2012-11-05 11:51:52 +0000
@@ -21,6 +21,8 @@
       'instance_settings' => array(),
       'default_widget' => 'machine_name_default',
       'default_formatter' => 'machine_name_default',
+      // Expose the field as text for the Entity metadata framework.
+      'property_type' => 'text',
     ),
   );
 }
c-logemann’s picture

c-logemann
he/him
Primary language German
Location Frankfurt/M, Germany
commented

Sounds right and important and I will check this, but what about your tag "Search API on Drupal.org"? Is the Search API integration already an official way?
I just committed direct apache solr support as part of the "Drupal.org D7 Upgrade Initiative" and tagged the code for a special stable version.
#1822252: Allow machine_name to provide field indexing to ApacheSolr
#1647904: Machine Name needs a 1.0 stable release as part of the Drupal.org D7 Upgrade Initiative

kscheirer’s picture

kscheirer
Primary language English
Location Vallejo
commented
Status: Active » Needs review
StatusFileSize
new machine_name-search-api-1832328-3.patch458 bytes

Here's the above as a patch, not sure how to test it.

Status: Needs review » Needs work

The last submitted patch, machine_name-search-api-1832328-3.patch, failed testing.

kscheirer’s picture

kscheirer
Primary language English
Location Vallejo
commented

Huh? Whats wrong with my patch?

c-logemann’s picture

c-logemann
he/him
Primary language German
Location Frankfurt/M, Germany
commented

@kscheirer follow "View details" link above: "Ensure the patch only contains unix-style line endings."
Not every text editor allows to control text encoding.

kscheirer’s picture

kscheirer
Primary language English
Location Vallejo
commented
Status: Needs work » Needs review
StatusFileSize
new machine_name-search-api-1832328-7.patch458 bytes

Hmm, trying again then.

Status: Needs review » Needs work

The last submitted patch, machine_name-search-api-1832328-7.patch, failed testing.

kscheirer’s picture

kscheirer
Primary language English
Location Vallejo
commented
Status: Needs work » Needs review
StatusFileSize
new machine_name-search-api-1832328-9.patch457 bytes

arg...

damien tournoud’s picture

damien tournoud commented
Status: Needs review » Reviewed & tested by the community

Thanks!

c-logemann’s picture

c-logemann
he/him
Primary language German
Location Frankfurt/M, Germany
commented
Status: Reviewed & tested by the community » Fixed

Committed in 7.x-2.x branch: http://drupalcode.org/project/machine_name.git/commit/100cb10

c-logemann’s picture

c-logemann
he/him
Primary language German
Location Frankfurt/M, Germany
commented
Version: 7.x-1.x-dev » 7.x-2.x-dev

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