[REF] web: make _render functions overridable

As `_renderInnerGroupRow` is returning the row, appended to the
previous result, and not only the row, it was not possible to
override the method to add, for example, another row.

This commit doesn't change the behaviour, only the return of
private functions.

1 files changed, 5 insertions, 4 deletions

diff --git a/addons/web/static/src/js/views/form/form_renderer.js b/addons/web/static/src/js/views/form/form_renderer.js
index 285b7349ada..2370b1af19a 100644
--- a/addons/web/static/src/js/views/form/form_renderer.js
+++ b/addons/web/static/src/js/views/form/form_renderer.js
@@ -392,7 +392,7 @@ return BasicRenderer.extend({
                     fieldNodes.push(children[i+1]);
                     i++;
                 }
-                $result = this._renderInnerGroupRow($result, fieldNodes);
+                $result.append(this._renderInnerGroupRow(fieldNodes));
             } else if (children[i].tag === 'label') {
                 var label =  children[i];
                 // If there is a "for" attribute, we expect to have an id concerned in the next node.
@@ -425,17 +425,18 @@ return BasicRenderer.extend({
         return $result.append($tr);
     },
     /**
+     * Render a group row, with all the nodes inside.
+     *
      * @private
-     * @param {jQueryElement} $result
      * @param {Object[]} nodes
      * @returns {jQueryElement}
      */
-    _renderInnerGroupRow: function ($result, nodes) {
+    _renderInnerGroupRow: function (nodes) {
         var $tr = $('<tr>');
         for (var i = 0; i < nodes.length; i++) {
             $tr.append(this._renderInnerField(nodes[i]).contents());
         }
-        return $result.append($tr);
+        return $tr;
     },
     /**
      * Render a node, from the arch of the view. It is a generic method, that