Skip to content

Commit

Permalink
feat(unification): replace internal references to interface #370
Browse files Browse the repository at this point in the history
  • Loading branch information
ValentinPostindustria committed Mar 4, 2022
1 parent 318088e commit a73b9ef
Show file tree
Hide file tree
Showing 6 changed files with 18 additions and 10 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@
import org.prebid.mobile.tasksmanager.TasksManager;
import org.prebid.mobile.unification.AdUnitConfiguration;
import org.prebid.mobile.unification.BaseAdUnitConfiguration;
import org.prebid.mobile.unification.BaseAdUnitConfigurationInterface;

import java.util.*;

Expand All @@ -35,7 +36,7 @@ public abstract class AdUnit {

private int periodMillis = 0; // No auto refresh
private DemandFetcher fetcher;
protected BaseAdUnitConfiguration configuration = createConfiguration();
protected BaseAdUnitConfigurationInterface configuration = createConfiguration();

AdUnit(@NonNull String configId, @NonNull AdType adType) {
configuration.setConfigId(configId);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
package org.prebid.mobile;

import androidx.annotation.MainThread;
import org.prebid.mobile.unification.BaseAdUnitConfiguration;
import org.prebid.mobile.unification.BaseAdUnitConfigurationInterface;

import java.util.HashMap;

Expand All @@ -40,7 +40,7 @@ interface DemandAdapter {
* @param auctionId an unique identifier
*/

void requestDemand(BaseAdUnitConfiguration configuration, DemandAdapterListener listener, String auctionId);
void requestDemand(BaseAdUnitConfigurationInterface configuration, DemandAdapterListener listener, String auctionId);

void stopRequest(String auctionId);

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
import androidx.annotation.NonNull;
import androidx.annotation.VisibleForTesting;
import org.prebid.mobile.tasksmanager.TasksManager;
import org.prebid.mobile.unification.BaseAdUnitConfiguration;
import org.prebid.mobile.unification.BaseAdUnitConfigurationInterface;

import java.util.HashMap;
import java.util.UUID;
Expand All @@ -43,7 +43,7 @@ enum STATE {
private RequestRunnable requestRunnable;
private long lastFetchTime = -1;
private long timePausedAt = -1;
private BaseAdUnitConfiguration configuration;
private BaseAdUnitConfigurationInterface configuration;

DemandFetcher(@NonNull Object adObj) {
this.state = STATE.STOPPED;
Expand All @@ -59,7 +59,7 @@ void setListener(OnCompleteListener listener) {
this.listener = listener;
}

void setConfiguration(BaseAdUnitConfiguration configuration) {
void setConfiguration(BaseAdUnitConfigurationInterface configuration) {
this.configuration = configuration;
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@
import org.prebid.mobile.http.NoContextException;
import org.prebid.mobile.http.TaskResult;
import org.prebid.mobile.tasksmanager.TasksManager;
import org.prebid.mobile.unification.BaseAdUnitConfiguration;
import org.prebid.mobile.unification.BaseAdUnitConfigurationInterface;
import org.prebid.mobile.unification.NativeAdUnitConfiguration;

import java.lang.ref.WeakReference;
Expand All @@ -53,7 +53,7 @@ class PrebidServerAdapter implements DemandAdapter {
}

@Override
public void requestDemand(BaseAdUnitConfiguration configuration, DemandAdapterListener listener, String auctionId) {
public void requestDemand(BaseAdUnitConfigurationInterface configuration, DemandAdapterListener listener, String auctionId) {
final ServerConnector connector = new ServerConnector(this, listener, configuration, auctionId, cacheIdSaver);
serverConnectors.add(connector);
connector.execute();
Expand All @@ -78,7 +78,7 @@ static class ServerConnector extends HTTPPost {
private final WeakReference<PrebidServerAdapter> prebidServerAdapter;
private final TimeoutCountDownTimer timeoutCountDownTimer;

private final BaseAdUnitConfiguration configuration;
private final BaseAdUnitConfigurationInterface configuration;
private final String auctionId;

private DemandAdapterListener listener;
Expand All @@ -90,7 +90,7 @@ static class ServerConnector extends HTTPPost {

private CacheIdSaver cacheIdSaver;

ServerConnector(PrebidServerAdapter prebidServerAdapter, DemandAdapterListener listener, BaseAdUnitConfiguration configuration, String auctionId, CacheIdSaver cacheIdSaver) {
ServerConnector(PrebidServerAdapter prebidServerAdapter, DemandAdapterListener listener, BaseAdUnitConfigurationInterface configuration, String auctionId, CacheIdSaver cacheIdSaver) {
this.prebidServerAdapter = new WeakReference<>(prebidServerAdapter);
this.listener = listener;
this.configuration = configuration;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -139,13 +139,15 @@ public void clearContextKeywords() {
contextKeywordsSet.clear();
}

@Override
public AdUnitConfiguration castToOriginal() {
if (this instanceof AdUnitConfiguration) {
return (AdUnitConfiguration) this;
}
throw new IllegalStateException("Can't cast to AdUnitConfiguration.");
}

@Override
public NativeAdUnitConfiguration castToNative() {
if (this instanceof NativeAdUnitConfiguration) {
return (NativeAdUnitConfiguration) this;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -58,4 +58,9 @@ public abstract interface BaseAdUnitConfigurationInterface {

public void clearContextKeywords();


public AdUnitConfiguration castToOriginal();

public NativeAdUnitConfiguration castToNative();

}

0 comments on commit a73b9ef

Please sign in to comment.