Every instance of the portlet uses the same copy of the variables (not intentionally) in Liferay

[Al Razor]

Hi,

I have a small logistics-related portlet in Liferay 6.0.6.
I got the following problem: it seems that every instance of the portlet uses the same copy of the variables. Even when I change user, I still receive the same information that the last user has selected.
Maybe you could help me to find what causes it?

Here is the necessary information (please remember that it is just a working copy ):
Main Java file (I left out additional function because they are unrelated in this case.

	@Override
	public void init() throws PortletException
		{
		country_list = getCountries();
		}

	@Override
	protected void doView(RenderRequest request, RenderResponse response) throws PortletException, IOException
		{
		request.setAttribute("countryList", country_list);

		if ((departure_country != null) && (!"".equals(departure_country)))
			{
			departure_port_list = getPorts(departure_country);
			System.out.printf("departure_port_list size is [%d]\n", departure_port_list.size());
			request.setAttribute("departurePortList", departure_port_list);
			}


		if ((arrival_country != null) && (!"".equals(arrival_country)))
			{
			arrival_port_list = getPorts(arrival_country);
			request.setAttribute("arrivalPortList", arrival_port_list);
			}

		if (shipping_line_list != null)
			request.setAttribute("shippingLineList", shipping_line_list);

		response.setContentType("text/html");
		PortletContext portletContext = getPortletContext();
		PortletRequestDispatcher prd = portletContext.getRequestDispatcher("/view.jsp");
		prd.include(request, response);
		}

	@Override
	public void processAction(ActionRequest request, ActionResponse response) throws PortletException, IOException
		{
		departure_country = request.getParameter("departureCountry");
		arrival_country = request.getParameter("arrivalCountry");
		departure_port = request.getParameter("departurePort");
		arrival_port = request.getParameter("arrivalPort");

		country_list = getCountries();

		// If user pressed Submit button, check if departure and arrival ports are selected
		String submit_trigger = request.getParameter("submit_trigger");

		if (submit_trigger != null)
			{
			if ((departure_port != null) && (!"".equals(departure_port)) && (arrival_port != null)
					&& (!"".equals(arrival_port)))
				{
				request.setAttribute("departurePort", departure_port);
				request.setAttribute("arrivalPort", arrival_port);
				shipping_line_list = getShippingLines(departure_port, arrival_port);
				}
			}


		request.setAttribute("departureCountry", departure_country);
		request.setAttribute("arrivalCountry", arrival_country);
		}
	}

view.jsp (the only JSP I got) only contains JSTL and a bit of JavaScript

<%@ taglib uri="http://java.sun.com/portlet_2_0" prefix="portlet"%>
<%@ taglib uri="http://java.sun.com/jsp/jstl/core" prefix="c"%>
<%@ taglib uri="http://java.sun.com/jsp/jstl/functions" prefix="fn"%>

<portlet:defineObjects />

<script type="text/javascript" src="js/main.js"></script>


<c:if test="${fn:length(shippingLineList) > 0}">
	<table id="travel" class="sample" summary="Shipping lines services">
		<caption>Shipping lines services 5</caption>
		<thead>
			<tr>
				<th scope="col">Company</th>
				<th scope="col">Website (URL)</th>
				<th scope="col">Frequency</th>
				<th scope="col">Transit Time</th>
			</tr>
		</thead>
	
		<tbody>
			<tr>
				<c:forEach items="${shippingLineList}" var="entry">
					<tr>
						<td><c:out value="${entry.company}" /></td>
						<td><c:out value="${entry.url}" /></td>
						<td><c:out value="${entry.frequency}" /></td>
						<td><c:out value="${entry.transitTime}" /></td>
					</tr>
				</c:forEach>
			</tr>
		</tbody>
	</table>
</c:if>
<br />

<div id="wrap">
	<form name="mainForm" action="<portlet:actionURL></portlet:actionURL>" method="post">
		<div id="fields">
			<select name="departureCountry" onchange="changeSelectedOption()" id="left">
				<c:forEach var="entry"	items="${countryList}">
					<c:choose>
						<c:when test="${entry.value == departureCountry}">
							<option selected value='<c:out value="${entry.value}"/>'>
								<c:out value="${entry.key}" />
							</option>
						</c:when>
						<c:otherwise>
							<option value='<c:out value="${entry.value}"/>'>
								<c:out value="${entry.key}" />
							</option>
						</c:otherwise>
					</c:choose>
				</c:forEach>
			</select>

			<%--  --%>

			<select name="arrivalCountry" onchange="changeSelectedOption()" id="right">
				<c:forEach var="entry"	items="${countryList}">
					<c:choose>
						<c:when test="${entry.value == arrivalCountry}">
							<option selected value='<c:out value="${entry.value}"/>'>
								<c:out value="${entry.key}" />
							</option>
						</c:when>
						<c:otherwise>
							<option value='<c:out value="${entry.value}"/>'>
								<c:out value="${entry.key}" />
							</option>
						</c:otherwise>
					</c:choose>
				</c:forEach>
			</select> <br />

			<div class="clear"></div>

			<select name="departurePort" id="left">
				<c:forEach var="entry" 	items="${departurePortList}">
					<c:choose>
						<c:when test="${entry.value == departurePort}">
							<option selected value='<c:out value="${entry.value}"/>'>
								<c:out value="${entry.key}" />
							</option>
						</c:when>
						<c:otherwise>
							<option value='<c:out value="${entry.value}"/>'>
								<c:out value="${entry.key}" />
							</option>
						</c:otherwise>
					</c:choose>
				</c:forEach>
			</select> 
			<select name="arrivalPort" id="right">
				<c:forEach var="entry" items="${arrivalPortList}">					
					<c:choose>
						<c:when test="${entry.value == arrivalPort}">
							<option selected value='<c:out value="${entry.value}"/>'>
								<c:out value="${entry.key}" />
							</option>
						</c:when>
						<c:otherwise>
							<option value='<c:out value="${entry.value}"/>'>
								<c:out value="${entry.key}" />
							</option>
						</c:otherwise>
					</c:choose>
				</c:forEach>
			</select> <br />
		</div>

		<input name="submit_trigger" type="checkbox" style="display: none;" /> <input
			name="find_shipping_lines" type="button" value="Search" onClick="submitButtonClick()">
	</form>
</div>

XML configs.

portlet.xml

<?xml version="1.0"?>

<portlet-app
	version="2.0"
	xmlns="http://java.sun.com/xml/ns/portlet/portlet-app_2_0.xsd"
	xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
	xsi:schemaLocation="http://java.sun.com/xml/ns/portlet/portlet-app_2_0.xsd http://java.sun.com/xml/ns/portlet/portlet-app_2_0.xsd"
>
	<portlet>
		<portlet-name>PORTLETNAME</portlet-name>
		<display-name>PORTLETNAME</display-name>
		<portlet-class>some.package.here</portlet-class>
		<init-param>
			<name>view-jsp</name>
			<value>/view.jsp</value>
		</init-param>
		<expiration-cache>0</expiration-cache>
		<supports>
			<mime-type>text/html</mime-type>
		</supports>
		<portlet-info>
			<title>PORTLETNAME</title>
			<short-title>PORTLETNAME</short-title>
			<keywords>PORTLETNAME</keywords>
		</portlet-info>
		<security-role-ref>
			<role-name>administrator</role-name>
		</security-role-ref>
		<security-role-ref>
			<role-name>guest</role-name>
		</security-role-ref>
		<security-role-ref>
			<role-name>power-user</role-name>
		</security-role-ref>
		<security-role-ref>
			<role-name>user</role-name>
		</security-role-ref>
	</portlet>
</portlet-app>

liferay-portlet.xml

<?xml version="1.0"?>
<!DOCTYPE liferay-portlet-app PUBLIC "-//Liferay//DTD Portlet Application 6.0.0//EN" "http://www.liferay.com/dtd/liferay-portlet-app_6_0_0.dtd">

<liferay-portlet-app>
	<portlet>
		<portlet-name>PORTLETNAME</portlet-name>
		<icon>/icon.png</icon>
		<instanceable>true</instanceable>
		<header-portlet-css>/css/main.css</header-portlet-css>
		<header-portlet-javascript>/js/main.js</header-portlet-javascript>
		<css-class-wrapper>PORTLETNAME</css-class-wrapper>
	</portlet>
	<role-mapper>
		<role-name>administrator</role-name>
		<role-link>Administrator</role-link>
	</role-mapper>
	<role-mapper>
		<role-name>guest</role-name>
		<role-link>Guest</role-link>
	</role-mapper>
	<role-mapper>
		<role-name>power-user</role-name>
		<role-link>Power User</role-link>
	</role-mapper>
	<role-mapper>
		<role-name>user</role-name>
		<role-link>User</role-link>
	</role-mapper>
</liferay-portlet-app>

Thanks for any help!

[Guy deLyonesse]

You're setting an awful lot of request attributes. Are you using a new instance of the browser between testing sessions?

[Al Razor]

Guy deLyonesse wrote:You're setting an awful lot of request attributes. Are you using a new instance of the browser between testing sessions?

Hello Guy,

Yes, I always remove cookies and session after each test.
Also, I was able to solve this problem by removing class members and moving the logic to processAction.