From 9c4f5bc33734da00a092d3694ce78cddf4dac9f6 Mon Sep 17 00:00:00 2001 From: Pablo Fernandez Date: Thu, 13 Jan 2011 23:49:08 -0300 Subject: [PATCH] Refactored OOB checking in fbook api --- src/main/java/org/scribe/builder/api/FacebookApi.java | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/main/java/org/scribe/builder/api/FacebookApi.java b/src/main/java/org/scribe/builder/api/FacebookApi.java index ad57e20..c945520 100644 --- a/src/main/java/org/scribe/builder/api/FacebookApi.java +++ b/src/main/java/org/scribe/builder/api/FacebookApi.java @@ -1,17 +1,16 @@ package org.scribe.builder.api; -import org.scribe.exceptions.*; import org.scribe.model.*; import org.scribe.utils.*; public class FacebookApi extends DefaultApi20 { private static final String AUTHORIZE_URL = "https://graph.facebook.com/oauth/authorize?response_type=token&client_id=%s&redirect_uri=%s"; + @Override public String getAuthorizationUrl(OAuthConfig config) { - if(OAuthConstants.OUT_OF_BAND.equals(config.getCallback())) - throw new OAuthException("Facebook does not support oob authentication."); + Preconditions.checkValidUrl(config.getCallback(), "Must provide a valid url as callback. Facebook does not support OOB"); return String.format(AUTHORIZE_URL, config.getApiKey(), URLUtils.percentEncode(config.getCallback())); } }