source: branches/3.0-stable/www/index.jsp @ 5947

Last change on this file since 5947 was 5947, checked in by Nicklas Nordborg, 11 years ago

References #1660: Internet Explorer 9 won't open popup windows

I think this fixes the problems with IE9. I have not been able to see the issue after the changes. With the old code it usually possible to see the problem withing 5-10 clicks. The fixes include:

  • Always use the code in the top-level window (frameset) to open popup windows. This get rid of the closed-parent problem which seems to be a possible cause. But we still need to use the true parent window since a lot of popup windows uses the window.opener property to communicate values back and forth.


  • Check the URL length before opening a blank window. If the POST form is not needed we simply open the window with the correct URL to begin with. If a POST form is needed we open a window with a real URL referencing a blank window. This should hopefully get rid of cross-site-scripting issues.


  • Use exception handling when getting old popup window information. If we can't access the information the old popup is probably already closed and the code can at least continue and open a new popup.
  • Property svn:eol-style set to native
  • Property svn:keywords set to Id
File size: 2.0 KB
Line 
1<%-- $Id: index.jsp 5947 2012-02-06 14:12:05Z nicklas $
2  ------------------------------------------------------------------
3  Copyright (C) 2005 Nicklas Nordborg
4  Copyright (C) 2006 Jari Häkkinen, Nicklas Nordborg
5
6  This file is part of BASE - BioArray Software Environment.
7  Available at http://base.thep.lu.se/
8
9  BASE is free software; you can redistribute it and/or
10  modify it under the terms of the GNU General Public License
11  as published by the Free Software Foundation; either version 3
12  of the License, or (at your option) any later version.
13
14  BASE is distributed in the hope that it will be useful,
15  but WITHOUT ANY WARRANTY; without even the implied warranty of
16  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
17  GNU General Public License for more details.
18
19  You should have received a copy of the GNU General Public License
20  along with BASE. If not, see <http://www.gnu.org/licenses/>.
21  ------------------------------------------------------------------
22
23  The main page is divided into two vertical frames.
24
25  @author Nicklas
26  @version 2.0
27--%>
28<%@ page pageEncoding="UTF-8" session="false"
29  import="net.sf.basedb.core.Application"
30  import="net.sf.basedb.core.SessionControl"
31  import="net.sf.basedb.clients.web.Base"
32  import="net.sf.basedb.clients.web.servlet.RssNewsFeed"
33%>
34<%@ taglib prefix="base" uri="/WEB-INF/base.tld" %>
35<%
36SessionControl sc = Base.getSessionControl(pageContext, true);
37String ID = sc.getId();
38request.setAttribute("ID", ID);
39%>
40<base:page type="frameset" title="">
41  <base:head>
42  <%
43  if (RssNewsFeed.isEnabled()) 
44  {
45    %>
46    <link rel="alternate" type="application/rss+xml" 
47      title="BASE news from <%=Application.getHostName()%><%=request.getContextPath()%>" 
48      href="info/news.rss"/>
49    <%
50  }
51  %>
52  </base:head>
53  <frameset rows="*,50" frameborder="no" border="0">
54    <frame name="main" src="main.jsp?ID=<%=ID%>" scrolling="auto" marginwidth=0 marginheight=0 noresize>
55    <frame name="footnote" src="footnote.jsp?ID=<%=ID%>" scrolling="no" marginwidth=0 marginheight=0 noresize>
56  </frameset>
57</base:page>
Note: See TracBrowser for help on using the repository browser.