For some imports it was useful, that the keyword filter, would only fitler the field, not the whole import item.

Files: 
CommentFileSizeAuthor
#5 skip_field_only-1372826-5.patch2 KBmErilainen
PASSED: [[SimpleTest]]: [MySQL] 821 pass(es). View
#2 skip_field_only-1372826-2.patch2 KBderhasi
FAILED: [[SimpleTest]]: [MySQL] 781 pass(es), 3 fail(s), and 10 exception(s). View
#1 skip_field_only-1372826-1.patch1.4 KBderhasi
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch skip_field_only-1372826-1.patch. Unable to apply patch. See the log in the details link for more information. View

Comments

derhasi’s picture

Status:Active» Needs review
StatusFileSize
new1.4 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch skip_field_only-1372826-1.patch. Unable to apply patch. See the log in the details link for more information. View

Most mappers will do just fine, with an empty value provided (like the fields implementation).

So, there is a patch for this functionality.

derhasi’s picture

StatusFileSize
new2 KB
FAILED: [[SimpleTest]]: [MySQL] 781 pass(es), 3 fail(s), and 10 exception(s). View

Oh, I missed the non regex part. A modified patch is attached.

mErilainen’s picture

I have used this in two projects and it has proven to be very useful.
But I get this error after applying the patch:

Deprecated function: Call-time pass-by-reference has been deprecated in ctools_plugin_load_includes() (line 475 of /Users/username/Sites/sitename/drupal/builds/build_2012-12-03-130253/sites/all/modules/contrib/ctools/includes/plugins.inc).

Status:Needs review» Needs work

The last submitted patch, skip_field_only-1372826-2.patch, failed testing.

mErilainen’s picture

Status:Needs work» Needs review
StatusFileSize
new2 KB
PASSED: [[SimpleTest]]: [MySQL] 821 pass(es). View

There were function calls with the &$field, when it should be just $field. This patch fixes it.

twistor’s picture

Issue summary:View changes
Status:Needs review» Needs work
Issue tags:+Needs tests

Looks ok, there have been some changes.

Sorry that I missed this.