Stable release of component mapped export and import feature #61

Merged
Llewellyn merged 2 commits from staging into master 2017-03-31 22:13:21 +00:00
Owner

Resolves gh-59 the repeatable field id problem. Also improved the export and import to insure snippets and linked admin views are re-mapped on import of the component. Also fixed the target and match field conditions in admin views to re-map the ids correctly.

Resolves gh-59 the repeatable field id problem. Also improved the export and import to insure snippets and linked admin views are re-mapped on import of the component. Also fixed the target and match field conditions in admin views to re-map the ids correctly.
ro-ot (Migrated from github.com) reviewed 2017-03-31 21:41:56 +00:00
ro-ot (Migrated from github.com) left a comment
Author
Owner

Will test this more heavily then the last one.

Will test this more heavily then the last one.
ro-ot (Migrated from github.com) reviewed 2017-03-31 21:57:26 +00:00
@ -480,29 +516,116 @@ class ComponentbuilderModelJoomla_components extends JModelList
// add search array templates and layouts
foreach ($searchArray as $scripter)
ro-ot (Migrated from github.com) commented 2017-03-31 21:57:25 +00:00
Author
Owner

Why did you remove this, what if the linked view is another components view?

Why did you remove this, what if the linked view is another components view?
Llewellyn reviewed 2017-03-31 22:00:19 +00:00
@ -480,29 +516,116 @@ class ComponentbuilderModelJoomla_components extends JModelList
// add search array templates and layouts
foreach ($searchArray as $scripter)
Author
Owner

That option is not working at this time, so my idea was to just use the actual views of the component... I could add it back. At first it caused a loop, but I fixed that.

That option is not working at this time, so my idea was to just use the actual views of the component... I could add it back. At first it caused a loop, but I fixed that.
ro-ot (Migrated from github.com) reviewed 2017-03-31 22:05:48 +00:00
@ -474,0 +505,4 @@
if (ComponentbuilderHelper::checkArray($fieldsSets))
{
$this->setData($user, $db, 'field', array('field' => $fieldsSets), 'field');
}
ro-ot (Migrated from github.com) commented 2017-03-31 22:05:48 +00:00
Author
Owner

Smart move to prevent the loop 👍

Smart move to prevent the loop :+1:
ro-ot (Migrated from github.com) approved these changes 2017-03-31 22:11:25 +00:00
ro-ot (Migrated from github.com) left a comment
Author
Owner

Works well, I have tested it with all the components I have, and it moved them from one JCB to another JCB without any errors.

Works well, I have tested it with all the components I have, and it moved them from one JCB to another JCB without any errors.
Sign in to join this conversation.
No reviewers
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: joomla/Component-Builder#61
No description provided.