merge various Cells in bootstrap Table

I’m new to bootstrap table, I’m using the method MergeCells bootstraptable, I once operated at what I did the following code to unify several rows, I wonder what I’m doing wrong.

$(function () {

            var table = document.getElementById('table');
            var rowLength = table.rows.length - 1;
            var count = 0;
            var row = table.rows[1].cells[0].innerHTML;
            var saveIndex = 0;
            for (var i = 0; i < rowLength; i += 1) {
                if (row === table.rows[i].cells[0].innerHTML) {
                    count++;
                } else {
                    row === table.rows[i].cells[0].innerHTML;
                    mergeRows(saveIndex, count);
                    saveIndex = i;
                    count = 0;
                }
            }
        });

        function mergeRows(index, rowspan) {
            $('#table').bootstrapTable('mergeCells', {
                index: index,
                field: 'ciudad',
                rowspan: rowspan
            });

        }

What I do is call bootstrap function and unify according to the types of records

[dabros]

Solution: http://jsfiddle.net/dabros/sz42rymd/4/

Read the lines i changed carefully, your use of starting ints was wrong (super obvious when debugging) and placing console.log calls in merging function helped fix saveIndex issue.

Seemed to be indexing and a typo on your part (what was the second “===” doing there in an assignment???)

Still, read note in previous comment and refactor that code into a ‘OnPostBody’ or similar callback if still wanting to use it, even if you dont intent to add sorting, filters or any more functionality to the table - its still better form in case you use that code anywhere else or go back to it later to add more.

Its not thoroughly tested either, but should work on any data-set size.

For future referance, always debug first - liberal use of console.log is recommended (see my fiddle). 90% of the time it is always something simple like indexing or variable mismatch, which is why console exists : )

[Janbsi]

thanks @dabros 👍