Field Import / Export

This is something we could add:

- to each field as an export tab
- to the entity type admin as in import tab

Patches gratefully accepted! :)

Thanks for the patch! Sorry it's taken me a while to look at it -- big patches are daunting, and I was hoping other users would give it a try too.

This is just a first-pass eyeball review. There are quite a few minor problems that need fixing. Sorry if this seems like a lot of nitpicking, but as a maintainer, I need to keep the code in the module in a form that is readable and understandable. Hence I'm stickler for Drupal coding standards -- please do take some time to familiarize yourself with them :)

+++ b/field_tools.admin.inc
@@ -463,6 +463,374 @@ function field_tools_field_clone_form_submit($form, &$form_state) {
+ * Export all fields of a bundle
+ *
+ * @param string $form ¶
+ * @param string $form_state ¶
+ * @param string $entity_type ¶
+ * @param string $bundle ¶

Can you check the formatting of your docblocks please?

- first line needs a full stop.
- @param lines need a description
- no @return needed if there is no return

+++ b/field_tools.admin.inc
@@ -463,6 +463,374 @@ function field_tools_field_clone_form_submit($form, &$form_state) {
+  ¶

Please trim whitespace before rolling a patch. Also, this blank line shouldn't be here anyway.

+++ b/field_tools.admin.inc
@@ -463,6 +463,374 @@ function field_tools_field_clone_form_submit($form, &$form_state) {
+  // Get the bundle name if the bundle name is really a bundle object.
+  $bundle_name = is_object($bundle) ? $bundle->type : $bundle;

Why would this parameter sometimes be an object and sometimes not? The @param docs need to explain this.

+++ b/field_tools.admin.inc
@@ -463,6 +463,374 @@ function field_tools_field_clone_form_submit($form, &$form_state) {
+  if (isset($form_state['storage']['fields'])) {

What is this testing?

+++ b/field_tools.admin.inc
@@ -463,6 +463,374 @@ function field_tools_field_clone_form_submit($form, &$form_state) {
+    ¶

No need for this blank line.

+++ b/field_tools.admin.inc
@@ -463,6 +463,374 @@ function field_tools_field_clone_form_submit($form, &$form_state) {
+function field_tools_bundle_export_form_submit($form, &$form_state) {

Needs a docblock -- though you can skip the @params since they are standard.

+++ b/field_tools.admin.inc
@@ -463,6 +463,374 @@ function field_tools_field_clone_form_submit($form, &$form_state) {
+  if ($form_state['values']['op'] == $form_state['values']['import_force']) {

What is this testing?

+++ b/field_tools.admin.inc
@@ -463,6 +463,374 @@ function field_tools_field_clone_form_submit($form, &$form_state) {
+    form_set_error('code');

Shouldn't this set an error message too?

+++ b/field_tools.admin.inc
@@ -463,6 +463,374 @@ function field_tools_field_clone_form_submit($form, &$form_state) {
+  unset($group->id);
+  unset($group->identifier);
+  unset($group->bundle);
+  unset($group->entity_type);

This needs to be explained.

+++ b/field_tools.admin.inc
@@ -733,3 +1101,25 @@ function field_tools_field_instances_list($field) {
+function field_tools_textarea($label, $value = NULL) {

I'm not terribly keen on this. It makes forms harder to understand. Do we really need it? How many times is it used? How many times is the form array it returns tweaked a bit by the caller?

+++ b/field_tools.module
@@ -226,12 +267,87 @@ function _field_tools_field_already_attached($entity_type, $bundle_type, $field_
+  if ($form_id == 'field_ui_field_overview_form') {

Use hook_form_FORM_ID_alter() here.

+++ b/field_tools.module
@@ -226,12 +267,87 @@ function _field_tools_field_already_attached($entity_type, $bundle_type, $field_
+    if (isset($form['#after_build']) && is_array($form['#after_build'])) {

What is this testing?

First of all, thanks for this useful patch.

In my tests, I found the following error when trying to Export the fields from an entity created with ECK:

Notice: Trying to get property of non-object in field_tools_bundle_export_form() (line 471 of ...\field_tools.admin.inc).

This is the line 471:

  // Get the bundle name
  $bundle_name = $bundle->type;

The same error appears when trying to Export the fields from any Field Collection.

In these cases, $bundle is just a string, not an array, the opposite that happens in Content Types.

Committed a modified version of the patch with a few tweaks:

- Fixed button label on export form.
- Added CTools as dependency.
- Moved menu items under Tools tab.
- Spacing and comments formatting.

git commit -m "Issue #1859388 by berliner: Added tools for field import / export." --author="berliner "

Thanks!

> You can add that functionality to the list on the project page as well.

Already done :)

You're welcome! Thanks again for all your work on this new feature.

I just used the import feature. Thank you for saving me hours of recreating fields!

I'm looking all over for the "import feature" and can't find where this happens. The "export" portion is pretty clear. Is there a UI option for importing the field? If so, where do I find it?

@stpaultim

I've come across the same thing and have raised it in a new thread.

https://www.drupal.org/node/2681079