"Sorry, but that's not right at all"
My point was that IE does run into issue when it comes to reserved
words, which "row" is....
and as you can (or obviously can't) tell from my reply, you're right,
i was just taking some stabs in the dark to at least make an attempt
to help..... i don't think i needed a whole paragraph dedicated to me
in your reply because i tried to help...
lots of people makes posts on here that "it's broke" without any
indication or why or how it could have happened... but if some of us
didn't at least try to help, then this mailing list would be
practically nothing but topics started with replies of "post code/
link"
whatever i suppose....
On Dec 22, 11:04 pm, Michael Geary <m...@mg.to> wrote:
> Sorry, but that's not right at all. There is no problem with using 'row' as
> a function parameter name in IE.
>
> We're veering into Cargo Cult Programming here, but it's not your fault,
> MorningZ. The problem is that we can't see the actual code that's going
> wrong, so we're all reduced to guessing.
>
> GJK, could you please post a link to a test page that demonstrates the
> problem? That's the only way we can see the real issue and offer useful
> advice.
>
> -Mike
>
> On Tue, Dec 22, 2009 at 6:50 PM, MorningZ <morni...@gmail.com> wrote:
> > IE has a really hard time with "reserved words" used as function names
> > and/or properties
>
> > try changing "row" to something else, even "Row" would be different
> > enough, as IE *might* be thinking you mean a table row
>
> > so instead of
>
> > function DisplayResult(row) {
> > var output = '<dd id="">';
> > output += '<a href="' + row['link'] + '">';
> > output += '<p>';
> > output += '<b>' + row['title'] + '</b>';
> > output += row['summary'];
> > output += '</p></a></dd>';
> > return output;
>
> > }
>
> > try
>
> > function DisplayResult(ThisRow) {
> > var output = '<dd id="">';
> > output += '<a href="' + ThisRow['link'] + '">';
> > output += '<p>';
> > output += '<b>' + ThisRow['title'] + '</b>';
> > output += row['summary'];
> > output += '</p></a></dd>';
> > return output;
> > }
>
> > if that doesn't work, see if the value passed in is indeed a JSON
> > object
>
> > function DisplayResult(row) {
> > $.each(row, function(key, val) {
> > alert(key + ": " + val);
> > });
> > }
>
> > if that doesn't work then you are not passing an object into the
> > function (perhaps it's a string?)
>
> > On Dec 22, 8:35 pm, GJK <gerlofva...@gmail.com> wrote:
> > > Ok thank you,
>
> > > Why does this code work in Firefox etc. And not in internet explorer
> > > 7/8.
>
> > > And how to fix this for internet explorer?
>
> > > Greatly appreciate it!
>
> > > On Dec 23, 1:38 am, MorningZ <morni...@gmail.com> wrote:
>
> > > > that means that "title" is not a valid property of the "row"
> > > > object....
>
> > > > On Dec 22, 5:46 pm, GJK <gerlofva...@gmail.com> wrote:
>
> > > > > The error was wrong, the good one from internet explorer:
> > > > > -----------------------------------------
> > > > > Line: 158
> > > > > Character: 2
> > > > > Code: 0
> > > > > Error Message: 'title' is null or not an object
> > > > > URL:http://localhost.com/js/search.js
>
> > > > > And it's coming from this function:
>
> > > > > -----------------------------------------
> > > > > function DisplayResult(row) {
> > > > > var output = '<dd id="">';
> > > > > output += '<a href="' + row['link'] + '">';
> > > > > output += '<p>';
> > > > > output += '<b>' + row['title'] + '</b>';
> > > > > output += row['summary'];
> > > > > output += '</p></a></dd>';
> > > > > return output;
>
> > > > > }
>
> > > > > On Dec 22, 9:53 pm, GJK <gerlofva...@gmail.com> wrote:
>
> > > > > > Hello,
>
> > > > > > I got this code for search auto suggest and everything works fine
> > in
> > > > > > Firefox, Chrome, Opera etc. But it breaks in internet explorer 7
> > and
> > > > > > 8.
>
> > > > > > I get this error:
> > > > > > -----------------------------------------
> > > > > > Line: 158
> > > > > > Character: 2
> > > > > > Code: 0
> > > > > > Error Message: 'fghds' is null or not an object
> > > > > > URL:http://localhost.com/js/search.js
> > > > > > -----------------------------------------
>
> > > > > > This is the jquery script:
> > > > > > -----------------------------------------
>
> > > > > > var SEARCH_BOX_DEFAULT_TEXT = 'Enter the keywords:';
>
> > > > > > var AJAX_PENDING_TIMER;
> > > > > > var CURRENT_PAGE = 1;
> > > > > > var CURRENT_LIMIT = 5;
>
> > > > > > function init() {
>
> > > > > > var sTextBox = $("#search_val");
>
> > > > > > sTextBox.focus(function() {
> > > > > > if(this.value == SEARCH_BOX_DEFAULT_TEXT) {
> > > > > > this.value = '';
> > > > > > }
> > > > > > });
> > > > > > sTextBox.blur(function() {
> > > > > > if(this.value == '') {
> > > > > > this.value = SEARCH_BOX_DEFAULT_TEXT;
> > > > > > }
> > > > > > });
> > > > > > sTextBox.blur();
>
> > > > > > sTextBox.keyup(function() {
> > > > > > var q = $("#search_val").val();
> > > > > > if( q == SEARCH_BOX_DEFAULT_TEXT || q == '' || q ==
> > undefined ||
> > > > > > q.length<=3) {
> > > > > > HideLiveSearch();
> > > > > > }
> > > > > > else {
> > > > > > clearTimeout(AJAX_PENDING_TIMER);
> > > > > > CURRENT_PAGE = 1;
> > > > > > AJAX_PENDING_TIMER =
> > setTimeout("PerformLiveSearch()",300);
> > > > > > }
>
> > > > > > });
>
> > > > > > $("#livesearch_result_close_link").click(function() {
> > > > > > HideLiveSearch();
> > > > > > });
>
> > > > > > }
>
> > > > > > function NextPage(p) {
> > > > > > if(p['has_next']) {
> > > > > > AJAX_IS_RUNNING = false;
> > > > > > CURRENT_PAGE++;
> > > > > > PerformLiveSearch();
> > > > > > }}
>
> > > > > > function PrevPage(p) {
> > > > > > if(p['has_prev']) {
> > > > > > AJAX_IS_RUNNING = false;
> > > > > > CURRENT_PAGE--;
> > > > > > PerformLiveSearch();
> > > > > > }
>
> > > > > > }
>
> > > > > > function ShowLoaders() {
> > > > > > $('#ajaxloader').fadeIn('fast');
>
> > > > > > if( $('#livesearch').css('display') == 'block' ) {
> > > > > > var h = $('#livesearch').height() - 5;
> > > > > > var w = $('#livesearch').width() - 45;
> > > > > > $('#loader_div').width(w);
> > > > > > $('#loader_div').height(h);
> > > > > > $('#loader_div p').css('margin-top',(h/2)+20);
> > > > > > $('#loader_div').fadeIn('fast');
> > > > > > }
>
> > > > > > }
>
> > > > > > function HideLoaders() {
> > > > > > $('#ajaxloader').fadeOut('fast');
> > > > > > $('#loader_div').hide();
>
> > > > > > }
>
> > > > > > var AJAX_IS_RUNNING = false;
> > > > > > function PerformLiveSearch() {
>
> > > > > > if(AJAX_IS_RUNNING == true) {
> > > > > > return;
> > > > > > }
>
> > > > > > var query = $("#search_val");
> > > > > > var q_val = (query.val() && query.val() !=
> > > > > > SEARCH_BOX_DEFAULT_TEXT) ? query.val() : '';
>
> > > > > > if(q_val == '') {
> > > > > > return;
> > > > > > }
> > > > > > AJAX_IS_RUNNING = true;
>
> > > > > > $.ajax({
> > > > > > url: './search',
> > > > > > data: {
> > > > > > query: q_val,
> > > > > > output: 'json',
> > > > > > page: CURRENT_PAGE,
> > > > > > limit: CURRENT_LIMIT
> > > > > > },
> > > > > > type: 'get',
> > > > > > timeout: '5000',
> > > > > > dataType: 'json',
> > > > > > beforeSend: function() {
> > > > > > // Before send request
> > > > > > // Show the loader
> > > > > > AJAX_IS_RUNNING = true;
> > > > > > ShowLoaders();
> > > > > > },
> > > > > > complete: function() {
> > > > > > // When Sent request
> > > > > > // Hide the loader
> > > > > > AJAX_IS_RUNNING = false;
> > > > > > HideLoaders();
> > > > > > },
> > > > > > success: function(data, textStatus) {
> > > > > > AJAX_IS_RUNNING = false;
> > > > > > HideLoaders();
> > > > > > $('#livesearch').slideDown();
>
> > > > > > // clear the results
> > > > > > $(".livesearch_results dd").remove();
> > > > > > var resultsNav = $('.livesearch_results
> > dt');
>
> > > > > > if( data['results'].length ) {
>
> > > > > > // add the new results (in reverse
> > since the
> > > > > > // append inserts right after the
> > header and not
> > > > > > // at the end of the result list
> > > > > > var current = resultsNav;
>
> > for(i=data['results'].length;i>0;i--) {
> > > > > > current.after(
>
> > DisplayResult(data['results'][i-1])
> > > > > > );
> > > > > > }
> > > > > > }
> > > > > > else {
> > > > > > resultsNav.after('<dd id=""><p>No
> > articles found with these search
> > > > > > terms</p></dd>');
> > > > > > }
>
> > > > > > // Pagination at the bottom of LiveSearch
> > panel
>
> > Pagination(data['paging'],".livesearch_navbody");
>
> > > > > > },
>
> > > > > > // We're gonna hide everything when get error
> > > > > > error: function(XMLHttpRequest, textStatus,
> > errorThrown) {
> > > > > > AJAX_IS_RUNNING = false;
> > > > > > HideLoaders();
> > > > > > HideLiveSearch();
> > > > > > }
> > > > > > });
>
> > > > > > }
>
> > > > > > function DisplayResult(row) {
> > > > > > var output = '<dd id="">';
> > > > > > output += '<a href="' + row['link'] + '">';
> > > > > > output += '<p>';
>
> ...
>
> read more »
