JQuery Pluggin Question

I have this plugin working just fine on this page:  Tagit Working Here.  Just start typing the word "book" in one of the fields and you'll see it works fine.

However, I'd like to see if I can put the ajax call inside a function and just call the function instead of having all that code on the page repeated.  I have tried but it isn't working.  I have no idea why, so I've turned to you experts for help!

Here is an example of the code that isn't working:
function get_tags(request,table){
	$.ajaxSetup({ cache: false });
	$.ajax({
		url: "php/profile-tags.php",
		dataType: "json",
		data: {
			txt: request.term,
			t: table
		},
		success: function( data ) {
			return data;
		}
	});								
}
	
$(function() {							
	$('#tagit_1_bands').tagit({
		tagSource:function( request, response ) {
			var data = get_tags(request,'bands');
			response( data );
		},
		triggerKeys:['enter', 'comma', 'tab'],
		minLength:2,
		maxTags:1,
		select: true,										
		placeholder:'Your 1st favorite band...'							
	});
});

Open in new window


The HTML the jquery references is probably not the problem at all, but here it is:
<ul id="tagit_1_bands" class="tagit" data-name="bands1[]"></ul>

Open in new window


The page its not working on is here: not working page.

Thanks for the help!
LVL 2
ElxnbossAsked:
Who is Participating?

[Product update] Infrastructure Analysis Tool is now available with Business Accounts.Learn More

x
I wear a lot of hats...

"The solutions and answers provided on Experts Exchange have been extremely helpful to me over the last few years. I wear a lot of hats - Developer, Database Administrator, Help Desk, etc., so I know a lot of things but not a lot about one thing. Experts Exchange gives me answers from people who do know a lot about one thing, in a easy to use platform." -Todd S.

vr6rCommented:
It looks to me like your ajax call IS working, it's just not returning any data - it's giving back an empty array.  Here's an example - this is the request that was sent from the 'broken' page:

http://girlzunderground.com/php/profile-tags.php?txt=book&t=bands&_=1416731255215

Click it and you'll see it just returns the opening/closing array brackets but no actual data is in the array.
But if you switch the "t" paramater back to "books", you get a response with valid data as expected.

http://girlzunderground.com/php/profile-tags.php?txt=book&t=books&_=1416731255215

Just sort out what is going on in the profile-tags.php script and you should be in good shape.
Hope that helps!
0
ElxnbossAuthor Commented:
Yeah, I know the bands table won't return data.  The only table with data in it is the books table.  So everything needs to be done with the books table for now.  But the problem is, on the page in my site where its not working, when i type stuff in a "books" field it still doesn't work for some reason.  

Its returning something too, I put an alert() in the get_tags function and it shows me data, but it doesn't look like JSON when I alert it for some reason, just comma separated values are shown in the alert box.  Perhaps this is the problem.  I've tried converting the return data to JSON again (even though shouldn't it be JSON already?) but this doesn't seem to work either.

Go to this link and try typing "book" into the favorite books field.  Just click on Top 10's and you'll see the field below.

Thanks.
0
vr6rCommented:
Oh ok I see the book fields now - I didn't scroll down far enough.

The return values look ok - looks like a string array which is normal to appear as a comma delimited list when alerted out.  It appears the result is just not being returned from the get_tags function.

Try this - simplifies the generic  function call a little and should eliminate the callback issue...
function get_tags( request, response ) {
	$.ajaxSetup({ cache: false });
	$.ajax({
		url: "php/profile-tags.php",
		dataType: "json",
		data: {
			txt: request.term,
			t: table
		},
		success: function( data ) {
			response( data );
		}
	});
}

$(function() {
	$('#tagit1').tagit({
		tagSource:get_tags,
		triggerKeys:['enter', 'comma', 'tab'],
		minLength:2,
		maxTags:3,
		allowNewTags: true,
		select: true,
		inputWidth:250,				
		placeholder:'Your 1st favorite book...'	});

	$('#tagit2').tagit({
		tagSource:get_tags,
		triggerKeys:['enter', 'comma', 'tab'],
		minLength:2,
		maxTags:3,
		allowNewTags: true,
		select: true,
		inputWidth:250,
		placeholder:'Your 2nd favorite book...'	});
});

Open in new window

0

Experts Exchange Solution brought to you by

Your issues matter to us.

Facing a tech roadblock? Get the help and guidance you need from experienced professionals who care. Ask your question anytime, anywhere, with no hassle.

Start your 7-day free trial
Determine the Perfect Price for Your IT Services

Do you wonder if your IT business is truly profitable or if you should raise your prices? Learn how to calculate your overhead burden with our free interactive tool and use it to determine the right price for your IT services. Download your free eBook now!

ElxnbossAuthor Commented:
Yes great it is working fine on the books now.

However, one question remains.  I need to pass get_tags the table name so it can work for all the different tables.

As you can see on this page (http://girlzunderground.com/profile.php) there are other tables besides the "books" table.

Thanks for your help!
0
ElxnbossAuthor Commented:
Well I think I figured it out myself.  I modified the get_tags function and added a third parameter and wrapped the tagSource get_tags function in an anonymous function and it worked!

function get_tags( request, response, table ) {
	$.ajaxSetup({ cache: false });
	$.ajax({
		url: "php/profile-tags.php",
		dataType: "json",
		data: {
			txt: request.term,
			t: table
		},
		success: function( data ) {
			response( data );
		}
	});
}

$(function() {																						
	$('#tagit_1_bands').tagit({
		tagSource:function(request, response) {
			get_tags( request, response, 'bands' );
		},
		triggerKeys:['enter', 'comma', 'tab'],
		minLength:2,
		maxTags:1,
		select: true,
				
		placeholder:'Your 1st favorite band...'								
	});
});

Open in new window

0
ElxnbossAuthor Commented:
The expert who commented on this question didn't answer my additional question.  I had a need for the "get_tags" function to have a way to understand which table i needed to query to get the tags and he didn't show me how.  So I went and figured it out myself and it is a working solution now.  But, it was me who figured this extra thing out, so I have to say both solutions played a part in answering the question.
0
It's more than this solution.Get answers and train to solve all your tech problems - anytime, anywhere.Try it for free Edge Out The Competitionfor your dream job with proven skills and certifications.Get started today Stand Outas the employee with proven skills.Start learning today for free Move Your Career Forwardwith certification training in the latest technologies.Start your trial today
jQuery

From novice to tech pro — start learning today.