You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by no...@apache.org on 2009/11/18 19:47:14 UTC

svn commit: r881871 - /james/hupa/trunk/client/src/main/java/org/apache/hupa/client/mvp/MainPresenter.java

Author: norman
Date: Wed Nov 18 18:47:14 2009
New Revision: 881871

URL: http://svn.apache.org/viewvc?rev=881871&view=rev
Log:
Fix NPE introduced by myself.. shame on me

Modified:
    james/hupa/trunk/client/src/main/java/org/apache/hupa/client/mvp/MainPresenter.java

Modified: james/hupa/trunk/client/src/main/java/org/apache/hupa/client/mvp/MainPresenter.java
URL: http://svn.apache.org/viewvc/james/hupa/trunk/client/src/main/java/org/apache/hupa/client/mvp/MainPresenter.java?rev=881871&r1=881870&r2=881871&view=diff
==============================================================================
--- james/hupa/trunk/client/src/main/java/org/apache/hupa/client/mvp/MainPresenter.java (original)
+++ james/hupa/trunk/client/src/main/java/org/apache/hupa/client/mvp/MainPresenter.java Wed Nov 18 18:47:14 2009
@@ -507,10 +507,14 @@
     public void revealDisplay(User user) {
         this.user = user;
         loadTreeItems();  
-        firePresenterChangedEvent();
         revealDisplay();
     }
     
+    @Override
+    protected void onRevealDisplay() {
+        showMessageTable(user, folder, searchValue);
+        super.onRevealDisplay();
+    }
     
     public void openLink(String url) {
         Window.open(url, "_blank", "");



---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org


Re: svn commit: r881871 - /james/hupa/trunk/client/src/main/java/org/apache/hupa/client/mvp/MainPresenter.java

Posted by Manuel Carrasco MoƱino <ma...@gmail.com>.
Thanks, I was wondering why folder was null in IMAPMessagePresenter

On Wed, Nov 18, 2009 at 7:47 PM, <no...@apache.org> wrote:

> Author: norman
> Date: Wed Nov 18 18:47:14 2009
> New Revision: 881871
>
> URL: http://svn.apache.org/viewvc?rev=881871&view=rev
> Log:
> Fix NPE introduced by myself.. shame on me
>
> Modified:
>
>  james/hupa/trunk/client/src/main/java/org/apache/hupa/client/mvp/MainPresenter.java
>
> Modified:
> james/hupa/trunk/client/src/main/java/org/apache/hupa/client/mvp/MainPresenter.java
> URL:
> http://svn.apache.org/viewvc/james/hupa/trunk/client/src/main/java/org/apache/hupa/client/mvp/MainPresenter.java?rev=881871&r1=881870&r2=881871&view=diff
>
> ==============================================================================
> ---
> james/hupa/trunk/client/src/main/java/org/apache/hupa/client/mvp/MainPresenter.java
> (original)
> +++
> james/hupa/trunk/client/src/main/java/org/apache/hupa/client/mvp/MainPresenter.java
> Wed Nov 18 18:47:14 2009
> @@ -507,10 +507,14 @@
>     public void revealDisplay(User user) {
>         this.user = user;
>         loadTreeItems();
> -        firePresenterChangedEvent();
>         revealDisplay();
>     }
>
> +    @Override
> +    protected void onRevealDisplay() {
> +        showMessageTable(user, folder, searchValue);
> +        super.onRevealDisplay();
> +    }
>
>     public void openLink(String url) {
>         Window.open(url, "_blank", "");
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
> For additional commands, e-mail: server-dev-help@james.apache.org
>
>