Solved

Javascript function needing to adjust multiple spans with the same ID

Posted on 2007-11-14
3
416 Views
Last Modified: 2008-02-01
I'm currently developing a web application for scientific paper database management system, and am having issues with an AJAX function that is supposed to update multiple divs with the same name.

Schema background in simple terms is as follows:

There are multiple [papers] inside a [project] - call these [project_papers]. There are also 0 to many [subprojects] within the [project]. In the case of the screenshot example below there are 2 subprojects. A [project_paper] can exist in multiple [subprojects] within the project, or in none at all. Each row representing a [project_paper] on the screen has a "pin", which kind of acts like a sticky row when the contents of the project are contracted (hiding the ones with the pin off). The pin is a span with an id and name of "pPaper{project_paper_id}", which is normally fine.

However, and this is where the problem exists, if the [project_paper] exists in two [subprojects] simultaneously, the span will of course have the same id and name as there is no differentiation between subprojects. Personally, I think this should be correct, and any javascript designed to update the contents of the span should apply to all instances on the page, but it only updates the top one - the database side of the AJAX fires correctly, and therefore the [project_paper] has the pin flag changed, but the display ends up confusingly incorrect.

Is there any way to update the javascript to update the content of both spans?

Screenshot example (rows in yellow are the one with the same id -- pin is red thing on the left):

http://rdms.powerwatch.org.uk/rdms1.png

Screenshot example with changed pin (top row pin changed but not bottom row):

http://rdms.powerwatch.org.uk/rdms2.png

HTML that calls the javascript function:

----------
<span name="pPaper<%=CLng(RS("project_paper_id"))%>" id="pPaper<%
  =CLng(RS("project_paper_id"))%>"><img src="/images/buttons/pin_<%
  =projPaperPin%>.gif" border="0" onClick="setProjectPin(<%
  =CLng(RS("project_paper_id"))%>,<%=theUID%>)" /></span>
-----------

Javascript functions:

-----------
/* Adjusts the pin status of a project paper within project or subproject for the active user */

var projPapPin;

function setProjectPin(projectPaperID,userID)
{
  projPapPin = projectPaperID;
  xmlHttp=GetXmlHttpObject();

url="/includes/ajax/ch_projectpin.asp?projPaperID="+projectPaperID+"&userID="+userID+"&sid="+Math.random();
  xmlHttp.onreadystatechange=displayProjectPin;
  xmlHttp.open("GET",url,true);
  xmlHttp.send(null);
}

function displayProjectPin()
{
  if (xmlHttp.readyState==4 || xmlHttp.readyState=="complete")
  {
    document.getElementById('pPaper'+projPapPin).innerHTML = xmlHttp.responseText;
  }
}
-----------
0
Comment
Question by:topazg
  • 2
3 Comments
 
LVL 19

Accepted Solution

by:
nschafer earned 500 total points
ID: 20280293
Your problem is that Element ID's should be unique and Javascript assumes that they are.  So when you use getElementById() you are only getting the first element with that ID.  You may want to try using getElementsByName.  This will return a collection of elements with the name and you can then loop through them updating the innerHTML.

I hope this helps,
Neal.

example:
------------------------------------------------------------
function displayProjectPin()
{
  if (xmlHttp.readyState==4 || xmlHttp.readyState=="complete")
  {
    var el = document.getElementsByName('pPaper'+projPapPin);
    for (i=0;i<el.length;i++)
    {
        el[i].innerHTML = xmlHttp.responseText;
    }
  }
}

------------------------------------------------------------
0
 
LVL 3

Author Comment

by:topazg
ID: 20280528
Awesome, works perfectly, many thanks :)

I've updated the id to pPaper{paper_project_id}Sub{subproject_id} and left the name as it is, adopting your code example.

Thanks for the fast response!
0
 
LVL 19

Expert Comment

by:nschafer
ID: 20280730
Glad I could help,

Neal.
0

Featured Post

Announcing the Most Valuable Experts of 2016

MVEs are more concerned with the satisfaction of those they help than with the considerable points they can earn. They are the types of people you feel privileged to call colleagues. Join us in honoring this amazing group of Experts.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Suggested Solutions

New Relic: Our company recently started researching several products to figure out what were the best ways for us to increase our web page speed and to quickly identify performance problems that we may be having. One of the products we evaluated wa…
Real-time is more about the business, not the technology. In day-to-day life, to make real-time decisions like buying or investing, business needs the latest information(e.g. Gold Rate/Stock Rate). Unlike traditional days, you need not wait for a fe…
This video teaches users how to migrate an existing Wordpress website to a new domain.
Learn how to set-up custom confirmation messages to users who complete your Wufoo form. Include inputs from fields in your form, webpage redirects, and more with Wufoo’s confirmation options.

816 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question

Need Help in Real-Time?

Connect with top rated Experts

8 Experts available now in Live!

Get 1:1 Help Now