diff --git a/briar-android/src/org/briarproject/android/AppModule.java b/briar-android/src/org/briarproject/android/AppModule.java
index 2ad15515cec97ef8be2917063c0a8e39d89f9819..4ab69314a04df18f242d787f831fe3d63ad7d715 100644
--- a/briar-android/src/org/briarproject/android/AppModule.java
+++ b/briar-android/src/org/briarproject/android/AppModule.java
@@ -77,7 +77,7 @@ public class AppModule {
return new DatabaseConfig() {
private volatile SecretKey key = null;
- private volatile String nickName;
+ private volatile String nickname;
@Override
public boolean databaseExists() {
@@ -97,13 +97,13 @@ public class AppModule {
}
@Override
- public void setAuthorNick(String nickName) {
- this.nickName = nickName;
+ public void setAuthorNick(String nickname) {
+ this.nickname = nickname;
}
@Override
public String getAuthorNick() {
- return nickName;
+ return nickname;
}
@Override
diff --git a/briar-android/src/org/briarproject/android/BriarService.java b/briar-android/src/org/briarproject/android/BriarService.java
index 812399e56a5cde3e60d6734563060abde056ce99..b6fc6b28eb2a3d64eaa9778bb7a8d23cff72a1bb 100644
--- a/briar-android/src/org/briarproject/android/BriarService.java
+++ b/briar-android/src/org/briarproject/android/BriarService.java
@@ -93,8 +93,8 @@ public class BriarService extends Service {
new Thread() {
@Override
public void run() {
- StartResult result = lifecycleManager
- .startServices(databaseConfig.getAuthorNick());
+ String nickname = databaseConfig.getAuthorNick();
+ StartResult result = lifecycleManager.startServices(nickname);
if (result == SUCCESS) {
started = true;
} else if (result == ALREADY_RUNNING) {
diff --git a/briar-android/src/org/briarproject/android/SetupActivity.java b/briar-android/src/org/briarproject/android/SetupActivity.java
index e5c1b4b7c1608f2779e09aa8db216d10502617a1..0dcce3d72a28a3d4eeed9bee8584cf1c97e38a99 100644
--- a/briar-android/src/org/briarproject/android/SetupActivity.java
+++ b/briar-android/src/org/briarproject/android/SetupActivity.java
@@ -135,7 +135,7 @@ public class SetupActivity extends BaseActivity implements OnClickListener,
String nickname = nicknameEntry.getText().toString();
String password = passwordEntry.getText().toString();
- setupController.storeAuthorInfo(password, nickname,
+ setupController.storeAuthorInfo(nickname, password,
new UiResultHandler<Void>(this) {
@Override
public void onResultUi(Void result) {
diff --git a/briar-android/src/org/briarproject/android/controller/SetupController.java b/briar-android/src/org/briarproject/android/controller/SetupController.java
index 0a671cf58d79d5486aa983180672cbd13d036db6..947d6991ab946f09a686dca664bd35a8aeec34ac 100644
--- a/briar-android/src/org/briarproject/android/controller/SetupController.java
+++ b/briar-android/src/org/briarproject/android/controller/SetupController.java
@@ -6,7 +6,7 @@ public interface SetupController {
float estimatePasswordStrength(String password);
- void storeAuthorInfo(String password, String nickname,
+ void storeAuthorInfo(String nickname, String password,
ResultHandler<Void> resultHandler);
}
diff --git a/briar-android/src/org/briarproject/android/controller/SetupControllerImpl.java b/briar-android/src/org/briarproject/android/controller/SetupControllerImpl.java
index dc310422d1774a1d6c616026abfc450a48018b16..a881422d63b85235208f10ce68786144c8efcc7a 100644
--- a/briar-android/src/org/briarproject/android/controller/SetupControllerImpl.java
+++ b/briar-android/src/org/briarproject/android/controller/SetupControllerImpl.java
@@ -10,16 +10,12 @@ import org.briarproject.api.crypto.SecretKey;
import org.briarproject.api.db.DatabaseConfig;
import java.util.concurrent.Executor;
-import java.util.logging.Logger;
import javax.inject.Inject;
public class SetupControllerImpl extends PasswordControllerImpl
implements SetupController {
- private static final Logger LOG =
- Logger.getLogger(SetupControllerImpl.class.getName());
-
private final PasswordStrengthEstimator strengthEstimator;
@Inject
@@ -37,16 +33,16 @@ public class SetupControllerImpl extends PasswordControllerImpl
}
@Override
- public void storeAuthorInfo(final String password, final String nickname,
+ public void storeAuthorInfo(final String nickname, final String password,
final ResultHandler<Void> resultHandler) {
cryptoExecutor.execute(new Runnable() {
@Override
public void run() {
+ databaseConfig.setAuthorNick(nickname);
SecretKey key = crypto.generateSecretKey();
databaseConfig.setEncryptionKey(key);
String hex = encryptDatabaseKey(key, password);
storeEncryptedDatabaseKey(hex);
- databaseConfig.setAuthorNick(nickname);
resultHandler.onResult(null);
}
});
diff --git a/briar-android/test/java/org/briarproject/android/SetupActivityTest.java b/briar-android/test/java/org/briarproject/android/SetupActivityTest.java
index f06c4f90c236267476f25e0c40aac477572dbba8..efc0491d7e31bf5b33e6b477af4a6b59af1aaf82 100644
--- a/briar-android/test/java/org/briarproject/android/SetupActivityTest.java
+++ b/briar-android/test/java/org/briarproject/android/SetupActivityTest.java
@@ -52,7 +52,6 @@ public class SetupActivityTest {
private TestSetupActivity setupActivity;
private TextInputLayout nicknameEntryWrapper;
- private TextInputLayout passwordEntryWrapper;
private TextInputLayout passwordConfirmationWrapper;
private EditText nicknameEntry;
private EditText passwordEntry;
@@ -71,8 +70,6 @@ public class SetupActivityTest {
setupActivity = Robolectric.setupActivity(TestSetupActivity.class);
nicknameEntryWrapper = (TextInputLayout) setupActivity
.findViewById(R.id.nickname_entry_wrapper);
- passwordEntryWrapper = (TextInputLayout) setupActivity
- .findViewById(R.id.password_entry_wrapper);
passwordConfirmationWrapper = (TextInputLayout) setupActivity
.findViewById(R.id.password_confirm_wrapper);
nicknameEntry =
@@ -130,7 +127,7 @@ public class SetupActivityTest {
// Verify that the controller's method was called with the correct
// params and get the callback
verify(mockedController, times(1))
- .storeAuthorInfo(eq(safePass), eq(nick),
+ .storeAuthorInfo(eq(nick), eq(safePass),
authorCaptor.capture());
authorCaptor.getValue().onResult(null);
// execute the callback
diff --git a/briar-api/src/org/briarproject/api/db/DatabaseConfig.java b/briar-api/src/org/briarproject/api/db/DatabaseConfig.java
index 50484b41661e2a61a191997dd017fbe83da41a8a..7d34a184727ab162cd3442741d8bd826ee648a71 100644
--- a/briar-api/src/org/briarproject/api/db/DatabaseConfig.java
+++ b/briar-api/src/org/briarproject/api/db/DatabaseConfig.java
@@ -14,7 +14,7 @@ public interface DatabaseConfig {
SecretKey getEncryptionKey();
- void setAuthorNick(String nickName);
+ void setAuthorNick(String nickname);
String getAuthorNick();
diff --git a/briar-api/src/org/briarproject/api/lifecycle/LifecycleManager.java b/briar-api/src/org/briarproject/api/lifecycle/LifecycleManager.java
index 5ee3a559d4723c28ae3b8003a79d5ae172295487..c0795ccdeb35b0c07a6ad87a6364bfda34ea70a9 100644
--- a/briar-api/src/org/briarproject/api/lifecycle/LifecycleManager.java
+++ b/briar-api/src/org/briarproject/api/lifecycle/LifecycleManager.java
@@ -41,11 +41,11 @@ public interface LifecycleManager {
/**
* Opens the {@link org.briarproject.api.db.DatabaseComponent
- * DatabaseComponent}, creates a local author with the provided nick, and
- * starts any registered {@link org.briarproject.api.clients.Client Clients}
- * and {@link Service Services}.
+ * DatabaseComponent}, creates a local author with the provided nickname,
+ * and starts any registered {@link org.briarproject.api.clients.Client
+ * Clients} and {@link Service Services}.
*/
- StartResult startServices(@Nullable String authorNick);
+ StartResult startServices(@Nullable String nickname);
/**
* Stops any registered {@link Service Services}, shuts down any
diff --git a/briar-core/src/org/briarproject/identity/IdentityManagerImpl.java b/briar-core/src/org/briarproject/identity/IdentityManagerImpl.java
index 48b5b2d129d98ae755ac3954132e7390e980e133..61316e8470f0b96040f72b0cf6348e789c85221a 100644
--- a/briar-core/src/org/briarproject/identity/IdentityManagerImpl.java
+++ b/briar-core/src/org/briarproject/identity/IdentityManagerImpl.java
@@ -25,7 +25,7 @@ class IdentityManagerImpl implements IdentityManager {
private static final Logger LOG =
Logger.getLogger(IdentityManagerImpl.class.getName());
- // Make sure that this variable is immutable
+ // The local author is immutable so we can cache it
private volatile LocalAuthor cachedAuthor;
@Inject
@@ -40,7 +40,7 @@ class IdentityManagerImpl implements IdentityManager {
db.addLocalAuthor(txn, localAuthor);
txn.setComplete();
cachedAuthor = localAuthor;
- LOG.info("Local Author created");
+ LOG.info("Local author registered");
} finally {
db.endTransaction(txn);
}
@@ -52,7 +52,7 @@ class IdentityManagerImpl implements IdentityManager {
Transaction txn = db.startTransaction(true);
try {
cachedAuthor = loadLocalAuthor(txn);
- LOG.info("Author loaded from db");
+ LOG.info("Local author loaded");
txn.setComplete();
} finally {
db.endTransaction(txn);
@@ -66,6 +66,7 @@ class IdentityManagerImpl implements IdentityManager {
public LocalAuthor getLocalAuthor(Transaction txn) throws DbException {
if (cachedAuthor == null) {
cachedAuthor = loadLocalAuthor(txn);
+ LOG.info("Local author loaded");
}
return cachedAuthor;
}
@@ -87,10 +88,7 @@ class IdentityManagerImpl implements IdentityManager {
@Override
public Status getAuthorStatus(Transaction txn, AuthorId authorId)
throws DbException {
-
- // Compare to the IDs of the user's identity
if (getLocalAuthor(txn).getId().equals(authorId)) return OURSELVES;
-
Collection<Contact> contacts = db.getContactsByAuthorId(txn, authorId);
if (contacts.isEmpty()) return UNKNOWN;
for (Contact c : contacts) {
diff --git a/briar-core/src/org/briarproject/introduction/IntroduceeManager.java b/briar-core/src/org/briarproject/introduction/IntroduceeManager.java
index 01ac71211ff08b49ef5f6227a65f42fea11a2feb..450fc023a39a017b21354abeabb101b2d082e181 100644
--- a/briar-core/src/org/briarproject/introduction/IntroduceeManager.java
+++ b/briar-core/src/org/briarproject/introduction/IntroduceeManager.java
@@ -305,8 +305,7 @@ class IntroduceeManager {
boolean alice = comp < 0;
// get our local author
- LocalAuthor author =
- identityManager.getLocalAuthor(txn);
+ LocalAuthor author = identityManager.getLocalAuthor(txn);
SecretKey secretKey;
byte[] privateKeyBytes = localState.getRaw(OUR_PRIVATE_KEY);
diff --git a/briar-core/src/org/briarproject/lifecycle/LifecycleManagerImpl.java b/briar-core/src/org/briarproject/lifecycle/LifecycleManagerImpl.java
index 895f80c284e0426354464164cb2cda38c9d3a17a..f2857cad6ad3d57398831790415e975b4bee1379 100644
--- a/briar-core/src/org/briarproject/lifecycle/LifecycleManagerImpl.java
+++ b/briar-core/src/org/briarproject/lifecycle/LifecycleManagerImpl.java
@@ -93,7 +93,7 @@ class LifecycleManagerImpl implements LifecycleManager {
.createLocalAuthor(nickname, publicKey, privateKey);
long duration = System.currentTimeMillis() - now;
if (LOG.isLoggable(INFO))
- LOG.info("Identity creation took " + duration + " ms");
+ LOG.info("Creating local author took " + duration + " ms");
return localAuthor;
}
@@ -102,12 +102,11 @@ class LifecycleManagerImpl implements LifecycleManager {
identityManager.registerLocalAuthor(author);
long duration = System.currentTimeMillis() - now;
if (LOG.isLoggable(INFO))
- LOG.info("Author registration took " + duration +
- " ms");
+ LOG.info("Registering local author took " + duration + " ms");
}
@Override
- public StartResult startServices(@Nullable String authorNick) {
+ public StartResult startServices(@Nullable String nickname) {
if (!startStopSemaphore.tryAcquire()) {
LOG.info("Already starting or stopping");
return ALREADY_RUNNING;
@@ -124,8 +123,8 @@ class LifecycleManagerImpl implements LifecycleManager {
else LOG.info("Creating database took " + duration + " ms");
}
- if (authorNick != null) {
- registerLocalAuthor(createLocalAuthor(authorNick));
+ if (nickname != null) {
+ registerLocalAuthor(createLocalAuthor(nickname));
}
dbLatch.countDown();
diff --git a/briar-tests/src/org/briarproject/TestDatabaseConfig.java b/briar-tests/src/org/briarproject/TestDatabaseConfig.java
index 3954a2a20be0eb5417a6f3390a0c7301c9963ff2..e4fd78f727b21f932978f805d7f418fc0027059e 100644
--- a/briar-tests/src/org/briarproject/TestDatabaseConfig.java
+++ b/briar-tests/src/org/briarproject/TestDatabaseConfig.java
@@ -39,7 +39,7 @@ public class TestDatabaseConfig implements DatabaseConfig {
}
@Override
- public void setAuthorNick(String nickName) {
+ public void setAuthorNick(String nickname) {
}
diff --git a/briar-tests/src/org/briarproject/TestLifecycleModule.java b/briar-tests/src/org/briarproject/TestLifecycleModule.java
index 02813eb76df71fe0d1477a3fa43a2706b2fac46b..b97bb8251d2d4f4bfbd9609f534dfb4323c6b093 100644
--- a/briar-tests/src/org/briarproject/TestLifecycleModule.java
+++ b/briar-tests/src/org/briarproject/TestLifecycleModule.java
@@ -38,7 +38,7 @@ public class TestLifecycleModule {
}
@Override
- public StartResult startServices(String authorNick) {
+ public StartResult startServices(String nickname) {
return StartResult.SUCCESS;
}