Fix layer generator with 2nd layer (#204)

* Added logic to add layer in the useEffect when the source is already added to the view.

Co-authored-by: Víctor Velarde <victor.velarde@gmail.com>
Co-authored-by: Dani Arlandis <daniarlandis@gmail.com>

CHANGELOG.md

@@ -9,6 +9,7 @@
 - Change BigQuery source type name from 'bq' to 'bigquery' [#203](https://github.com/CartoDB/carto-react-template/pull/203)
 - Add new hygen generators for components, models and slices [#171](https://github.com/CartoDB/carto-react-template/pull/171)
 - Adapt to new **multi-package** structure for carto-react libs [#206](https://github.com/CartoDB/carto-react-lib/pull/206)
+- Fix layer generator with second layer [#204](https://github.com/CartoDB/carto-react-template/pull/204)
 
 ## 1.0.0-beta12 (2021-02-08)
 - Refactor on basic JSX & JS stuff [#170](https://github.com/CartoDB/carto-react-template/pull/170)

hygen/_templates/layer/new/inject_view_import_actionsSlice.ejs.t

@@ -1,7 +1,8 @@
 ---
 inject: true
 to: "<%= attach ? `src/${view_path}` : null %>"
-after: "'react'"
-skip_if: "addLayer"
+after: "from 'react'"
+skip_if: \{ addLayer,
 ---
+
 import { addLayer, removeLayer, addSource, removeSource } from '@carto/react-redux';

hygen/_templates/layer/new/inject_view_import_layer_id.ejs.t

@@ -4,4 +4,4 @@ to: "<%= attach ? `src/${view_path}` : null %>"
 after: "from 'react'"
 skip_if: "import { <%= h.changeCase.constantCase(name) %>_ID"
 ---
-import { <%= h.changeCase.constantCase(name) %>_ID } from 'components/layers/<%= h.changeCase.pascalCase(name) -%>';
+import { <%= h.changeCase.constantCase(name) %>_ID } from 'components/layers/<%= h.changeCase.pascalCase(name) -%>';

hygen/_templates/layer/new/inject_view_import_source.ejs.t

@@ -4,4 +4,4 @@ to: "<%= attach ? `src/${view_path}` : null %>"
 after: "from 'react'"
 skip_if: "<%= `data/sources/${source_file}` %>"
 ---
-import <%= h.changeCase.camelCase(source_file) %> from 'data/sources/<%= h.changeCase.camelCase(source_file) %>';
+import <%= h.changeCase.camelCase(source_file) %> from 'data/sources/<%= h.changeCase.camelCase(source_file) %>';

hygen/_templates/layer/new/inject_view_splitted_useEffect_addLayer.ejs.t

@@ -0,0 +1,13 @@
+---
+inject: true
+to: "<%= attach ? `src/${view_path}` : null %>"
+after: addSource\(<%= h.changeCase.camelCase(source_file) %>\)\n[ \t]*\)|addSource\(<%= h.changeCase.camelCase(source_file) %>\)\);
+skip_if: <%= h.changeCase.constantCase(name) %>_ID,
+---
+
+    dispatch(
+      addLayer({
+        id: <%= h.changeCase.constantCase(name) %>_ID,
+        source: <%= h.changeCase.camelCase(source_file) %>.id,
+      })
+    );

hygen/_templates/layer/new/inject_view_splitted_useEffect_remove.ejs.t

@@ -0,0 +1,8 @@
+---
+inject: true
+to: "<%= attach ? `src/${view_path}` : null %>"
+before: dispatch\(removeSource\(<%= h.changeCase.camelCase(source_file) %>.id\)\);
+skip_if: dispatch\(removeLayer\(<%= h.changeCase.constantCase(name) %>_ID\)\);
+---
+
+      dispatch(removeLayer(<%= h.changeCase.constantCase(name) %>_ID));