Tagged: 

Viewing 3 posts - 1 through 3 (of 3 total)
  • Author
    Posts
  • #64086
    Anonymous
    Inactive

    First line of eme_add_task_function() – doesn’t know what myel is.

    eme_admin.js?ver=2.2.72:224 Uncaught ReferenceError: myel is not defined
    at eme_add_task_function (eme_admin.js?ver=2.2.72:224:24)
    at HTMLAnchorElement.<anonymous> (eme_admin.js?ver=2.2.72:217:3)
    at HTMLAnchorElement.dispatch (jquery.min.js?ver=3.6.0:2:43064)
    at HTMLAnchorElement.v.handle (jquery.min.js?ver=3.6.0:2:41048)

    function eme_add_task_function() {
    var selectedItem = $(myel.parent().parent().get(0));
    var currentId = selectedItem.attr(‘id’).replace(’eme_row_task_’,”);
    //Get All meta rows
    //var metas = $(‘#eme_tasks_tbody’).children();
    //Copy first row and change values, but not the events (that causes trouble for cloned datepickers)
    //var metaCopy = $(metas[0]).clone(false);
    var metaCopy = selectedItem.clone(false);

    #64087
    Anonymous
    Inactive

    Was able to bypass this by editing w/ dev tools – just need to pass the $(this) into the method and define a parameter.

    Be sure to test adding multiple tasks – had to come back to add a third one.
    Does it make sense to have the “+” add button on each row – seems like not?

    #64088
    Franky
    Keymaster

    Indeed a js-error sneaked in. I’m surprised on how that one got through …
    This is the fix:
    https://plugins.trac.wordpress.org/changeset/2710069/

    (and the “+” on each row is to copy that row for the date/times, but for the rest all “+” act the same)

Viewing 3 posts - 1 through 3 (of 3 total)
  • The forum ‘Bug fixed or feature request implemented’ is closed to new topics and replies.
Scroll to Top