gatein SVN: r8572 - components/wsrp/trunk/jcr-impl/src/main/java/org/gatein/wsrp/consumer/registry.
by do-not-reply@jboss.org
Author: chris.laprun(a)jboss.com
Date: 2012-03-12 15:37:32 -0400 (Mon, 12 Mar 2012)
New Revision: 8572
Modified:
components/wsrp/trunk/jcr-impl/src/main/java/org/gatein/wsrp/consumer/registry/JCRConsumerRegistry.java
Log:
- GTNWSRP-279: Avoid updating ProducerInfo when not needed and added changes detection.
Modified: components/wsrp/trunk/jcr-impl/src/main/java/org/gatein/wsrp/consumer/registry/JCRConsumerRegistry.java
===================================================================
--- components/wsrp/trunk/jcr-impl/src/main/java/org/gatein/wsrp/consumer/registry/JCRConsumerRegistry.java 2012-03-12 19:35:45 UTC (rev 8571)
+++ components/wsrp/trunk/jcr-impl/src/main/java/org/gatein/wsrp/consumer/registry/JCRConsumerRegistry.java 2012-03-12 19:37:32 UTC (rev 8572)
@@ -1,6 +1,6 @@
/*
* JBoss, a division of Red Hat
- * Copyright 2011, Red Hat Middleware, LLC, and individual
+ * Copyright 2012, Red Hat Middleware, LLC, and individual
* contributors as indicated by the @authors tag. See the
* copyright.txt in the distribution for a full listing of
* individual contributors.
@@ -254,7 +254,8 @@
}
else
{
- throw new IllegalArgumentException("There is no ProducerInfo with id '" + id + "'");
+ log.debug("There is no ProducerInfo with id '" + id + "'. Return Long.MIN_VALUE for last modified time.");
+ return Long.MIN_VALUE;
}
}
finally
12 years, 10 months
gatein SVN: r8571 - components/wsrp/trunk.
by do-not-reply@jboss.org
Author: chris.laprun(a)jboss.com
Date: 2012-03-12 15:35:45 -0400 (Mon, 12 Mar 2012)
New Revision: 8571
Modified:
components/wsrp/trunk/pom.xml
Log:
- Updated to Chromattic 1.1.3.
Modified: components/wsrp/trunk/pom.xml
===================================================================
--- components/wsrp/trunk/pom.xml 2012-03-12 19:29:39 UTC (rev 8570)
+++ components/wsrp/trunk/pom.xml 2012-03-12 19:35:45 UTC (rev 8571)
@@ -61,7 +61,7 @@
<jsf.version>1.2_12</jsf.version>
<javax.xml.ws.version>2.1</javax.xml.ws.version>
<javax.xml.soap.saaj.version>1.3</javax.xml.soap.saaj.version>
- <org.chromattic.version>1.1.2</org.chromattic.version>
+ <org.chromattic.version>1.1.3</org.chromattic.version>
<org.jboss.arquillian.version>1.0.0.Alpha2</org.jboss.arquillian.version>
<org.mockito.version>1.8.5</org.mockito.version>
</properties>
12 years, 10 months
gatein SVN: r8570 - in components/wsrp/branches/2.1.x/consumer/src: main/java/org/gatein/wsrp/consumer/registry and 1 other directories.
by do-not-reply@jboss.org
Author: chris.laprun(a)jboss.com
Date: 2012-03-12 15:29:39 -0400 (Mon, 12 Mar 2012)
New Revision: 8570
Modified:
components/wsrp/branches/2.1.x/consumer/src/main/java/org/gatein/wsrp/consumer/ProducerInfo.java
components/wsrp/branches/2.1.x/consumer/src/main/java/org/gatein/wsrp/consumer/RegistrationInfo.java
components/wsrp/branches/2.1.x/consumer/src/main/java/org/gatein/wsrp/consumer/registry/AbstractConsumerRegistry.java
components/wsrp/branches/2.1.x/consumer/src/main/java/org/gatein/wsrp/consumer/registry/InMemoryConsumerRegistry.java
components/wsrp/branches/2.1.x/consumer/src/test/java/org/gatein/wsrp/consumer/ProducerInfoTestCase.java
Log:
- GTNWSRP-279: avoid updating ProducerInfo when it's not needed and added modification detection.
Modified: components/wsrp/branches/2.1.x/consumer/src/main/java/org/gatein/wsrp/consumer/ProducerInfo.java
===================================================================
--- components/wsrp/branches/2.1.x/consumer/src/main/java/org/gatein/wsrp/consumer/ProducerInfo.java 2012-03-12 19:29:29 UTC (rev 8569)
+++ components/wsrp/branches/2.1.x/consumer/src/main/java/org/gatein/wsrp/consumer/ProducerInfo.java 2012-03-12 19:29:39 UTC (rev 8570)
@@ -105,7 +105,6 @@
/** The activated status of the associated Consumer */
private boolean persistentActive;
- // GTNWSRP-239: information that's currently transient but should probably be persistent
/**
* GTNWSRP-239: whether or not this ProducerInfo requires ModifyRegistration to be called, currently persisted via
* mixin
@@ -175,7 +174,7 @@
{
return false;
}
- if (!persistentId.equals(that.persistentId))
+ if (!getId().equals(that.getId()))
{
return false;
}
@@ -187,7 +186,7 @@
public int hashCode()
{
int result = key != null ? key.hashCode() : 0;
- result = 31 * result + persistentId.hashCode();
+ result = 31 * result + getId().hashCode();
return result;
}
@@ -197,7 +196,7 @@
final StringBuilder sb = new StringBuilder();
sb.append("ProducerInfo");
sb.append("{key='").append(key).append('\'');
- sb.append(", id='").append(persistentId).append('\'');
+ sb.append(", id='").append(getId()).append('\'');
sb.append('}');
return sb.toString();
}
@@ -297,13 +296,46 @@
*/
public void setActive(boolean active)
{
+ setInternalActive(active);
+ }
+
+ private boolean setInternalActive(boolean active)
+ {
+ final boolean modified = modifyNowIfNeeded(persistentActive, active);
this.persistentActive = active;
+ return modified;
}
+ public String getId()
+ {
+ return persistentId;
+ }
+
+ public void setId(String id)
+ {
+ modifyNowIfNeeded(persistentId, id);
+ this.persistentId = id;
+ }
+
+ private boolean modifyNowIfNeeded(Object oldValue, Object newValue)
+ {
+ if (ParameterValidation.isOldAndNewDifferent(oldValue, newValue))
+ {
+ modifyNow();
+ return true;
+ }
+ else
+ {
+ return false;
+ }
+ }
+
public void setActiveAndSave(boolean active)
{
- setActive(active);
- registry.updateProducerInfo(this);
+ if(setInternalActive(active))
+ {
+ registry.updateProducerInfo(this);
+ }
}
public boolean isModifyRegistrationRequired()
@@ -315,6 +347,7 @@
public void setModifyRegistrationRequired(boolean modifyRegistrationRequired)
{
+ modifyNowIfNeeded(isModifyRegistrationRequired, modifyRegistrationRequired);
this.isModifyRegistrationRequired = modifyRegistrationRequired;
}
@@ -562,16 +595,16 @@
}
result.setRegistrationResult(registrationResult);
-
- return result;
}
else
{
log.debug("Registration not required");
persistentRegistrationInfo = new RegistrationInfo(this, false);
extractOfferedPortlets(serviceDescription);
- return result;
}
+
+ modifyNow();
+ return result;
}
private Map<String, ItemDescription> toMap(List<ItemDescription> itemDescriptions)
@@ -591,16 +624,6 @@
}
}
- public String getId()
- {
- return persistentId;
- }
-
- public void setId(String id)
- {
- this.persistentId = id;
- }
-
/**
* Extracts a map of offered Portlet objects from ServiceDescription
*
@@ -661,7 +684,7 @@
{
log.warn("Portlet '" + portletHandle
+ "' uses the GET method in forms. Since we don't handle this, this portlet will be excluded from " +
- "the list of offered portlets for producer " + persistentId);
+ "the list of offered portlets for producer " + getId());
}
else
{
@@ -737,7 +760,7 @@
}
catch (Exception e)
{
- log.debug("Couldn't get portlet via getPortletDescription for producer '" + persistentId
+ log.debug("Couldn't get portlet via getPortletDescription for producer '" + getId()
+ "'. Attempting to retrieve it from the service description as this producer might not support the PortletManagement interface.", e);
justRefreshed = refresh(true);
@@ -819,24 +842,26 @@
public void setExpirationCacheSeconds(Integer expirationCacheSeconds)
{
- // record the previous cache expiration duration
- Integer previousMS = getSafeExpirationCacheSeconds() * 1000;
+ if (modifyNowIfNeeded(persistentExpirationCacheSeconds, expirationCacheSeconds))
+ {
+ // record the previous cache expiration duration
+ Integer previousMS = getSafeExpirationCacheSeconds() * 1000;
- // assign the new value
- this.persistentExpirationCacheSeconds = expirationCacheSeconds;
+ // assign the new value
+ this.persistentExpirationCacheSeconds = expirationCacheSeconds;
- // recompute the expiration time based on previous value and new one
- long lastExpirationTimeChange = expirationTimeMillis - previousMS;
- int newMS = getSafeExpirationCacheSeconds() * 1000;
- if (lastExpirationTimeChange > 0)
- {
- expirationTimeMillis = lastExpirationTimeChange + newMS;
+ // recompute the expiration time based on previous value and new one
+ long lastExpirationTimeChange = expirationTimeMillis - previousMS;
+ int newMS = getSafeExpirationCacheSeconds() * 1000;
+ if (lastExpirationTimeChange > 0)
+ {
+ expirationTimeMillis = lastExpirationTimeChange + newMS;
+ }
+ else
+ {
+ expirationTimeMillis = System.currentTimeMillis();
+ }
}
- else
- {
- expirationTimeMillis = System.currentTimeMillis();
- }
-
}
/**
@@ -972,7 +997,7 @@
{
Throwable cause = e.getCause();
throw new InvokerUnavailableException("Problem getting service description for producer "
- + persistentId + ", please see the logs for more information. ", cause == null ? e : cause);
+ + getId() + ", please see the logs for more information. ", cause == null ? e : cause);
}
public RegistrationContext getRegistrationContext() throws PortletInvokerException
@@ -990,9 +1015,15 @@
persistentRegistrationInfo.resetRegistration();
invalidateCache();
+ modifyNow();
registry.updateProducerInfo(this);
}
+ void modifyNow()
+ {
+ setLastModified(System.currentTimeMillis());
+ }
+
// make package only after package reorg
public PortletPropertyDescriptionResponse getPropertyDescriptionsFor(String portletHandle)
@@ -1082,7 +1113,7 @@
if (serviceDescription.isRequiresRegistration())
{
// check if the configured registration information is correct and if we can get the service description
- RefreshResult result = persistentRegistrationInfo.refresh(serviceDescription, persistentId, true, forceRefresh, false);
+ RefreshResult result = persistentRegistrationInfo.refresh(serviceDescription, getId(), true, forceRefresh, false);
if (!result.hasIssues())
{
try
@@ -1110,7 +1141,7 @@
if (debug)
{
- String msg = "Consumer with id '" + persistentId + "' successfully registered with handle: '"
+ String msg = "Consumer with id '" + getId() + "' successfully registered with handle: '"
+ registrationContext.getRegistrationHandle() + "'";
log.debug(msg);
}
@@ -1124,7 +1155,7 @@
{
persistentRegistrationInfo.resetRegistration();
setActive(false);
- throw new PortletInvokerException("Couldn't register with producer '" + persistentId + "'", e);
+ throw new PortletInvokerException("Couldn't register with producer '" + getId() + "'", e);
}
}
else
@@ -1149,11 +1180,11 @@
{
RegistrationContext registrationContext = getRegistrationContext();
persistentEndpointInfo.getRegistrationService().deregister(registrationContext, UserAccess.getUserContext());
- log.info("Consumer with id '" + persistentId + "' deregistered.");
+ log.info("Consumer with id '" + getId() + "' deregistered.");
}
catch (Exception e)
{
- throw new PortletInvokerException("Couldn't deregister with producer '" + persistentId + "'", e);
+ throw new PortletInvokerException("Couldn't deregister with producer '" + getId() + "'", e);
}
finally
{
@@ -1162,7 +1193,7 @@
}
else
{
- throw new IllegalStateException("Cannot deregister producer '" + persistentId + "' as it's not registered");
+ throw new IllegalStateException("Cannot deregister producer '" + getId() + "' as it's not registered");
}
}
@@ -1210,20 +1241,20 @@
// update state
persistentRegistrationInfo.setRegistrationState(registrationState.value);
- log.info("Consumer with id '" + persistentId + "' sucessfully modified its registration.");
+ log.info("Consumer with id '" + getId() + "' sucessfully modified its registration.");
// reset cache to be able to see new offered portlets on the next refresh
invalidateCache();
}
catch (Exception e)
{
- throw new PortletInvokerException("Couldn't modify registration with producer '" + persistentId + "'", e);
+ throw new PortletInvokerException("Couldn't modify registration with producer '" + getId() + "'", e);
}
}
}
else
{
- throw new IllegalStateException("Cannot modify registration for producer '" + persistentId
+ throw new IllegalStateException("Cannot modify registration for producer '" + getId()
+ "' as it's not registered");
}
}
@@ -1239,9 +1270,9 @@
private RefreshResult internalRefreshRegistration(ServiceDescription serviceDescription, boolean mergeWithLocalInfo, boolean forceRefresh, boolean forceCheckOfExtraProps) throws PortletInvokerException
{
RefreshResult result =
- persistentRegistrationInfo.refresh(serviceDescription, persistentId, mergeWithLocalInfo, forceRefresh, forceCheckOfExtraProps);
+ persistentRegistrationInfo.refresh(serviceDescription, getId(), mergeWithLocalInfo, forceRefresh, forceCheckOfExtraProps);
- log.debug("Refreshed registration information for consumer with id '" + persistentId + "'");
+ log.debug("Refreshed registration information for consumer with id '" + getId() + "'");
return result;
}
@@ -1253,7 +1284,7 @@
|| persistentEndpointInfo.isRefreshNeeded();
if (result)
{
- log.debug("Refresh needed for producer '" + persistentId + "'");
+ log.debug("Refresh needed for producer '" + getId() + "'");
}
return result;
}
Modified: components/wsrp/branches/2.1.x/consumer/src/main/java/org/gatein/wsrp/consumer/RegistrationInfo.java
===================================================================
--- components/wsrp/branches/2.1.x/consumer/src/main/java/org/gatein/wsrp/consumer/RegistrationInfo.java 2012-03-12 19:29:29 UTC (rev 8569)
+++ components/wsrp/branches/2.1.x/consumer/src/main/java/org/gatein/wsrp/consumer/RegistrationInfo.java 2012-03-12 19:29:39 UTC (rev 8570)
@@ -745,6 +745,9 @@
}
regenerateRegistrationData = true;
+
+ // make sure that the parent is marked as modified so that changes can be properly saved
+ parent.modifyNow();
}
private void setModifyRegistrationNeeded(boolean modifyRegistrationNeeded)
Modified: components/wsrp/branches/2.1.x/consumer/src/main/java/org/gatein/wsrp/consumer/registry/AbstractConsumerRegistry.java
===================================================================
--- components/wsrp/branches/2.1.x/consumer/src/main/java/org/gatein/wsrp/consumer/registry/AbstractConsumerRegistry.java 2012-03-12 19:29:29 UTC (rev 8569)
+++ components/wsrp/branches/2.1.x/consumer/src/main/java/org/gatein/wsrp/consumer/registry/AbstractConsumerRegistry.java 2012-03-12 19:29:39 UTC (rev 8570)
@@ -65,7 +65,7 @@
private static final String CONSUMER_WITH_ID = "Consumer with id '";
private static final String RELEASE_SESSIONS_LISTENER = "release_sessions_listener_";
- private static final Logger log = LoggerFactory.getLogger(AbstractConsumerRegistry.class);
+ protected static final Logger log = LoggerFactory.getLogger(AbstractConsumerRegistry.class);
protected ConsumerCache consumerCache;
@@ -268,7 +268,7 @@
ProducerInfo producerInfo = loadProducerInfo(id);
if (producerInfo == null)
{
- return Long.MAX_VALUE;
+ return Long.MIN_VALUE;
}
else
{
@@ -280,25 +280,33 @@
{
ParameterValidation.throwIllegalArgExceptionIfNull(producerInfo, "ProducerInfo");
- String oldId = update(producerInfo);
-
- // if we updated and oldId is not null, we need to update the local information
- if (oldId != null)
+ // only save producer info if we have local modifications that postdate last persisted change
+ if (producerInfo.getLastModified() > getPersistedLastModifiedForProducerInfoWith(producerInfo.getId()))
{
- WSRPConsumer consumer = createConsumerFrom(producerInfo, true);
+ String oldId = update(producerInfo);
- // update the federating portlet invoker if needed
- if (federatingPortletInvoker.isResolved(oldId))
+ // if we updated and oldId is not null, we need to update the local information
+ if (oldId != null)
{
- federatingPortletInvoker.unregisterInvoker(oldId);
+ WSRPConsumer consumer = createConsumerFrom(producerInfo, true);
+
+ // update the federating portlet invoker if needed
+ if (federatingPortletInvoker.isResolved(oldId))
+ {
+ federatingPortletInvoker.unregisterInvoker(oldId);
+ }
+
+ // update cache
+ consumerCache.removeConsumer(oldId);
+ consumerCache.putConsumer(producerInfo.getId(), consumer);
}
- // update cache
- consumerCache.removeConsumer(oldId);
- consumerCache.putConsumer(producerInfo.getId(), consumer);
+ return oldId;
}
-
- return oldId;
+ else
+ {
+ return null;
+ }
}
public void start() throws Exception
Modified: components/wsrp/branches/2.1.x/consumer/src/main/java/org/gatein/wsrp/consumer/registry/InMemoryConsumerRegistry.java
===================================================================
--- components/wsrp/branches/2.1.x/consumer/src/main/java/org/gatein/wsrp/consumer/registry/InMemoryConsumerRegistry.java 2012-03-12 19:29:29 UTC (rev 8569)
+++ components/wsrp/branches/2.1.x/consumer/src/main/java/org/gatein/wsrp/consumer/registry/InMemoryConsumerRegistry.java 2012-03-12 19:29:39 UTC (rev 8570)
@@ -134,7 +134,8 @@
{
if (keysToIds.containsValue(id))
{
- return consumers.get(id).getProducerInfo();
+ final WSRPConsumer consumer = consumers.get(id);
+ return consumer != null ? consumer.getProducerInfo() : null;
}
else
{
Modified: components/wsrp/branches/2.1.x/consumer/src/test/java/org/gatein/wsrp/consumer/ProducerInfoTestCase.java
===================================================================
--- components/wsrp/branches/2.1.x/consumer/src/test/java/org/gatein/wsrp/consumer/ProducerInfoTestCase.java 2012-03-12 19:29:29 UTC (rev 8569)
+++ components/wsrp/branches/2.1.x/consumer/src/test/java/org/gatein/wsrp/consumer/ProducerInfoTestCase.java 2012-03-12 19:29:39 UTC (rev 8570)
@@ -81,6 +81,64 @@
info.setEndpointConfigurationInfo(eci);
}
+ public void testSettersWithoutModificationShouldNotChangeLastModified()
+ {
+ final long initial = info.getLastModified();
+
+ info.setActive(info.isActive());
+ assertEquals(initial, info.getLastModified());
+
+ info.setActiveAndSave(info.isActive());
+ assertEquals(initial, info.getLastModified());
+
+ info.setExpirationCacheSeconds(info.getExpirationCacheSeconds());
+ assertEquals(initial, info.getLastModified());
+
+ info.setId(info.getId());
+ assertEquals(initial, info.getLastModified());
+
+ info.setModifyRegistrationRequired(info.isModifyRegistrationRequired());
+ assertEquals(initial, info.getLastModified());
+ }
+
+ public void testSettersWithModificationShouldChangeLastModified() throws InterruptedException
+ {
+ long initial = info.getLastModified();
+ Thread.sleep(10); // to allow for System.currentTimeMillis() to catch up
+ info.setActive(!info.isActive());
+
+ initial = info.getLastModified();
+ Thread.sleep(10); // to allow for System.currentTimeMillis() to catch up
+ info.setActiveAndSave(!info.isActive());
+ assertTrue(initial != info.getLastModified());
+
+ initial = info.getLastModified();
+ Thread.sleep(10); // to allow for System.currentTimeMillis() to catch up
+ info.setExpirationCacheSeconds(info.getExpirationCacheSeconds() + 1);
+ assertTrue(initial != info.getLastModified());
+
+ initial = info.getLastModified();
+ Thread.sleep(10); // to allow for System.currentTimeMillis() to catch up
+ info.setId(info.getId() + "other");
+ assertTrue(initial != info.getLastModified());
+
+ initial = info.getLastModified();
+ Thread.sleep(10); // to allow for System.currentTimeMillis() to catch up
+ info.setModifyRegistrationRequired(!info.isModifyRegistrationRequired());
+ assertTrue(initial != info.getLastModified());
+ }
+
+ public void testSetKeyDoesNotChangeLastModified()
+ {
+ long initial = info.getLastModified();
+ info.setKey(info.getKey());
+ assertEquals(initial, info.getLastModified());
+
+ initial = info.getLastModified();
+ info.setKey(info.getKey() + "other");
+ assertEquals(initial, info.getLastModified());
+ }
+
public void testSetRegistrationInfo()
{
RegistrationInfo regInfo = new RegistrationInfo(info);
12 years, 10 months
gatein SVN: r8569 - components/wsrp/branches/2.1.x/jcr-impl/src/main/java/org/gatein/wsrp/consumer/registry.
by do-not-reply@jboss.org
Author: chris.laprun(a)jboss.com
Date: 2012-03-12 15:29:29 -0400 (Mon, 12 Mar 2012)
New Revision: 8569
Modified:
components/wsrp/branches/2.1.x/jcr-impl/src/main/java/org/gatein/wsrp/consumer/registry/JCRConsumerRegistry.java
Log:
- GTNWSRP-279: avoid updating ProducerInfo when it's not needed and added modification detection.
Modified: components/wsrp/branches/2.1.x/jcr-impl/src/main/java/org/gatein/wsrp/consumer/registry/JCRConsumerRegistry.java
===================================================================
--- components/wsrp/branches/2.1.x/jcr-impl/src/main/java/org/gatein/wsrp/consumer/registry/JCRConsumerRegistry.java 2012-03-12 19:16:53 UTC (rev 8568)
+++ components/wsrp/branches/2.1.x/jcr-impl/src/main/java/org/gatein/wsrp/consumer/registry/JCRConsumerRegistry.java 2012-03-12 19:29:29 UTC (rev 8569)
@@ -254,7 +254,8 @@
}
else
{
- throw new IllegalArgumentException("There is no ProducerInfo with id '" + id + "'");
+ log.debug("There is no ProducerInfo with id '" + id + "'. Return Long.MIN_VALUE for last modified time.");
+ return Long.MIN_VALUE;
}
}
finally
12 years, 10 months
gatein SVN: r8568 - components/wsrp/branches/2.1.x.
by do-not-reply@jboss.org
Author: chris.laprun(a)jboss.com
Date: 2012-03-12 15:16:53 -0400 (Mon, 12 Mar 2012)
New Revision: 8568
Modified:
components/wsrp/branches/2.1.x/pom.xml
Log:
- Updated to Chromattic 1.1.3.
Modified: components/wsrp/branches/2.1.x/pom.xml
===================================================================
--- components/wsrp/branches/2.1.x/pom.xml 2012-03-12 18:43:14 UTC (rev 8567)
+++ components/wsrp/branches/2.1.x/pom.xml 2012-03-12 19:16:53 UTC (rev 8568)
@@ -60,7 +60,7 @@
<jsf.version>1.2_12</jsf.version>
<javax.xml.ws.version>2.1</javax.xml.ws.version>
<javax.xml.soap.saaj.version>1.3</javax.xml.soap.saaj.version>
- <org.chromattic.version>1.1.2</org.chromattic.version>
+ <org.chromattic.version>1.1.3</org.chromattic.version>
<org.jboss.arquillian.version>1.0.0.Alpha2</org.jboss.arquillian.version>
<org.mockito.version>1.8.5</org.mockito.version>
</properties>
12 years, 10 months
gatein SVN: r8567 - portal/trunk/web/portal/src/main/webapp/WEB-INF/classes/locale/portal.
by do-not-reply@jboss.org
Author: chris.laprun(a)jboss.com
Date: 2012-03-12 14:43:14 -0400 (Mon, 12 Mar 2012)
New Revision: 8567
Modified:
portal/trunk/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_en.properties
Log:
- Removed duplicated properties.
Modified: portal/trunk/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_en.properties
===================================================================
--- portal/trunk/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_en.properties 2012-03-12 18:41:26 UTC (rev 8566)
+++ portal/trunk/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_en.properties 2012-03-12 18:43:14 UTC (rev 8567)
@@ -572,10 +572,7 @@
UIPageNodeForm.tab.label.PageNodeSetting=Page Node Setting
UIPageNodeForm.tab.label.Icon=#{word.icon}
UIPageNodeForm.tab.label.UIPageSelector=Page Selector
-UIPageNodeForm.action.Save=#{word.save}
-UIPageNodeForm.action.Back=#{word.back}
-
#############################################################################
# org.exoplatform.portal.component.customization.UIPageBrowser #
#############################################################################
12 years, 10 months
gatein SVN: r8566 - portal/trunk/web/portal/src/main/webapp/WEB-INF/classes/locale/portal.
by do-not-reply@jboss.org
Author: chris.laprun(a)jboss.com
Date: 2012-03-12 14:41:26 -0400 (Mon, 12 Mar 2012)
New Revision: 8566
Modified:
portal/trunk/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_en.properties
portal/trunk/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_fr.properties
portal/trunk/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_ja.properties
Log:
- Updated copyright information.
Modified: portal/trunk/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_en.properties
===================================================================
--- portal/trunk/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_en.properties 2012-03-12 18:35:45 UTC (rev 8565)
+++ portal/trunk/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_en.properties 2012-03-12 18:41:26 UTC (rev 8566)
@@ -284,7 +284,7 @@
UILoginForm.label.Signin=Sign in
UILoginForm.label.ForAccount=Register now for an account
UILoginForm.label.SigninFail=Sign in failed. Wrong username or password.
-UILoginForm.label.Copyright=Copyright © 2011. All rights reserved, Red Hat, Inc and eXo Platform SAS
+UILoginForm.label.Copyright=Copyright © 2009-2012. All rights reserved, Red Hat, Inc and eXo Platform SAS
#############################################################################
# Message Error #
Modified: portal/trunk/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_fr.properties
===================================================================
--- portal/trunk/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_fr.properties 2012-03-12 18:35:45 UTC (rev 8565)
+++ portal/trunk/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_fr.properties 2012-03-12 18:41:26 UTC (rev 8566)
@@ -238,7 +238,7 @@
UILoginForm.label.Signup=Enregistrement
UILoginForm.label.ForAccount=Créer un compte
UILoginForm.label.SigninFail=La connexion a échouée. Le nom d'utilisateur ou mot de passe est incorrecte.
-UILoginForm.label.Copyright=Copyright © 2011. All rights reserved, Red Hat, Inc. and eXo Platform SAS
+UILoginForm.label.Copyright=Copyright © 2009-2012. All rights reserved, Red Hat, Inc. and eXo Platform SAS
#############################################################################
# Message Error #
Modified: portal/trunk/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_ja.properties
===================================================================
--- portal/trunk/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_ja.properties 2012-03-12 18:35:45 UTC (rev 8565)
+++ portal/trunk/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_ja.properties 2012-03-12 18:41:26 UTC (rev 8566)
@@ -253,7 +253,7 @@
UILoginForm.label.Signin=\u30b5\u30a4\u30f3\u30a4\u30f3
UILoginForm.label.ForAccount=\u30a2\u30ab\u30a6\u30f3\u30c8\u767b\u9332
UILoginForm.label.SigninFail=\u30b5\u30a4\u30f3\u30a4\u30f3\u306b\u5931\u6557\u3057\u307e\u3057\u305f\u3002\u8aa4\u3063\u305f\u30e6\u30fc\u30b6\u30fc\u540d\u307e\u305f\u306f\u30d1\u30b9\u30ef\u30fc\u30c9\u3067\u3059\u3002
-UILoginForm.label.Copyright=Copyright © 2009-2011. All rights reserved, Red Hat, Inc
+UILoginForm.label.Copyright=Copyright © 2009-2012. All rights reserved, Red Hat, Inc and eXo Platform SAS
#############################################################################
12 years, 10 months
gatein SVN: r8565 - epp/portal/branches/EPP_5_2_Branch/web/portal/src/main/webapp/WEB-INF/classes/locale/portal.
by do-not-reply@jboss.org
Author: chris.laprun(a)jboss.com
Date: 2012-03-12 14:35:45 -0400 (Mon, 12 Mar 2012)
New Revision: 8565
Modified:
epp/portal/branches/EPP_5_2_Branch/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_en.properties
Log:
- Removed duplicated properties.
Modified: epp/portal/branches/EPP_5_2_Branch/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_en.properties
===================================================================
--- epp/portal/branches/EPP_5_2_Branch/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_en.properties 2012-03-12 18:32:43 UTC (rev 8564)
+++ epp/portal/branches/EPP_5_2_Branch/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_en.properties 2012-03-12 18:35:45 UTC (rev 8565)
@@ -556,10 +556,7 @@
UIPageNodeForm.tab.label.PageNodeSetting=Page Node Setting
UIPageNodeForm.tab.label.Icon=#{word.icon}
UIPageNodeForm.tab.label.UIPageSelector=Page Selector
-UIPageNodeForm.action.Save=#{word.save}
-UIPageNodeForm.action.Back=#{word.back}
-
#############################################################################
# org.exoplatform.portal.component.customization.UIPageBrowser #
#############################################################################
12 years, 10 months
gatein SVN: r8564 - epp/portal/branches/EPP_5_2_Branch/web/portal/src/main/webapp/WEB-INF/classes/locale/portal.
by do-not-reply@jboss.org
Author: chris.laprun(a)jboss.com
Date: 2012-03-12 14:32:43 -0400 (Mon, 12 Mar 2012)
New Revision: 8564
Modified:
epp/portal/branches/EPP_5_2_Branch/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_en.properties
epp/portal/branches/EPP_5_2_Branch/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_fr.properties
epp/portal/branches/EPP_5_2_Branch/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_ja.properties
Log:
- Updated copyright information.
Modified: epp/portal/branches/EPP_5_2_Branch/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_en.properties
===================================================================
--- epp/portal/branches/EPP_5_2_Branch/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_en.properties 2012-03-12 18:21:31 UTC (rev 8563)
+++ epp/portal/branches/EPP_5_2_Branch/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_en.properties 2012-03-12 18:32:43 UTC (rev 8564)
@@ -268,7 +268,7 @@
UILoginForm.label.Signin=Sign in
UILoginForm.label.ForAccount=Register now for an account
UILoginForm.label.SigninFail=Sign in failed. Wrong username or password.
-UILoginForm.label.Copyright=Copyright © 2009-2010. All rights reserved, Red Hat, Inc
+UILoginForm.label.Copyright=Copyright © 2009-2012. All rights reserved, Red Hat, Inc
#############################################################################
# Message Error #
Modified: epp/portal/branches/EPP_5_2_Branch/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_fr.properties
===================================================================
--- epp/portal/branches/EPP_5_2_Branch/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_fr.properties 2012-03-12 18:21:31 UTC (rev 8563)
+++ epp/portal/branches/EPP_5_2_Branch/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_fr.properties 2012-03-12 18:32:43 UTC (rev 8564)
@@ -238,7 +238,7 @@
UILoginForm.label.Signup=Enregistrement
UILoginForm.label.ForAccount=Créer un compte
UILoginForm.label.SigninFail=La connexion a échouée. Le nom d'utilisateur ou mot de passe est incorrecte.
-UILoginForm.label.Copyright=Copyright © 2009-2010. All rights reserved, Red Hat, Inc.
+UILoginForm.label.Copyright=Copyright © 2009-2012. All rights reserved, Red Hat, Inc.
#############################################################################
# Message Error #
Modified: epp/portal/branches/EPP_5_2_Branch/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_ja.properties
===================================================================
--- epp/portal/branches/EPP_5_2_Branch/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_ja.properties 2012-03-12 18:21:31 UTC (rev 8563)
+++ epp/portal/branches/EPP_5_2_Branch/web/portal/src/main/webapp/WEB-INF/classes/locale/portal/webui_ja.properties 2012-03-12 18:32:43 UTC (rev 8564)
@@ -253,7 +253,7 @@
UILoginForm.label.Signin=\u30b5\u30a4\u30f3\u30a4\u30f3
UILoginForm.label.ForAccount=\u30a2\u30ab\u30a6\u30f3\u30c8\u767b\u9332
UILoginForm.label.SigninFail=\u30b5\u30a4\u30f3\u30a4\u30f3\u306b\u5931\u6557\u3057\u307e\u3057\u305f\u3002\u8aa4\u3063\u305f\u30e6\u30fc\u30b6\u30fc\u540d\u307e\u305f\u306f\u30d1\u30b9\u30ef\u30fc\u30c9\u3067\u3059\u3002
-UILoginForm.label.Copyright=Copyright © 2009-2010. All rights reserved, Red Hat, Inc
+UILoginForm.label.Copyright=Copyright © 2009-2012. All rights reserved, Red Hat, Inc
#############################################################################
12 years, 10 months
gatein SVN: r8563 - epp/portal/branches/EPP_5_2_Branch/webui/portal/src/main/java/org/exoplatform/portal/webui/portal.
by do-not-reply@jboss.org
Author: hfnukal
Date: 2012-03-12 14:21:31 -0400 (Mon, 12 Mar 2012)
New Revision: 8563
Modified:
epp/portal/branches/EPP_5_2_Branch/webui/portal/src/main/java/org/exoplatform/portal/webui/portal/UIPortal.java
Log:
Bug 794410 Navigation loading fails with NPE when previewing a page
Modified: epp/portal/branches/EPP_5_2_Branch/webui/portal/src/main/java/org/exoplatform/portal/webui/portal/UIPortal.java
===================================================================
--- epp/portal/branches/EPP_5_2_Branch/webui/portal/src/main/java/org/exoplatform/portal/webui/portal/UIPortal.java 2012-03-12 04:20:44 UTC (rev 8562)
+++ epp/portal/branches/EPP_5_2_Branch/webui/portal/src/main/java/org/exoplatform/portal/webui/portal/UIPortal.java 2012-03-12 18:21:31 UTC (rev 8563)
@@ -27,8 +27,6 @@
import org.exoplatform.portal.mop.SiteType;
import org.exoplatform.portal.mop.user.UserNavigation;
import org.exoplatform.portal.mop.user.UserNode;
-import org.exoplatform.web.url.navigation.NodeURL;
-import org.exoplatform.web.url.navigation.NavigationResource;
import org.exoplatform.portal.webui.application.UIPortlet;
import org.exoplatform.portal.webui.container.UIContainer;
import org.exoplatform.portal.webui.page.UIPage;
@@ -50,16 +48,16 @@
import org.exoplatform.web.login.LogoutControl;
import org.exoplatform.web.security.security.AbstractTokenService;
import org.exoplatform.web.security.security.CookieTokenService;
+import org.exoplatform.web.url.navigation.NavigationResource;
+import org.exoplatform.web.url.navigation.NodeURL;
import org.exoplatform.webui.application.WebuiRequestContext;
import org.exoplatform.webui.config.annotation.ComponentConfig;
import org.exoplatform.webui.config.annotation.EventConfig;
import org.exoplatform.webui.core.UIComponent;
import org.exoplatform.webui.event.Event;
import org.exoplatform.webui.event.EventListener;
-
import java.util.HashMap;
import java.util.Map;
-
import javax.portlet.WindowState;
import javax.servlet.http.Cookie;
import javax.servlet.http.HttpServletRequest;
@@ -214,8 +212,8 @@
public UserNavigation getUserNavigation() throws Exception
{
- UIPortalApplication uiPortalApp = getAncestorOfType(UIPortalApplication.class);
- return uiPortalApp.getUserPortalConfig().getUserPortal().getNavigation(siteKey);
+ PortalRequestContext prc = Util.getPortalRequestContext();
+ return prc.getUserPortalConfig().getUserPortal().getNavigation(siteKey);
}
/**
12 years, 10 months