Data::Dump breaks Mojo::JSON https://rt.cpan.org/Public/Bug/Display.html?id=86592

[MojoFacets.git] / public / edit_table.js

diff --git a/public/edit_table.js b/public/edit_table.js
index 651f3ad..9914a3c 100644 (file)
--- a/public/edit_table.js
+++ b/public/edit_table.js
@@ -6,27 +6,40 @@ var cell_blur = function() {
 //             , $(this).val()
        );
 
+/*
        // FIXME primary key is fixed to 1st column
        var pk = $('table tr th:nth(0) > a').text();
        var id = $(this).parent().siblings(':nth(0)').text()
+*/
+       var _row_id = $(this).parent().parent().attr('title');
 
-       var x = $(this).parent().attr('cellIndex');
-       var y = $(this).parent().parent().attr('rowIndex');
+       var x = $(this).parent().index();
+       var y = $(this).parent().parent().index();
 
-       var content = $(this).val();
-//     $(this).replaceWith( content );
+       if ( ! x || ! y ) {
+               console.error('no x ', x, ' or y ', y, ' defined');
+               return;
+       }
+
+       var new_content = $(this).val();
+//     $(this).replaceWith( new_content );
 
        var name = $('table tr th:nth('+x+') > a').text();
-       console.info( x, y, pk, id, name, content );
+       console.info( x, y, _row_id, name, new_content );
 
        var update = $(this);
 
        $.post( '/data/edit', {
-               path: document.title, pk: pk, id: id,
-               name: name, content: content
-       } , function(data) {
-               console.debug( data );
-               var vals = content.split('¶');
+               path: document.title, _row_id: _row_id,
+               name: name, new_content: new_content
+       } , function(data, textStatus) {
+               console.debug( 'data:', data, 'status:', textStatus );
+               if ( ! data ) {
+                       data = new_content; // fallback to submited data for 304
+               } else {
+                       $('span.save_actions').show();
+               }
+               var vals = data.split('¶');
                data = vals.join('<span class=d>¶</span>');
                update.replaceWith( data );
        });
@@ -38,10 +51,17 @@ var cell_click = function(event) {
                , event
                , $(this).text()
        );
-       var content = $(this).text(); // we don't want para markup
-       var rows = content.split('¶').length * 2 + 1;
+
+       if ( $(this).find('textarea').length ) return; // allready editing
+
+       var new_content = $(this).text() // we don't want para markup
+               .replace(/^[ \n\r]+/,'')
+               .replace(/[ \n\r]+$/,'')
+       ;
+console.debug( 'new_content', new_content );
+       var rows = new_content.split('¶').length * 2 + 1;
        var textarea = $('<textarea rows='+rows+'/>');
-       textarea.val( content );
+       textarea.val( new_content );
        $(this).html( textarea );
        textarea.focus();
        textarea.blur( cell_blur )