Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(904)

Issue 3982052: Use GADGET context by default fro bad RenderingContext value

Can't Edit
Can't Publish+Mail
Start Review
Created:
15 years, 1 month ago by zhoresh
Modified:
15 years, 1 month ago
Reviewers:
johnfargo, dev-remailer
Base URL:
http://svn.apache.org/repos/asf/shindig/trunk
Visibility:
Public.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -9 lines) Patch
java/gadgets/src/main/java/org/apache/shindig/gadgets/RenderingContext.java View 1 chunk +10 lines, -9 lines 0 comments Download

Messages

Total messages: 2
zhoresh
15 years, 1 month ago (2011-01-27 21:39:02 UTC) #1
johnfargo
15 years, 1 month ago (2011-01-27 22:48:03 UTC) #2
LGTM++

On Thu, Jan 27, 2011 at 1:39 PM, <zhoresh@gmail.com> wrote:

> Reviewers: dev-remailer_shindig.apache.org, johnfargo,
>
>
>
> Please review this at http://codereview.appspot.com/3982052/
>
> Affected files:
>
>  java/gadgets/src/main/java/org/apache/shindig/gadgets/RenderingContext.java
>
>
> ### Eclipse Workspace Patch 1.0
> #P shindig-project
> Index:
> java/gadgets/src/main/java/org/apache/shindig/gadgets/RenderingContext.java
> ===================================================================
> ---
> java/gadgets/src/main/java/org/apache/shindig/gadgets/RenderingContext.java
> (revision 1064259)
> +++
> java/gadgets/src/main/java/org/apache/shindig/gadgets/RenderingContext.java
> (working copy)
> @@ -62,16 +62,17 @@
>   }
>
>   public static RenderingContext valueOfParam(String param) {
> -    // Exception: when no &c= parameter provided, default to GADGET.
> -    if (param == null) {
> -      return RenderingContext.GADGET;
> -    }
> -    for (RenderingContext rc : RenderingContext.values()) {
> -      String rcParam = rc.getParamValue();
> -      if (rcParam != null && rcParam.equals(param)) {
> -        return rc;
> +    // Exception: when no &c= parameter provided or bad, default to
> GADGET.
> +    RenderingContext c = RenderingContext.GADGET;
> +    if (param != null) {
> +      for (RenderingContext rc : RenderingContext.values()) {
> +        String rcParam = rc.getParamValue();
> +        if (rcParam != null && rcParam.equals(param)) {
> +          c = rc;
> +          break;
> +        }
>       }
>     }
> -    return null;
> +    return c;
>   }
>  }
>
>
>
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b