- This topic has 2 replies, 2 voices, and was last updated 2 years, 7 months ago by
.
-
Posts
-
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);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?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)
- The forum ‘Bug fixed or feature request implemented’ is closed to new topics and replies.