Try the following and Your code should work fine.
target.next().addClass("active");
As an alternate implementation, You can use the Jquery data() utility to store the active slide information. This way the traversal up the list can be avoided. I have done a bit of optimization so that multiple DOM traversals can be avoided.
Here is the code snippet.
$(function () {
//select the active li
var $activeSlide = $("#infoslider > li.active");
$("a.orbit-next").click(function () {
if (!$activeSlide.length) { return; }
var $nextSlide = $activeSlide.next();
if (!$nextSlide.length) { return; }
var slideGroup;
//test, if there is a string in "data-orbit-slide"
if ($nextSlide.attr("data-orbit-slide").length > 0) {
slideGroup = $nextSlide.attr("data-orbit-slide");
$(this).data("slideGroup", slideGroup);
} else {
slideGroup = $(this).data("slideGroup");
}
//remove the classes active
$("ul.infoslider > li").removeClass('active');
//select the coresponding a in div.infoslider
$("a[data-orbit-link='" + slideGroup + "']").parent().addClass("active");
$activeSlide.removeClass("active");
$nextSlide.addClass("active");
$activeSlide = $nextSlide;
});
})