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.