2.7-java-7

Fix duplication of user names in files list.
Release masterpassword-gui-2.7.6
2018-08-27 18:24:46 -04:00 · 2018-08-27 18:24:00 -04:00 · 2018-08-27 13:22:03 -04:00 · 2018-08-27 13:11:32 -04:00 · 2018-08-27 13:07:38 -04:00 · 2018-08-26 20:41:27 -04:00
48 changed files with 916 additions and 348 deletions
--- a/.gitignore
+++ b/.gitignore
@@ -7,6 +7,7 @@ Thumbs.db
 *.iml
 *.ipr
 *.iws
+out

 # Xcode IDE
 xcuserdata/
--- a/6
+++ b/6
@@ -1,12 +1,12 @@
-FROM debian:buster-slim
+FROM debian:stable-slim

 # For i386
-#FROM i386/debian:buster-slim
+#FROM i386/debian:stable-slim
 #ENTRYPOINT ["linux32", "--"]

 # https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=863199
 RUN mkdir -p /usr/share/man/man1

-RUN apt-get update && apt-get install openjdk-10-jdk-headless git-core bash libtool automake autoconf make g++
+RUN apt-get update && apt-get install -y default-jdk-headless git-core bash libtool automake autoconf make g++
 RUN git clone --depth=3 $(: --shallow-submodules) --recurse-submodules https://gitlab.com/MasterPassword/MasterPassword.git /mpw
 RUN cd /mpw/gradle && ./gradlew -i clean build
--- a/gradle/.idea/misc.xml
+++ b/gradle/.idea/misc.xml
@@ -5,21 +5,31 @@
    <option name="myDefaultNotNull" value="javax.annotation.Nonnull" />
    <option name="myNullables">
      <value>
-        <list size="4">
+        <list size="9">
          <item index="0" class="java.lang.String" itemvalue="org.jetbrains.annotations.Nullable" />
          <item index="1" class="java.lang.String" itemvalue="javax.annotation.Nullable" />
          <item index="2" class="java.lang.String" itemvalue="edu.umd.cs.findbugs.annotations.Nullable" />
          <item index="3" class="java.lang.String" itemvalue="android.support.annotation.Nullable" />
+          <item index="4" class="java.lang.String" itemvalue="javax.annotation.CheckForNull" />
+          <item index="5" class="java.lang.String" itemvalue="androidx.annotation.Nullable" />
+          <item index="6" class="java.lang.String" itemvalue="org.checkerframework.checker.nullness.qual.Nullable" />
+          <item index="7" class="java.lang.String" itemvalue="org.checkerframework.checker.nullness.compatqual.NullableDecl" />
+          <item index="8" class="java.lang.String" itemvalue="org.checkerframework.checker.nullness.compatqual.NullableType" />
        </list>
      </value>
    </option>
    <option name="myNotNulls">
      <value>
-        <list size="4">
+        <list size="9">
          <item index="0" class="java.lang.String" itemvalue="org.jetbrains.annotations.NotNull" />
          <item index="1" class="java.lang.String" itemvalue="javax.annotation.Nonnull" />
          <item index="2" class="java.lang.String" itemvalue="edu.umd.cs.findbugs.annotations.NonNull" />
          <item index="3" class="java.lang.String" itemvalue="android.support.annotation.NonNull" />
+          <item index="4" class="java.lang.String" itemvalue="javax.validation.constraints.NotNull" />
+          <item index="5" class="java.lang.String" itemvalue="androidx.annotation.NonNull" />
+          <item index="6" class="java.lang.String" itemvalue="org.checkerframework.checker.nullness.qual.NonNull" />
+          <item index="7" class="java.lang.String" itemvalue="org.checkerframework.checker.nullness.compatqual.NonNullDecl" />
+          <item index="8" class="java.lang.String" itemvalue="org.checkerframework.checker.nullness.compatqual.NonNullType" />
        </list>
      </value>
    </option>
--- a/gradle/README.md
+++ b/gradle/README.md
@@ -2,11 +2,11 @@ To build a release distribution:

 Desktop:

-    STORE_PW=$(mpw masterpassword.keystore) KEY_PW_DESKTOP=$(mpw masterpassword-desktop) gradle clean masterpassword-gui:shadowJar
+    STORE_PW=$(mpw masterpassword.keystore) KEY_PW_DESKTOP=$(mpw masterpassword-desktop) gradle --no-daemon clean masterpassword-gui:shadowJar

 Android:

-    STORE_PW=$(mpw masterpassword.keystore) KEY_PW_ANDROID=$(mpw masterpassword-android) gradle clean masterpassword-android:assembleRelease
+    STORE_PW=$(mpw masterpassword.keystore) KEY_PW_ANDROID=$(mpw masterpassword-android) gradle --no-daemon clean masterpassword-android:assembleRelease


 Note:
--- a/gradle/build.gradle
+++ b/gradle/build.gradle
@@ -2,7 +2,7 @@ allprojects {
    apply plugin: 'findbugs'

    group = 'com.lyndir.masterpassword'
-    version = '2.7.3'
+    version = '2.7.7'

    tasks.withType( JavaCompile ) {
        options.encoding = 'UTF-8'
--- a/lib/bin/build_lib
+++ b/lib/bin/build_lib
@@ -36,7 +36,7 @@ _needs() {
        IFS=: read pkg tools <<< "$spec"
        IFS=, read -a tools <<< "${tools:-$pkg}"
        for tool in "${tools[@]}"; do
-            hash "$tool" && continue 2
+            hash "$tool" 2>/dev/null && continue 2
        done

        echo >&2 "Missing: $pkg.  Please install this package."
@@ -64,7 +64,7 @@ _initialize_needs() {
    if [[ $platform = windows ]]; then
        needs cmd
        export VSINSTALLDIR="${VSINSTALLDIR:-$(cd "$(cygpath -F 0x002a)/Microsoft Visual Studio"/*/*/Common7/.. && pwd)}"
-        [[ -e "$VSINSTALLDIR/Common7/Tools/VsMSBuildCmd.bat" ]] || { echo >&2 "Missing: msbuild.  Please install 'Build Tools for Visual Studio'."; return 1; }
+        [[ -e "$VSINSTALLDIR/Common7/Tools/VsMSBuildCmd.bat" ]] || { echo >&2 "Missing: msbuild.  Please install 'Build Tools for Visual Studio'.  See https://visualstudio.microsoft.com/downloads/?q=build+tools"; return 1; }
    else
        needs libtool:libtoolize,glibtoolize automake autoconf make
    fi
@@ -202,7 +202,7 @@ _target_build() {

    if [[ $platform = windows ]]; then
        # I cannot for the life of me figure out how to pass this command directly into cmd.
-        printf '"%%VSINSTALLDIR%%\Common7\Tools\VsMSBuildCmd.bat" && msbuild /t:Rebuild /p:Configuration=ReleaseDLL;Platform=%s;OutDir=%s' "$arch" "$(cygpath -w "${prefix##$PWD/}/$arch/")" > .build.bat
+        printf '"%%VSINSTALLDIR%%\Common7\Tools\VsMSBuildCmd.bat" && msbuild /t:Rebuild /p:Configuration=Release;Platform=%s;OutDir=%s' "$arch" "$(cygpath -w "${prefix##$PWD/}/$arch/")" > .build.bat
        cmd //c .build.bat
        rm -f .build.bat
    else
@@ -278,7 +278,10 @@ _finalize_merge() {
 # By default, this will run `make clean`.
 finalize_clean() { _finalize_clean "$@"; }
 _finalize_clean() {
-    [[ ! -e Makefile ]] || make -s clean
+    if [[ $platform = windows ]]; then :
+    else
+        [[ ! -e Makefile ]] || make -s clean
+    fi
 }

 # build <name> [<platform>]
--- a/platform-independent/c/core/lib/linux/x86/libmpw.so
+++ b/platform-independent/c/core/lib/linux/x86/libmpw.so
--- a/platform-independent/c/core/lib/linux/x86_64/libmpw.so
+++ b/platform-independent/c/core/lib/linux/x86_64/libmpw.so
--- a/platform-independent/c/core/lib/windows/x86/mpw.dll
+++ b/platform-independent/c/core/lib/windows/x86/mpw.dll
--- a/platform-independent/c/core/lib/windows/x86_64/mpw.dll
+++ b/platform-independent/c/core/lib/windows/x86_64/mpw.dll
--- a/platform-independent/java/algorithm/src/main/java/com/lyndir/masterpassword/MPMasterKey.java
+++ b/platform-independent/java/algorithm/src/main/java/com/lyndir/masterpassword/MPMasterKey.java
@@ -66,6 +66,7 @@ public class MPMasterKey {
     *
     * @throws MPKeyUnavailableException {@link #invalidate()} has been called on this object.
     */
+    @Nonnull
    public byte[] getKeyID(final MPAlgorithm algorithm)
            throws MPKeyUnavailableException, MPAlgorithmException {

@@ -87,6 +88,7 @@ public class MPMasterKey {
        return !invalidated;
    }

+    @Nonnull
    private byte[] masterKey(final MPAlgorithm algorithm)
            throws MPKeyUnavailableException, MPAlgorithmException {
        Preconditions.checkArgument( masterPassword.length > 0 );
@@ -109,6 +111,7 @@ public class MPMasterKey {
        return masterKey;
    }

+    @Nonnull
    private byte[] siteKey(final String siteName, final MPAlgorithm algorithm, final UnsignedInteger siteCounter,
                           final MPKeyPurpose keyPurpose, @Nullable final String keyContext)
            throws MPKeyUnavailableException, MPAlgorithmException {
@@ -141,13 +144,19 @@ public class MPMasterKey {
     *                    In the case of {@link MPResultTypeClass#Stateful} types, the result of
     *                    {@link #siteState(String, MPAlgorithm, UnsignedInteger, MPKeyPurpose, String, MPResultType, String)}.
     *
+     * @return {@code null} if the result type is missing a required parameter.
+     *
     * @throws MPKeyUnavailableException {@link #invalidate()} has been called on this object.
     */
+    @Nullable
    public String siteResult(final String siteName, final MPAlgorithm algorithm, final UnsignedInteger siteCounter,
                             final MPKeyPurpose keyPurpose, @Nullable final String keyContext,
                             final MPResultType resultType, @Nullable final String resultParam)
            throws MPKeyUnavailableException, MPAlgorithmException {

+        if ((resultType.getTypeClass() == MPResultTypeClass.Stateful) && (resultParam == null))
+            return null;
+
        byte[] masterKey = masterKey( algorithm );
        byte[] siteKey   = siteKey( siteName, algorithm, siteCounter, keyPurpose, keyContext );

@@ -176,6 +185,7 @@ public class MPMasterKey {
     *
     * @throws MPKeyUnavailableException {@link #invalidate()} has been called on this object.
     */
+    @Nonnull
    public String siteState(final String siteName, final MPAlgorithm algorithm, final UnsignedInteger siteCounter,
                            final MPKeyPurpose keyPurpose, @Nullable final String keyContext,
                            final MPResultType resultType, final String resultParam)
--- a/platform-independent/java/algorithm/src/main/java/com/lyndir/masterpassword/MPResultType.java
+++ b/platform-independent/java/algorithm/src/main/java/com/lyndir/masterpassword/MPResultType.java
@@ -41,7 +41,7 @@ public enum MPResultType {
    /**
     * 16: pg^VMAUBk5x3p%HP%i4=
     */
-    GeneratedMaximum( "maximum", "Maximum Security", "pg^VMAUBk5x3p%HP%i4=", "20 characters, contains symbols.", //
+    GeneratedMaximum( "maximum", "Maximum Security", "pg^VMAUBk5x3p%HP%i4=", "20 characters, contains symbols", //
                      ImmutableList.of( new MPTemplate( "anoxxxxxxxxxxxxxxxxx" ),
                                        new MPTemplate( "axxxxxxxxxxxxxxxxxno" ) ), //
                      MPResultTypeClass.Template, 0x0 ),
@@ -49,7 +49,7 @@ public enum MPResultType {
    /**
     * 17: BiroYena8:Kixa
     */
-    GeneratedLong( "long", "Long Password", "BiroYena8:Kixa", "Copy-friendly, 14 characters, contains symbols.", //
+    GeneratedLong( "long", "Long Password", "BiroYena8:Kixa", "Copy-friendly, 14 characters, contains symbols", //
                   ImmutableList.of( new MPTemplate( "CvcvnoCvcvCvcv" ), new MPTemplate( "CvcvCvcvnoCvcv" ),
                                     new MPTemplate( "CvcvCvcvCvcvno" ), new MPTemplate( "CvccnoCvcvCvcv" ),
                                     new MPTemplate( "CvccCvcvnoCvcv" ), new MPTemplate( "CvccCvcvCvcvno" ),
@@ -66,7 +66,7 @@ public enum MPResultType {
    /**
     * 18: BirSuj0-
     */
-    GeneratedMedium( "medium", "Medium Password", "BirSuj0-", "Copy-friendly, 8 characters, contains symbols.", //
+    GeneratedMedium( "medium", "Medium Password", "BirSuj0-", "Copy-friendly, 8 characters, contains symbols", //
                     ImmutableList.of( new MPTemplate( "CvcnoCvc" ),
                                       new MPTemplate( "CvcCvcno" ) ), //
                     MPResultTypeClass.Template, 0x2 ),
@@ -74,14 +74,14 @@ public enum MPResultType {
    /**
     * 19: Bir8
     */
-    GeneratedShort( "short", "Short Password", "Bir8", "Copy-friendly, 4 characters, no symbols.", //
+    GeneratedShort( "short", "Short Password", "Bir8", "Copy-friendly, 4 characters, no symbols", //
                    ImmutableList.of( new MPTemplate( "Cvcn" ) ), //
                    MPResultTypeClass.Template, 0x3 ),

    /**
     * 20: pO98MoD0
     */
-    GeneratedBasic( "basic", "Basic Password", "pO98MoD0", "8 characters, no symbols.", //
+    GeneratedBasic( "basic", "Basic Password", "pO98MoD0", "8 characters, no symbols", //
                    ImmutableList.of( new MPTemplate( "aaanaaan" ),
                                      new MPTemplate( "aannaaan" ),
                                      new MPTemplate( "aaannaaa" ) ), //
@@ -90,44 +90,44 @@ public enum MPResultType {
    /**
     * 21: 2798
     */
-    GeneratedPIN( "pin", "PIN Code", "2798", "4 numbers.", //
+    GeneratedPIN( "pin", "PIN Code", "2798", "4 numbers", //
                  ImmutableList.of( new MPTemplate( "nnnn" ) ), //
                  MPResultTypeClass.Template, 0x5 ),

    /**
     * 30: birsujano
     */
-    GeneratedName( "name", "Name", "birsujano", "9 letter name.", //
+    GeneratedName( "name", "Name", "birsujano", "9 letter name", //
                   ImmutableList.of( new MPTemplate( "cvccvcvcv" ) ), //
                   MPResultTypeClass.Template, 0xE ),

    /**
     * 31: bir yennoquce fefi
     */
-    GeneratedPhrase( "phrase", "Phrase", "bir yennoquce fefi", "20 character sentence.", //
+    GeneratedPhrase( "phrase", "Phrase", "bir yennoquce fefi", "20 character sentence", //
                     ImmutableList.of( new MPTemplate( "cvcc cvc cvccvcv cvc" ),
                                       new MPTemplate( "cvc cvccvcvcv cvcv" ),
                                       new MPTemplate( "cv cvccv cvc cvcvccv" ) ), //
                     MPResultTypeClass.Template, 0xF ),

    /**
-     * 1056: Custom saved password.
+     * 1056: Custom saved value.
     */
-    StoredPersonal( "personal", "Saved Password", null, "AES-encrypted, exportable.", //
+    StoredPersonal( "personal", "Saved", null, "AES-encrypted, exportable", //
                    ImmutableList.<MPTemplate>of(), //
                    MPResultTypeClass.Stateful, 0x0, MPSiteFeature.ExportContent ),

    /**
-     * 2081: Custom saved password that should not be exported from the device.
+     * 2081: Custom saved value that should not be exported from the device.
     */
-    StoredDevicePrivate( "device", "Private Password", null, "AES-encrypted, not exported.", //
+    StoredDevicePrivate( "device", "Private", null, "AES-encrypted, not exported", //
                         ImmutableList.<MPTemplate>of(), //
                         MPResultTypeClass.Stateful, 0x1, MPSiteFeature.DevicePrivate ),

    /**
     * 4160: Derive a unique binary key.
     */
-    DeriveKey( "key", "Binary Key", null, "Encryption key.", //
+    DeriveKey( "key", "Binary Key", null, "Encryption key", //
               ImmutableList.<MPTemplate>of(), //
               MPResultTypeClass.Derive, 0x0, MPSiteFeature.Alternative );

--- a/platform-independent/java/algorithm/src/main/java/com/lyndir/masterpassword/impl/Native.java
+++ b/platform-independent/java/algorithm/src/main/java/com/lyndir/masterpassword/impl/Native.java
@@ -18,14 +18,14 @@

 package com.lyndir.masterpassword.impl;

-import static com.lyndir.lhunath.opal.system.util.ObjectUtils.*;
-
 import com.google.common.base.Joiner;
 import com.google.common.collect.ImmutableList;
+import com.google.common.collect.ImmutableSet;
 import com.google.common.io.ByteStreams;
 import com.lyndir.lhunath.opal.system.logging.Logger;
 import java.io.*;
-import java.util.Locale;
+import java.util.*;
+import java.util.function.Predicate;
 import javax.annotation.Nonnull;
 import javax.annotation.Nullable;

@@ -43,71 +43,161 @@ public final class Native {
    private static final String NATIVES_PATH        = "lib";

    @SuppressWarnings({ "HardcodedFileSeparator", "LoadLibraryWithNonConstantString" })
-    public static void load(final Class<?> context, final String name) {
+    public static boolean load(final Class<?> context, final String name) {

        // Try to load the library using the native system.
        try {
            System.loadLibrary( name );
-            return;
+            return true;
        }
        catch (@SuppressWarnings("ErrorNotRethrown") final UnsatisfiedLinkError ignored) {
        }

        // Try to find and open a stream to the packaged library resource.
-        try {
-            String      library          = System.mapLibraryName( name );
-            int         libraryDot       = library.lastIndexOf( EXTENSION_SEPARATOR );
-            String      libraryName      = (libraryDot > 0)? library.substring( 0, libraryDot ): library;
-            String      libraryExtension = (libraryDot > 0)? library.substring( libraryDot ): ".lib";
-            String      libraryResource  = getLibraryResource( library );
-            InputStream libraryStream    = context.getResourceAsStream( libraryResource );
-            if (libraryStream == null)
-                throw new IllegalStateException(
-                        "Library: " + name + " (" + libraryResource + "), not found in class loader for: " + context );
+        String             library          = System.mapLibraryName( name );
+        int                libraryDot       = library.lastIndexOf( EXTENSION_SEPARATOR );
+        String             libraryName      = (libraryDot > 0)? library.substring( 0, libraryDot ): library;
+        String             libraryExtension = (libraryDot > 0)? library.substring( libraryDot ): ".lib";

-            // Write the library resource to a temporary file.
-            File             libraryFile       = File.createTempFile( libraryName, libraryExtension );
-            FileOutputStream libraryFileStream = new FileOutputStream( libraryFile );
+        @Nullable
+        File libraryFile = null;
+        Set<String> libraryResources = getLibraryResources( library );
+        for (final String libraryResource : libraryResources) {
            try {
-                libraryFile.deleteOnExit();
-                ByteStreams.copy( libraryStream, libraryFileStream );
-            }
-            finally {
-                libraryFileStream.close();
-                libraryStream.close();
-            }
+                InputStream libraryStream = context.getResourceAsStream( libraryResource );
+                if (libraryStream == null) {
+                    logger.dbg( "No resource for library: %s", libraryResource );
+                    continue;
+                }

-            // Load the library from the temporary file.
-            System.load( libraryFile.getAbsolutePath() );
-        }
-        catch (final IOException e) {
-            throw new IllegalStateException( "Couldn't extract library: " + name, e );
+                // Write the library resource to a temporary file.
+                libraryFile = File.createTempFile( libraryName, libraryExtension );
+                FileOutputStream libraryFileStream = new FileOutputStream( libraryFile );
+                try {
+                    libraryFile.deleteOnExit();
+                    ByteStreams.copy( libraryStream, libraryFileStream );
+                }
+                finally {
+                    libraryFileStream.close();
+                    libraryStream.close();
+                }
+
+                // Load the library from the temporary file.
+                System.load( libraryFile.getAbsolutePath() );
+                return true;
+            }
+            catch (@SuppressWarnings("ErrorNotRethrown") final IOException | UnsatisfiedLinkError e) {
+                logger.dbg( e, "Couldn't load library: %s", libraryResource );
+
+                if (libraryFile != null)
+                    if (libraryFile.exists() && !libraryFile.delete())
+                        logger.wrn( "Couldn't clean up library file: %s", libraryFile );
+                libraryFile = null;
+            }
        }
+
+        return false;
    }

    @Nonnull
-    private static String getLibraryResource(final String library) {
-        String system       = ifNotNullElse( System.getProperty( "os.name" ), "linux" ).toLowerCase( Locale.ROOT );
-        String architecture = ifNotNullElse( System.getProperty( "os.arch" ), "x86" ).toLowerCase( Locale.ROOT );
-
+    private static Set<String> getLibraryResources(final String library) {
        // Standardize system naming in accordance with masterpassword-core.
-        if (system.contains( "windows" ))
-            system = "windows";
-        else if (system.contains( "mac os x" ) || system.contains( "darwin" ) || system.contains( "osx" ))
-            system = "macos";
-        else
-            system = "linux";
+        Sys system = Sys.findCurrent();

        // Standardize architecture naming in accordance with masterpassword-core.
-        if (ImmutableList.of( "arm", "arm-v7", "armv7", "arm32" ).contains( architecture ))
-            architecture = "arm";
-        else if (architecture.startsWith( "arm" ))
-            architecture = "arm64";
-        else if (ImmutableList.of( "x86_64", "amd64", "x64", "x86-64" ).contains( architecture ))
-            architecture = "x86_64";
-        else
-            architecture = "x86";
+        Collection<Arch> architectures = new LinkedHashSet<>();
+        architectures.add( Arch.findCurrent() );
+        architectures.addAll( Arrays.asList( Arch.values() ) );

-        return Joiner.on( RESOURCE_SEPARATOR ).join( "", NATIVES_PATH, system, architecture, library );
+        ImmutableSet.Builder<String> resources = ImmutableSet.builder();
+        for (final Arch arch : architectures)
+            resources.add( Joiner.on( RESOURCE_SEPARATOR ).join( "", NATIVES_PATH, system, arch, library ) );
+
+        return resources.build();
+    }
+
+    private enum Sys implements Predicate<String> {
+        windows {
+            @Override
+            public boolean test(final String system) {
+                return system.contains( "windows" );
+            }
+        },
+        macos {
+            @Override
+            public boolean test(final String system) {
+                return system.contains( "mac os x" ) || system.contains( "darwin" ) || system.contains( "osx" );
+            }
+        },
+        linux {
+            @Override
+            public boolean test(final String system) {
+                return system.contains( "linux" );
+            }
+        };
+
+        @Nonnull
+        public static Sys findCurrent() {
+            return find( System.getProperty( "os.name" ) );
+        }
+
+        @Nonnull
+        public static Sys find(@Nullable String name) {
+            if (name != null) {
+                name = name.toLowerCase( Locale.ROOT );
+
+                for (final Sys sys : values())
+                    if (sys.test( name ))
+                        return sys;
+            }
+
+            return linux;
+        }
+    }
+
+
+    private enum Arch implements Predicate<String> {
+        arm {
+            @Override
+            public boolean test(final String architecture) {
+                return ImmutableList.of( "arm", "arm-v7", "armv7", "arm32" ).contains( architecture );
+            }
+        },
+        arm64 {
+            @Override
+            public boolean test(final String architecture) {
+                return architecture.startsWith( "arm" ) && !arm.test( architecture );
+            }
+        },
+        x86_64 {
+            @Override
+            public boolean test(final String architecture) {
+                return ImmutableList.of( "x86_64", "amd64", "x64", "x86-64" ).contains( architecture );
+            }
+        },
+        x86 {
+            @Override
+            public boolean test(final String architecture) {
+                return ImmutableList.of( "x86", "i386", "i686" ).contains( architecture );
+            }
+        };
+
+        @Nonnull
+        public static Arch findCurrent() {
+            return find( System.getProperty( "os.arch" ) );
+        }
+
+        @Nonnull
+        public static Arch find(@Nullable String name) {
+            if (name != null) {
+                name = name.toLowerCase( Locale.ROOT );
+
+                for (final Arch arch : values())
+                    if (arch.test( name ))
+                        return arch;
+            }
+
+            return x86;
+        }
    }
 }
--- a/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/GUI.java
+++ b/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/GUI.java
@@ -1,36 +0,0 @@
-package com.lyndir.masterpassword.gui;
-
-import com.lyndir.lhunath.opal.system.logging.Logger;
-import com.lyndir.masterpassword.gui.util.Platform;
-import com.lyndir.masterpassword.gui.util.Res;
-import com.lyndir.masterpassword.gui.view.MasterPasswordFrame;
-import com.tulskiy.keymaster.common.Provider;
-import java.awt.*;
-
-
-/**
- * @author lhunath, 2018-07-28
- */
-public class GUI {
-
-    private static final Logger logger = Logger.get( GUI.class );
-
-    private final MasterPasswordFrame frame = new MasterPasswordFrame();
-
-    public GUI() {
-        Platform.get().installAppForegroundHandler( this::open );
-        Platform.get().installAppReopenHandler( this::open );
-
-        Provider.getCurrentProvider( true ).register( MPGuiConstants.ui_hotkey, hotKey -> open() );
-    }
-
-    public void open() {
-        Res.ui( () -> {
-            frame.setAlwaysOnTop( true );
-            frame.setVisible( true );
-            frame.setExtendedState( Frame.NORMAL );
-            frame.setAlwaysOnTop( false );
-            Platform.get().requestForeground();
-        } );
-    }
-}
--- a/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/MPConfig.java
+++ b/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/MPConfig.java
@@ -26,11 +26,11 @@ import com.lyndir.masterpassword.model.MPModelConstants;
 * @author lhunath, 2014-08-31
 */
@SuppressWarnings("CallToSystemGetenv")
-public class Config {
+public class MPConfig {

-    private static final Config instance = new Config();
+    private static final MPConfig instance = new MPConfig();

-    public static Config get() {
+    public static MPConfig get() {
        return instance;
    }

--- a/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/MasterPassword.java
+++ b/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/MasterPassword.java
@@ -24,8 +24,11 @@ import com.google.common.base.Charsets;
 import com.google.common.io.ByteSource;
 import com.lyndir.lhunath.opal.system.logging.Logger;
 import com.lyndir.lhunath.opal.system.util.ObjectUtils;
-import com.lyndir.masterpassword.gui.util.Components;
+import com.lyndir.masterpassword.gui.util.*;
+import com.lyndir.masterpassword.gui.view.MasterPasswordFrame;
 import com.lyndir.masterpassword.model.MPUser;
+import com.tulskiy.keymaster.common.Provider;
+import java.awt.*;
 import java.io.IOException;
 import java.io.InputStream;
 import java.net.*;
@@ -50,18 +53,21 @@ public final class MasterPassword {
    private final Collection<Listener> listeners = new CopyOnWriteArraySet<>();

    @Nullable
-    private MPUser<?> activeUser;
+    private MasterPasswordFrame frame;
+    @Nullable
+    private MPUser<?>           activeUser;

    public static MasterPassword get() {
        return instance;
    }

-    public boolean addListener(final Listener listener) {
-        return listeners.add( listener );
+    public void addListener(final Listener listener) {
+        if (listeners.add( listener ))
+            listener.onUserSelected( activeUser );
    }

-    public boolean removeListener(final Listener listener) {
-        return listeners.remove( listener );
+    public void removeListener(final Listener listener) {
+        listeners.remove( listener );
    }

    public void activateUser(final MPUser<?> user) {
@@ -78,6 +84,19 @@ public final class MasterPassword {
        return MasterPassword.class.getPackage().getImplementationVersion();
    }

+    public void open() {
+        Res.ui( () -> {
+            if (frame == null)
+                frame = new MasterPasswordFrame();
+
+            frame.setAlwaysOnTop( true );
+            frame.setVisible( true );
+            frame.setExtendedState( Frame.NORMAL );
+            Platform.get().requestForeground();
+            frame.setAlwaysOnTop( false );
+        } );
+    }
+
    public void checkUpdate() {
        try {
            String implementationVersion = version();
@@ -115,18 +134,22 @@ public final class MasterPassword {
        // Try and set the system look & feel, if available.
        try {
            UIManager.setLookAndFeel( UIManager.getSystemLookAndFeelClassName() );
+            Platform.get().installAppForegroundHandler( get()::open );
+            Platform.get().installAppReopenHandler( get()::open );
+            Provider.getCurrentProvider( true ).register( MPGuiConstants.ui_hotkey, hotKey -> get().open() );
        }
        catch (final UnsupportedLookAndFeelException | ClassNotFoundException | InstantiationException | IllegalAccessException ignored) {
        }

        // Create a platform-specific GUI and open it.
-        new GUI().open();
+        get().open();

        // Check online to see if this version has been superseded.
-        if (Config.get().checkForUpdates())
+        if (MPConfig.get().checkForUpdates())
            get().checkUpdate();
    }

+    @SuppressWarnings("InterfaceMayBeAnnotatedFunctional")
    public interface Listener {

        void onUserSelected(@Nullable MPUser<?> user);
--- a/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/model/MPIncognitoQuestion.java
+++ b/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/model/MPIncognitoQuestion.java
@@ -31,17 +31,7 @@ import javax.annotation.Nullable;
 */
 public class MPIncognitoQuestion extends MPBasicQuestion {

-    private final MPIncognitoSite site;
-
    public MPIncognitoQuestion(final MPIncognitoSite site, final String keyword, @Nullable final MPResultType type) {
-        super( keyword, ifNotNullElse( type, site.getAlgorithm().mpw_default_answer_type() ) );
-
-        this.site = site;
-    }
-
-    @Nonnull
-    @Override
-    public MPIncognitoSite getSite() {
-        return site;
+        super( site, keyword, ifNotNullElse( type, site.getAlgorithm().mpw_default_answer_type() ) );
    }
 }
--- a/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/model/MPIncognitoSite.java
+++ b/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/model/MPIncognitoSite.java
@@ -40,4 +40,10 @@ public class MPIncognitoSite extends MPBasicSite<MPIncognitoUser, MPIncognitoQue
                           @Nullable final MPResultType resultType, @Nullable final MPResultType loginType) {
        super( user, siteName, (algorithm == null)? user.getAlgorithm(): algorithm, counter, resultType, loginType );
    }
+
+    @Nonnull
+    @Override
+    public MPIncognitoQuestion addQuestion(final String keyword) {
+        return addQuestion( new MPIncognitoQuestion( this, keyword, null ) );
+    }
 }
--- a/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/model/MPIncognitoUser.java
+++ b/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/model/MPIncognitoUser.java
@@ -20,6 +20,7 @@ package com.lyndir.masterpassword.gui.model;

 import com.lyndir.masterpassword.MPAlgorithm;
 import com.lyndir.masterpassword.model.impl.MPBasicUser;
+import javax.annotation.Nonnull;
 import javax.annotation.Nullable;


@@ -38,6 +39,7 @@ public class MPIncognitoUser extends MPBasicUser<MPIncognitoSite> {
        return null;
    }

+    @Nonnull
    @Override
    public MPIncognitoSite addSite(final String siteName) {
        return addSite( new MPIncognitoSite( this, siteName ) );
--- a/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/model/MPNewQuestion.java
+++ b/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/model/MPNewQuestion.java
@@ -0,0 +1,15 @@
+package com.lyndir.masterpassword.gui.model;
+
+import com.lyndir.masterpassword.model.MPSite;
+import com.lyndir.masterpassword.model.impl.MPBasicQuestion;
+
+
+/**
+ * @author lhunath, 2018-07-27
+ */
+public class MPNewQuestion extends MPBasicQuestion {
+
+    public MPNewQuestion(final MPSite<?> site, final String keyword) {
+        super( site, keyword, site.getAlgorithm().mpw_default_answer_type() );
+    }
+}
--- a/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/model/MPNewSite.java
+++ b/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/model/MPNewSite.java
@@ -2,6 +2,7 @@ package com.lyndir.masterpassword.gui.model;

 import com.lyndir.masterpassword.model.*;
 import com.lyndir.masterpassword.model.impl.*;
+import javax.annotation.Nonnull;


 /**
@@ -12,4 +13,10 @@ public class MPNewSite extends MPBasicSite<MPUser<?>, MPQuestion> {
    public MPNewSite(final MPUser<?> user, final String siteName) {
        super( user, siteName );
    }
+
+    @Nonnull
+    @Override
+    public MPQuestion addQuestion(final String keyword) {
+        throw new UnsupportedOperationException( "Cannot add a question to a site that hasn't been created yet." );
+    }
 }
--- a/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/util/CollectionListModel.java
+++ b/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/util/CollectionListModel.java
@@ -1,6 +1,10 @@
 package com.lyndir.masterpassword.gui.util;

+import com.google.common.base.Predicates;
 import com.google.common.collect.ImmutableList;
+import com.google.common.collect.Iterables;
+import com.lyndir.lhunath.opal.system.logging.Logger;
+import com.lyndir.lhunath.opal.system.util.ObjectUtils;
 import java.util.*;
 import java.util.function.Consumer;
 import javax.annotation.Nullable;
@@ -16,27 +20,25 @@ import javax.swing.event.ListSelectionListener;
 public class CollectionListModel<E> extends AbstractListModel<E>
        implements ComboBoxModel<E>, ListSelectionListener, Selectable<E, CollectionListModel<E>> {

+    private static final Logger logger = Logger.get( CollectionListModel.class );
+
    private final List<E>     model = new LinkedList<>();
    @Nullable
-    private       E           selectedItem;
    private       JList<E>    list;
    @Nullable
+    private       E           selectedItem;
+    @Nullable
    private       Consumer<E> selectionConsumer;

    @SafeVarargs
-    public static <E> CollectionListModel<E> copy(final E... elements) {
-        return copy( Arrays.asList( elements ) );
+    public CollectionListModel(final E... elements) {
+        this( Arrays.asList( elements ) );
    }

-    public static <E> CollectionListModel<E> copy(final Collection<? extends E> elements) {
-        CollectionListModel<E> model = new CollectionListModel<>();
-        synchronized (model) {
-            model.model.addAll( elements );
-            model.selectedItem = model.getElementAt( 0 );
-            model.fireIntervalAdded( model, 0, model.model.size() );
-
-            return model;
-        }
+    public CollectionListModel(final Collection<? extends E> elements) {
+        model.addAll( elements );
+        selectedItem = getElementAt( 0 );
+        fireIntervalAdded( this, 0, model.size() );
    }

    @Override
@@ -44,8 +46,8 @@ public class CollectionListModel<E> extends AbstractListModel<E>
        return model.size();
    }

-    @Override
    @Nullable
+    @Override
    public synchronized E getElementAt(final int index) {
        return (index < model.size())? model.get( index ): null;
    }
@@ -56,16 +58,11 @@ public class CollectionListModel<E> extends AbstractListModel<E>
     * This operation will mutate the internal model to reflect the given model.
     * The given model will remain untouched and independent from this object.
     */
-    @SuppressWarnings({ "unchecked", "SuspiciousToArrayCall" })
-    public synchronized void set(final Collection<? extends E> elements) {
-        set( (E[]) elements.toArray( new Object[0] ) );
-    }
-
-    @SuppressWarnings("AssignmentToForLoopParameter")
-    public synchronized void set(final E... elements) {
+    @SuppressWarnings({ "Guava", "AssignmentToForLoopParameter" })
+    public synchronized void set(final Iterable<? extends E> elements) {
        ListIterator<E> oldIt = model.listIterator();
        for (int from = 0; oldIt.hasNext(); ++from) {
-            int to = Arrays.binarySearch( elements, oldIt.next() );
+            int to   = Iterables.indexOf( elements, Predicates.equalTo( oldIt.next() ) );

            if (to != from) {
                oldIt.remove();
@@ -74,19 +71,25 @@ public class CollectionListModel<E> extends AbstractListModel<E>
            }
        }

-        for (int to = 0; to < elements.length; ++to) {
-            E newSite = elements[to];
-
+        int to = 0;
+        for (final E newSite : elements) {
            if ((to >= model.size()) || !Objects.equals( model.get( to ), newSite )) {
                model.add( to, newSite );
                fireIntervalAdded( this, to, to );
            }
+
+            ++to;
        }

        if ((selectedItem == null) || !model.contains( selectedItem ))
            setSelectedItem( getElementAt( 0 ) );
    }

+    @SafeVarargs
+    public final synchronized void set(final E... elements) {
+        set( ImmutableList.copyOf( elements ) );
+    }
+
    @Override
    @SuppressWarnings({ "unchecked", "SuspiciousMethodCalls" })
    public synchronized void setSelectedItem(@Nullable final Object newSelectedItem) {
@@ -109,6 +112,11 @@ public class CollectionListModel<E> extends AbstractListModel<E>
        return selectedItem;
    }

+    public CollectionListModel<E> select(final E selectedItem) {
+        setSelectedItem( selectedItem );
+        return this;
+    }
+
    public synchronized void registerList(final JList<E> list) {
        // TODO: This class should probably implement ListSelectionModel instead.
        if (this.list != null)
--- a/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/util/Components.java
+++ b/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/util/Components.java
@@ -18,6 +18,7 @@

 package com.lyndir.masterpassword.gui.util;

+import com.google.common.base.Strings;
 import com.lyndir.lhunath.opal.system.logging.Logger;
 import java.awt.*;
 import java.awt.event.ActionEvent;
@@ -32,8 +33,8 @@ import javax.annotation.Nullable;
 import javax.swing.*;
 import javax.swing.border.Border;
 import javax.swing.border.CompoundBorder;
-import javax.swing.event.*;
-import javax.swing.text.DefaultFormatterFactory;
+import javax.swing.event.HyperlinkEvent;
+import javax.swing.text.*;
 import org.jetbrains.annotations.NonNls;


@@ -100,6 +101,7 @@ public abstract class Components {

    public static int showDialog(@Nullable final Component owner, @Nullable final String title, final JOptionPane pane) {
        JDialog dialog = pane.createDialog( owner, title );
+        dialog.setMinimumSize( new Dimension( 520, 0 ) );
        dialog.setModalityType( Dialog.ModalityType.DOCUMENT_MODAL );
        showDialog( dialog );

@@ -111,8 +113,13 @@ public abstract class Components {
        if (options == null)
            return (selectedValue instanceof Integer)? (Integer) selectedValue: JOptionPane.CLOSED_OPTION;

-        int option = Arrays.binarySearch( options, selectedValue );
-        return (option < 0)? JOptionPane.CLOSED_OPTION: option;
+        try {
+            int option = Arrays.binarySearch( options, selectedValue );
+            return (option < 0)? JOptionPane.CLOSED_OPTION: option;
+        }
+        catch (final ClassCastException ignored) {
+            return JOptionPane.CLOSED_OPTION;
+        }
    }

    @Nullable
@@ -143,7 +150,6 @@ public abstract class Components {
                                      title, Dialog.ModalityType.DOCUMENT_MODAL );
        dialog.setMinimumSize( new Dimension( 320, 0 ) );
        dialog.setLocationRelativeTo( owner );
-        dialog.setLocationByPlatform( true );
        dialog.setContentPane( content );

        return showDialog( dialog );
@@ -155,13 +161,18 @@ public abstract class Components {
        dialog.getRootPane().putClientProperty( "Window.style", "small" );
        dialog.pack();

+        dialog.setLocationByPlatform( true );
        dialog.setVisible( true );

        return dialog;
    }

    public static JTextField textField() {
-        return new JTextField() {
+        return textField( null );
+    }
+
+    public static JTextField textField(@Nullable final Document document) {
+        return new JTextField( document, null, 0 ) {
            {
                setBorder( BorderFactory.createCompoundBorder( BorderFactory.createLineBorder( Res.colors().controlBorder(), 1, true ),
                                                               BorderFactory.createEmptyBorder( 4, 4, 4, 4 ) ) );
@@ -171,41 +182,30 @@ public abstract class Components {

            @Override
            public Dimension getMaximumSize() {
-                return new Dimension( Integer.MAX_VALUE, getPreferredSize().height );
+                return new Dimension( Integer.MAX_VALUE, Integer.MAX_VALUE );
            }
        };
    }

-    public static JTextField textField(@Nullable final String text, @Nullable final Consumer<String> selection) {
-        return new JTextField( text ) {
+    public static JTextField textField(@Nullable final String text, @Nullable final Consumer<String> change) {
+        return textField( new DocumentModel( new PlainDocument() ).selection( text, change ).getDocument() );
+    }
+
+    public static JTextArea textArea() {
+        return new JTextArea() {
            {
                setBorder( BorderFactory.createCompoundBorder( BorderFactory.createLineBorder( Res.colors().controlBorder(), 1, true ),
                                                               BorderFactory.createEmptyBorder( 4, 4, 4, 4 ) ) );
                setFont( Res.fonts().valueFont( TEXT_SIZE_CONTROL ) );
                setAlignmentX( LEFT_ALIGNMENT );

-                if (selection != null)
-                    getDocument().addDocumentListener( new DocumentListener() {
-                        @Override
-                        public void insertUpdate(final DocumentEvent e) {
-                            selection.accept( getText() );
-                        }
-
-                        @Override
-                        public void removeUpdate(final DocumentEvent e) {
-                            selection.accept( getText() );
-                        }
-
-                        @Override
-                        public void changedUpdate(final DocumentEvent e) {
-                            selection.accept( getText() );
-                        }
-                    } );
+                setLineWrap( true );
+                setRows( 3 );
            }

            @Override
            public Dimension getMaximumSize() {
-                return new Dimension( Integer.MAX_VALUE, getPreferredSize().height );
+                return new Dimension( Integer.MAX_VALUE, Integer.MAX_VALUE );
            }
        };
    }
@@ -228,20 +228,24 @@ public abstract class Components {
    public static <E> JList<E> list(final ListModel<E> model, final Function<E, String> valueTransformer) {
        return new JList<E>( model ) {
            {
+                setAlignmentX( LEFT_ALIGNMENT );
                setFont( Res.fonts().valueFont( TEXT_SIZE_CONTROL ) );
                setCellRenderer( new DefaultListCellRenderer() {
                    @Override
                    @SuppressWarnings({ "unchecked", "SerializableStoresNonSerializable" })
                    public Component getListCellRendererComponent(final JList<?> list, final Object value, final int index,
                                                                  final boolean isSelected, final boolean cellHasFocus) {
+                        String label = valueTransformer.apply( (E) value );
                        super.getListCellRendererComponent(
-                                list, valueTransformer.apply( (E) value ), index, isSelected, cellHasFocus );
+                                list, Strings.isNullOrEmpty( label )? " ": label, index, isSelected, cellHasFocus );
                        setBorder( BorderFactory.createEmptyBorder( 2, 4, 2, 4 ) );

                        return this;
                    }
                } );
-                setAlignmentX( LEFT_ALIGNMENT );
+
+                if (model instanceof CollectionListModel)
+                    ((CollectionListModel<E>) model).registerList( this );
            }

            @Override
@@ -297,7 +301,6 @@ public abstract class Components {
    public static JButton button(final Action action) {
        return new JButton( action ) {
            {
-                setFont( Res.fonts().controlFont( TEXT_SIZE_CONTROL ) );
                setAlignmentX( LEFT_ALIGNMENT );

                if (getText() == null) {
@@ -374,7 +377,7 @@ public abstract class Components {
    public static JLabel heading(@Nullable final String heading, final int horizontalAlignment) {
        return new JLabel( heading, horizontalAlignment ) {
            {
-                setFont( Res.fonts().controlFont( TEXT_SIZE_HEADING ).deriveFont( Font.BOLD ) );
+                setFont( getFont().deriveFont( Font.BOLD, TEXT_SIZE_HEADING ) );
                setAlignmentX( LEFT_ALIGNMENT );
            }

@@ -409,7 +412,6 @@ public abstract class Components {
    public static JLabel label(@Nullable final String label, final int horizontalAlignment) {
        return new JLabel( label, horizontalAlignment ) {
            {
-                setFont( Res.fonts().controlFont( TEXT_SIZE_CONTROL ) );
                setAlignmentX( LEFT_ALIGNMENT );
            }

@@ -423,7 +425,6 @@ public abstract class Components {
    public static JCheckBox checkBox(final String label) {
        return new JCheckBox( label ) {
            {
-                setFont( Res.fonts().controlFont( TEXT_SIZE_CONTROL ) );
                setBackground( null );
                setAlignmentX( LEFT_ALIGNMENT );
            }
@@ -437,17 +438,17 @@ public abstract class Components {

    public static <E> JComboBox<E> comboBox(final E[] values, final Function<E, String> valueTransformer,
                                            @Nullable final E selectedItem, @Nullable final Consumer<E> selectionConsumer) {
-        return comboBox( CollectionListModel.copy( values ).selection( selectedItem, selectionConsumer ), valueTransformer );
+        return comboBox( new CollectionListModel<>( values ).selection( selectedItem, selectionConsumer ), valueTransformer );
    }

    public static <E> JComboBox<E> comboBox(final Collection<E> values, final Function<E, String> valueTransformer,
                                            @Nullable final Consumer<E> selectionConsumer) {
-        return comboBox( CollectionListModel.copy( values ).selection( selectionConsumer ), valueTransformer );
+        return comboBox( new CollectionListModel<>( values ).selection( selectionConsumer ), valueTransformer );
    }

    public static <E> JComboBox<E> comboBox(final Collection<E> values, final Function<E, String> valueTransformer,
                                            @Nullable final E selectedItem, @Nullable final Consumer<E> selectionConsumer) {
-        return comboBox( CollectionListModel.copy( values ).selection( selectedItem, selectionConsumer ), valueTransformer );
+        return comboBox( new CollectionListModel<>( values ).selection( selectedItem, selectionConsumer ), valueTransformer );
    }

    public static <E> JComboBox<E> comboBox(final ComboBoxModel<E> model, final Function<E, String> valueTransformer) {
@@ -460,8 +461,9 @@ public abstract class Components {
                    @SuppressWarnings({ "unchecked", "SerializableStoresNonSerializable" })
                    public Component getListCellRendererComponent(final JList<?> list, final Object value, final int index,
                                                                  final boolean isSelected, final boolean cellHasFocus) {
+                        String label = valueTransformer.apply( (E) value );
                        super.getListCellRendererComponent(
-                                list, valueTransformer.apply( (E) value ), index, isSelected, cellHasFocus );
+                                list, Strings.isNullOrEmpty( label )? " ": label, index, isSelected, cellHasFocus );
                        setBorder( BorderFactory.createEmptyBorder( 0, 4, 0, 4 ) );

                        return this;
--- a/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/util/ConsumingTrigger.java
+++ b/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/util/ConsumingTrigger.java
@@ -0,0 +1,27 @@
+package com.lyndir.masterpassword.gui.util;
+
+import java.util.function.Consumer;
+import javax.annotation.Nullable;
+
+
+/**
+ * @author lhunath, 2018-08-23
+ */
+public class ConsumingTrigger<T> implements Consumer<T> {
+
+    private final Runnable trigger;
+
+    @Nullable
+    private T value;
+
+    public ConsumingTrigger(final Runnable trigger) {
+        this.trigger = trigger;
+    }
+
+    @Override
+    public void accept(final T t) {
+        value = t;
+
+        trigger.run();
+    }
+}
--- a/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/util/DocumentModel.java
+++ b/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/util/DocumentModel.java
@@ -0,0 +1,95 @@
+package com.lyndir.masterpassword.gui.util;
+
+import com.lyndir.lhunath.opal.system.logging.Logger;
+import java.util.function.Consumer;
+import javax.annotation.Nonnull;
+import javax.annotation.Nullable;
+import javax.swing.event.DocumentEvent;
+import javax.swing.event.DocumentListener;
+import javax.swing.text.BadLocationException;
+import javax.swing.text.Document;
+
+
+/**
+ * @author lhunath, 2018-08-24
+ */
+public class DocumentModel implements Selectable<String, DocumentModel> {
+
+    private static final Logger logger = Logger.get( DocumentModel.class );
+
+    private final Document document;
+
+    @Nullable
+    private DocumentListener documentListener;
+
+    public DocumentModel(final Document document) {
+        this.document = document;
+    }
+
+    @Nonnull
+    public Document getDocument() {
+        return document;
+    }
+
+    @Nullable
+    public String getText() {
+        try {
+            return (document.getLength() > 0)? document.getText( 0, document.getLength() ): null;
+        }
+        catch (final BadLocationException e) {
+            logger.wrn( "While getting text for model", e );
+            return null;
+        }
+    }
+
+    public void setText(@Nullable final String text) {
+        try {
+            if (document.getLength() > 0)
+                document.remove( 0, document.getLength() );
+
+            if (text != null)
+                document.insertString( 0, text, null );
+        }
+        catch (final BadLocationException e) {
+            logger.err( "While setting text for model", e );
+        }
+    }
+
+    @Override
+    public DocumentModel selection(@Nullable final Consumer<String> selectionConsumer) {
+        if (documentListener != null)
+            document.removeDocumentListener( documentListener );
+
+        if (selectionConsumer != null)
+            document.addDocumentListener( documentListener = new DocumentListener() {
+                @Override
+                public void insertUpdate(final DocumentEvent e) {
+                    trigger();
+                }
+
+                @Override
+                public void removeUpdate(final DocumentEvent e) {
+                    trigger();
+                }
+
+                @Override
+                public void changedUpdate(final DocumentEvent e) {
+                    trigger();
+                }
+
+                private void trigger() {
+                    selectionConsumer.accept( getText() );
+                }
+            } );
+
+        return this;
+    }
+
+    @Override
+    public DocumentModel selection(@Nullable final String selectedItem, @Nullable final Consumer<String> selectionConsumer) {
+        selection( selectionConsumer );
+        setText( selectedItem );
+
+        return this;
+    }
+}
--- a/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/util/Res.java
+++ b/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/util/Res.java
@@ -25,12 +25,13 @@ import com.google.common.io.Resources;
 import com.google.common.util.concurrent.*;
 import com.lyndir.lhunath.opal.system.logging.Logger;
 import com.lyndir.masterpassword.MPIdenticon;
-import com.lyndir.masterpassword.gui.SwingExecutorService;
 import java.awt.*;
 import java.io.IOException;
 import java.lang.ref.SoftReference;
 import java.util.Map;
 import java.util.concurrent.*;
+import java.util.function.Consumer;
+import javax.annotation.Nullable;
 import javax.swing.*;
 import org.jetbrains.annotations.NonNls;
 import org.joda.time.*;
@@ -72,6 +73,15 @@ public abstract class Res {
        return job( job, 0, TimeUnit.MILLISECONDS );
    }

+    public static <V> void job(final Callable<V> job, final Consumer<V> callback) {
+        Futures.addCallback( job( job, 0, TimeUnit.MILLISECONDS ), new FailableCallback<V>( logger ) {
+            @Override
+            public void onSuccess(@Nullable final V result) {
+                callback.accept( result );
+            }
+        }, uiExecutor() );
+    }
+
    public static <V> ListenableFuture<V> job(final Callable<V> job, final long delay, final TimeUnit timeUnit) {
        return jobExecutor.schedule( job, delay, timeUnit );
    }
@@ -150,6 +160,14 @@ public abstract class Res {
            return icon( "media/icon_settings.png" );
        }

+        public Icon edit() {
+            return icon( "media/icon_edit.png" );
+        }
+
+        public Icon key() {
+            return icon( "media/icon_key.png" );
+        }
+
        public Icon avatar(final int index) {
            return icon( strf( "media/avatar-%d.png", index % avatars() ) );
        }
--- a/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/util/Selectable.java
+++ b/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/util/Selectable.java
@@ -11,5 +11,5 @@ public interface Selectable<E, T> {

    T selection(@Nullable Consumer<E> selectionConsumer);

-    T selection(E selectedItem, @Nullable Consumer<E> selectionConsumer);
+    T selection(@Nullable E selectedItem, @Nullable Consumer<E> selectionConsumer);
 }
--- a/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/util/SwingExecutorService.java
+++ b/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/util/SwingExecutorService.java
@@ -1,4 +1,4 @@
-package com.lyndir.masterpassword.gui;
+package com.lyndir.masterpassword.gui.util;

 import static com.lyndir.lhunath.opal.system.util.ObjectUtils.*;

@@ -29,9 +29,9 @@ public class SwingExecutorService extends AbstractExecutorService {

    @Override
    public void shutdown() {
-        shutdown = true;
-
        synchronized (pendingCommands) {
+            shutdown = true;
+
            if (pendingCommands.isEmpty())
                terminated.add( true );
        }
@@ -49,7 +49,9 @@ public class SwingExecutorService extends AbstractExecutorService {

    @Override
    public boolean isShutdown() {
-        return shutdown;
+        synchronized (pendingCommands) {
+            return shutdown;
+        }
    }

    @Override
@@ -65,10 +67,10 @@ public class SwingExecutorService extends AbstractExecutorService {

    @Override
    public void execute(@NotNull final Runnable command) {
-        if (shutdown)
-            throw new RejectedExecutionException( "Executor is shut down." );
-
        synchronized (pendingCommands) {
+            if (shutdown)
+                throw new RejectedExecutionException( "Executor is shut down." );
+
            pendingCommands.add( command );
        }

--- a/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/util/UnsignedIntegerModel.java
+++ b/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/util/UnsignedIntegerModel.java
@@ -109,13 +109,14 @@ public class UnsignedIntegerModel extends SpinnerNumberModel implements Selectab
    }

    @Override
-    public UnsignedIntegerModel selection(final UnsignedInteger selectedItem, @Nullable final Consumer<UnsignedInteger> selectionConsumer) {
+    public UnsignedIntegerModel selection(@Nullable final UnsignedInteger selectedItem,
+                                          @Nullable final Consumer<UnsignedInteger> selectionConsumer) {
        if (changeListener != null) {
            removeChangeListener( changeListener );
            changeListener = null;
        }

-        setValue( selectedItem );
+        setValue( (selectedItem != null)? selectedItem: getMinimum() );
        return selection( selectionConsumer );
    }

--- a/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/view/FilesPanel.java
+++ b/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/view/FilesPanel.java
@@ -9,8 +9,6 @@ import com.lyndir.masterpassword.model.MPUser;
 import com.lyndir.masterpassword.model.impl.MPFileUser;
 import com.lyndir.masterpassword.model.impl.MPFileUserManager;
 import java.awt.*;
-import java.util.Collection;
-import java.util.concurrent.CopyOnWriteArraySet;
 import javax.annotation.Nullable;
 import javax.swing.*;

@@ -25,9 +23,7 @@ public class FilesPanel extends JPanel implements MPFileUserManager.Listener, Ma
                                                            "Click to change the user's avatar." );

    private final CollectionListModel<MPUser<?>> usersModel =
-            CollectionListModel.<MPUser<?>>copy( MPFileUserManager.get().getFiles() ).selection( MasterPassword.get()::activateUser );
-    private final JComboBox<? extends MPUser<?>> userField  =
-            Components.comboBox( usersModel, user -> ifNotNull( user, MPUser::getFullName ) );
+            new CollectionListModel<MPUser<?>>( MPFileUserManager.get().getFiles() ).selection( MasterPassword.get()::activateUser );

    protected FilesPanel() {
        setOpaque( false );
@@ -46,7 +42,7 @@ public class FilesPanel extends JPanel implements MPFileUserManager.Listener, Ma
        add( Components.strut( Components.margin() ) );

        // User Selection
-        add( userField );
+        add( Components.comboBox( usersModel, user -> ifNotNull( user, MPUser::getFullName ) ) );

        MPFileUserManager.get().addListener( this );
        MasterPassword.get().addListener( this );
--- a/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/view/MasterPasswordFrame.java
+++ b/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/view/MasterPasswordFrame.java
@@ -3,12 +3,10 @@ package com.lyndir.masterpassword.gui.view;
 import com.lyndir.lhunath.opal.system.logging.Logger;
 import com.lyndir.masterpassword.gui.util.Components;
 import com.lyndir.masterpassword.gui.util.Res;
-import com.lyndir.masterpassword.model.MPUser;
 import com.lyndir.masterpassword.model.impl.MPFileUserManager;
 import java.awt.*;
 import java.awt.event.ComponentAdapter;
 import java.awt.event.ComponentEvent;
-import javax.annotation.Nullable;
 import javax.swing.*;
 import javax.swing.border.BevelBorder;

@@ -21,26 +19,24 @@ public class MasterPasswordFrame extends JFrame {

    private static final Logger logger = Logger.get( MasterPasswordFrame.class );

-    @SuppressWarnings("FieldCanBeLocal")
-    private final Components.GradientPanel root        = Components.borderPanel( Res.colors().frameBg(), BoxLayout.PAGE_AXIS );
-    private final FilesPanel               filesPanel  = new FilesPanel();
-    private final JPanel                   userPanel   = Components.panel( new BorderLayout( 0, 0 ) );
-    private final UserContentPanel         userContent = new UserContentPanel();
+    private final UserContentPanel         userContent;

    @SuppressWarnings("MagicNumber")
    public MasterPasswordFrame() {
        super( "Master Password" );

-        setContentPane( root );
-        root.add( filesPanel );
-        root.add( Components.strut() );
-        root.add( userPanel );
+        JPanel root, userPanel;
+        setContentPane( root = Components.borderPanel( Res.colors().frameBg(), BoxLayout.PAGE_AXIS ) );
+
+        root.add( new FilesPanel() );
+        root.add( Components.strut() );
+        root.add( userPanel = Components.panel( new BorderLayout( 0, 0 ) ) );

-        userPanel.add( userContent.getUserToolbar(), BorderLayout.LINE_START );
-        userPanel.add( userContent.getSiteToolbar(), BorderLayout.LINE_END );
        userPanel.add( Components.borderPanel(
                BorderFactory.createBevelBorder( BevelBorder.RAISED, Res.colors().controlBorder(), Res.colors().frameBg() ),
-                Res.colors().controlBg(), BoxLayout.PAGE_AXIS, userContent ), BorderLayout.CENTER );
+                Res.colors().controlBg(), BoxLayout.PAGE_AXIS, userContent = new UserContentPanel() ), BorderLayout.CENTER );
+        userPanel.add( userContent.getUserToolbar(), BorderLayout.LINE_START );
+        userPanel.add( userContent.getSiteToolbar(), BorderLayout.LINE_END );

        addComponentListener( new ComponentHandler() );
        setPreferredSize( new Dimension( 800, 560 ) );
--- a/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/view/UserContentPanel.java
+++ b/platform-independent/java/gui/src/main/java/com/lyndir/masterpassword/gui/view/UserContentPanel.java
@@ -10,12 +10,12 @@ import com.lyndir.lhunath.opal.system.util.ObjectUtils;
 import com.lyndir.masterpassword.*;
 import com.lyndir.masterpassword.gui.MPGuiConstants;
 import com.lyndir.masterpassword.gui.MasterPassword;
-import com.lyndir.masterpassword.gui.model.MPIncognitoUser;
-import com.lyndir.masterpassword.gui.model.MPNewSite;
+import com.lyndir.masterpassword.gui.model.*;
 import com.lyndir.masterpassword.gui.util.*;
 import com.lyndir.masterpassword.gui.util.Platform;
 import com.lyndir.masterpassword.model.*;
 import com.lyndir.masterpassword.model.impl.*;
+import com.lyndir.masterpassword.util.Utilities;
 import java.awt.*;
 import java.awt.datatransfer.StringSelection;
 import java.awt.datatransfer.Transferable;
@@ -34,6 +34,7 @@ import javax.annotation.Nullable;
 import javax.swing.*;
 import javax.swing.event.DocumentEvent;
 import javax.swing.event.DocumentListener;
+import javax.swing.text.PlainDocument;


 /**
@@ -42,9 +43,11 @@ import javax.swing.event.DocumentListener;
@SuppressWarnings("SerializableStoresNonSerializable")
 public class UserContentPanel extends JPanel implements MasterPassword.Listener, MPUser.Listener {

-    private static final Random  random     = new Random();
-    private static final Logger  logger     = Logger.get( UserContentPanel.class );
-    private static final JButton iconButton = Components.button( Res.icons().user(), null, null );
+    private static final Random    random             = new Random();
+    private static final int       SIZE_RESULT        = 48;
+    private static final Logger    logger             = Logger.get( UserContentPanel.class );
+    private static final JButton   iconButton         = Components.button( Res.icons().user(), null, null );
+    private static final KeyStroke copyLoginKeyStroke = KeyStroke.getKeyStroke( KeyEvent.VK_ENTER, InputEvent.SHIFT_DOWN_MASK );

    private final JButton addButton    = Components.button( Res.icons().add(), event -> addUser(),
                                                            "Add a new user to Master Password." );
@@ -290,9 +293,9 @@ public class UserContentPanel extends JPanel implements MasterPassword.Listener,
        private final JButton resetButton  = Components.button( Res.icons().reset(), event -> resetUser(),
                                                                "Change the master password for this user." );

-        private final JPasswordField masterPasswordField = Components.passwordField();
-        private final JLabel         errorLabel          = Components.label();
-        private final JLabel         identiconLabel      = Components.label( SwingConstants.CENTER );
+        private final JPasswordField masterPasswordField;
+        private final JLabel         errorLabel;
+        private final JLabel         identiconLabel;

        private Future<?> identiconJob;

@@ -312,16 +315,16 @@ public class UserContentPanel extends JPanel implements MasterPassword.Listener,
            add( Components.heading( user.getFullName(), SwingConstants.CENTER ) );
            add( Components.strut() );

-            add( identiconLabel );
+            add( identiconLabel = Components.label( SwingConstants.CENTER ) );
            identiconLabel.setFont( Res.fonts().emoticonsFont( Components.TEXT_SIZE_CONTROL ) );
            add( Box.createGlue() );

            add( Components.label( "Master Password:" ) );
            add( Components.strut() );
-            add( masterPasswordField );
+            add( masterPasswordField = Components.passwordField() );
            masterPasswordField.addActionListener( this );
            masterPasswordField.getDocument().addDocumentListener( this );
-            add( errorLabel );
+            add( errorLabel = Components.label() );
            errorLabel.setForeground( Res.colors().errorFg() );
            add( Box.createGlue() );
        }
@@ -460,29 +463,31 @@ public class UserContentPanel extends JPanel implements MasterPassword.Listener,

    private final class AuthenticatedUserPanel extends JPanel implements KeyListener, MPUser.Listener {

-        public static final int SIZE_RESULT = 48;
-
        private final JButton userButton      = Components.button( Res.icons().user(), event -> showUserPreferences(),
                                                                   "Show user preferences." );
        private final JButton logoutButton    = Components.button( Res.icons().lock(), event -> logoutUser(),
                                                                   "Sign out and lock user." );
        private final JButton settingsButton  = Components.button( Res.icons().settings(), event -> showSiteSettings(),
                                                                   "Show site settings." );
-        private final JButton questionsButton = Components.button( Res.icons().question(), null,
+        private final JButton questionsButton = Components.button( Res.icons().question(), event -> showSiteQuestions(),
                                                                   "Show site recovery questions." );
+        private final JButton editButton      = Components.button( Res.icons().edit(), event -> showSiteValues(),
+                                                                   "Set/save personal password/login." );
+        private final JButton keyButton       = Components.button( Res.icons().key(), event -> showSiteKeys(),
+                                                                   "Cryptographic site keys." );
        private final JButton deleteButton    = Components.button( Res.icons().delete(), event -> deleteSite(),
                                                                   "Delete the site from the user." );

        @Nonnull
        private final MPUser<?>                      user;
-        private final JLabel                         passwordLabel = Components.label( SwingConstants.CENTER );
-        private final JLabel                         passwordField = Components.heading( SwingConstants.CENTER );
-        private final JLabel                         queryLabel    = Components.label();
-        private final JTextField                     queryField    = Components.textField( null, this::updateSites );
-        private final CollectionListModel<MPSite<?>> sitesModel    =
-                new CollectionListModel<MPSite<?>>().selection( this::showSiteResult );
-        private final JList<MPSite<?>>               sitesList     =
-                Components.list( sitesModel, this::getSiteDescription );
+        private final JLabel                         passwordLabel;
+        private final JLabel                         passwordField;
+        private final JLabel                         answerLabel;
+        private final JLabel                         answerField;
+        private final JLabel                         queryLabel;
+        private final JTextField                     queryField;
+        private final CollectionListModel<MPSite<?>> sitesModel;
+        private final JList<MPSite<?>>               sitesList;

        private Future<?> updateSitesJob;

@@ -499,30 +504,49 @@ public class UserContentPanel extends JPanel implements MasterPassword.Listener,

            siteToolbar.add( settingsButton );
            siteToolbar.add( questionsButton );
+            siteToolbar.add( editButton );
+            siteToolbar.add( keyButton );
            siteToolbar.add( deleteButton );
            settingsButton.setEnabled( false );
+            questionsButton.setEnabled( false );
+            editButton.setEnabled( false );
+            keyButton.setEnabled( false );
            deleteButton.setEnabled( false );

+            answerLabel = Components.label( "Answer:" );
+            answerField = Components.heading( SwingConstants.CENTER );
+            answerField.setForeground( Res.colors().highlightFg() );
+            answerField.setFont( Res.fonts().bigValueFont( SIZE_RESULT ) );
+
            add( Components.heading( user.getFullName(), SwingConstants.CENTER ) );

-            add( passwordLabel );
-            add( passwordField );
+            add( passwordLabel = Components.label( SwingConstants.CENTER ) );
+            add( passwordField = Components.heading( SwingConstants.CENTER ) );
            passwordField.setForeground( Res.colors().highlightFg() );
            passwordField.setFont( Res.fonts().bigValueFont( SIZE_RESULT ) );
            add( Box.createGlue() );
            add( Components.strut() );

-            add( queryLabel );
+            add( queryLabel = Components.label() );
            queryLabel.setText( strf( "%s's password for:", user.getFullName() ) );
-            add( queryField );
+            add( queryField = Components.textField( null, this::updateSites ) );
            queryField.putClientProperty( "JTextField.variant", "search" );
-            queryField.addActionListener( event -> useSite() );
+            queryField.addActionListener( this::useSite );
+            queryField.getInputMap().put( copyLoginKeyStroke, JTextField.notifyAction );
            queryField.addKeyListener( this );
            queryField.requestFocusInWindow();
            add( Components.strut() );
-            add( Components.scrollPane( sitesList ) );
-            sitesModel.registerList( sitesList );
-            add( Box.createGlue() );
+
+            add( Components.scrollPane( sitesList = Components.list(
+                    sitesModel = new CollectionListModel<MPSite<?>>().selection( this::showSiteResult ),
+                    this::getSiteDescription ) ) );
+            add( Components.strut() );
+
+            add( Components.label( strf(
+                    "Press %s to copy password, %s+%s to copy login name.",
+                    KeyEvent.getKeyText( KeyEvent.VK_ENTER ),
+                    InputEvent.getModifiersExText( copyLoginKeyStroke.getModifiers() ),
+                    KeyEvent.getKeyText( copyLoginKeyStroke.getKeyCode() ) ) ) );

            addHierarchyListener( e -> {
                if (null != SwingUtilities.windowForComponent( this ))
@@ -572,12 +596,14 @@ public class UserContentPanel extends JPanel implements MasterPassword.Listener,
                            Components.strut() );

            components.add( Components.label( "Password Type:" ),
-                            Components.comboBox( MPResultType.values(), MPResultType::getLongName,
+                            Components.comboBox( MPResultType.values(), type -> getTypeDescription(
+                                    type, user.getDefaultType(), user.getAlgorithm().mpw_default_result_type() ),
                                                 site.getResultType(), site::setResultType ),
                            Components.strut() );

            components.add( Components.label( "Login Type:" ),
-                            Components.comboBox( MPResultType.values(), MPResultType::getLongName,
+                            Components.comboBox( MPResultType.values(), type -> getTypeDescription(
+                                    type, user.getAlgorithm().mpw_default_login_type() ),
                                                 site.getLoginType(), site::setLoginType ),
                            Components.strut() );

@@ -587,10 +613,179 @@ public class UserContentPanel extends JPanel implements MasterPassword.Listener,
                                Components.textField( fileSite.getUrl(), fileSite::setUrl ),
                                Components.strut() );

-            Components.showDialog( this, site.getSiteName(), new JOptionPane( Components.panel(
+            Components.showDialog( this, strf( "Settings for %s", site.getSiteName() ), new JOptionPane( Components.panel(
                    BoxLayout.PAGE_AXIS, components.build().toArray( new Component[0] ) ) ) );
        }

+        private String getTypeDescription(final MPResultType type, final MPResultType... defaults) {
+            boolean isDefault = false;
+            for (final MPResultType d : defaults)
+                if (isDefault = type == d)
+                    break;
+
+            return strf( "<html>%s%s%s, %s", isDefault? "<b>": "", type.getLongName(), isDefault? "</b>": "", type.getDescription() );
+        }
+
+        public void showSiteQuestions() {
+            MPSite<?> site = sitesModel.getSelectedItem();
+            if (site == null)
+                return;
+
+            CollectionListModel<MPQuestion> questionsModel = new CollectionListModel<MPQuestion>().selection( this::showQuestionResult );
+            JList<MPQuestion> questionsList = Components.list(
+                    questionsModel, question -> Strings.isNullOrEmpty( question.getKeyword() )? "<site>": question.getKeyword() );
+            JTextField queryField = Components.textField( null, query -> Res.job( () -> {
+                Collection<MPQuestion> questions = new LinkedList<>( site.findQuestions( query ) );
+                if (questions.stream().noneMatch( question -> question.getKeyword().equalsIgnoreCase( query ) ))
+                    questions.add( new MPNewQuestion( site, Utilities.ifNotNullElse( query, "" ) ) );
+
+                Res.ui( () -> questionsModel.set( questions ) );
+            } ) );
+            queryField.putClientProperty( "JTextField.variant", "search" );
+            queryField.addActionListener( event -> useQuestion( questionsModel.getSelectedItem() ) );
+            queryField.addKeyListener( new KeyAdapter() {
+                @Override
+                public void keyPressed(final KeyEvent event) {
+                    if ((event.getKeyCode() == KeyEvent.VK_UP) || (event.getKeyCode() == KeyEvent.VK_DOWN))
+                        questionsList.dispatchEvent( event );
+                }
+
+                @Override
+                public void keyReleased(final KeyEvent event) {
+                    if ((event.getKeyCode() == KeyEvent.VK_UP) || (event.getKeyCode() == KeyEvent.VK_DOWN))
+                        questionsList.dispatchEvent( event );
+                }
+            } );
+
+            Components.showDialog( this, strf( "Recovery answers for %s", site.getSiteName() ), new JOptionPane( Components.panel(
+                    BoxLayout.PAGE_AXIS,
+                    Components.label( "Security Question Keyword:" ), queryField,
+                    Components.strut(),
+                    answerLabel, answerField,
+                    Components.strut(),
+                    Components.scrollPane( questionsList ) ) ) {
+                @Override
+                public void selectInitialValue() {
+                    queryField.requestFocusInWindow();
+                }
+            } );
+        }
+
+        public void showSiteValues() {
+            MPSite<?> site = sitesModel.getSelectedItem();
+            if (site == null)
+                return;
+
+            try {
+                JTextField passwordField = Components.textField( site.getResult(), null );
+                JTextField loginField    = Components.textField( site.getLogin(), null );
+                passwordField.setEditable( site.getResultType().getTypeClass() == MPResultTypeClass.Stateful );
+                loginField.setEditable( site.getLoginType().getTypeClass() == MPResultTypeClass.Stateful );
+
+                if (JOptionPane.OK_OPTION == Components.showDialog( this, site.getSiteName(), new JOptionPane(
+                        Components.panel(
+                                BoxLayout.PAGE_AXIS,
+                                Components.label( strf( "<html>Site Login (currently set to: <b>%s</b>):",
+                                                        getTypeDescription( site.getLoginType() ) ) ),
+                                loginField,
+                                Components.strut(),
+                                Components.label( strf( "<html>Site Password (currently set to: <b>%s</b>):",
+                                                        getTypeDescription( site.getResultType() ) ) ),
+                                passwordField,
+                                Components.strut(),
+                                Components.label( "<html>To save a personal value in these fields,\n" +
+                                                  "change the type to <b>Saved</b> in the site's settings." ) ),
+                        JOptionPane.PLAIN_MESSAGE, JOptionPane.OK_CANCEL_OPTION ) {
+                    @Override
+                    public void selectInitialValue() {
+                        passwordField.requestFocusInWindow();
+                    }
+                } )) {
+                    if (site instanceof MPFileSite) {
+                        MPFileSite fileSite = (MPFileSite) site;
+
+                        if (site.getResultType().getTypeClass() == MPResultTypeClass.Stateful)
+                            fileSite.setSitePassword( site.getResultType(), passwordField.getText() );
+                        if (site.getLoginType().getTypeClass() == MPResultTypeClass.Stateful)
+                            fileSite.setLoginName( site.getLoginType(), loginField.getText() );
+                    }
+                }
+            }
+            catch (final MPKeyUnavailableException | MPAlgorithmException e) {
+                logger.err( e, "While computing site edit results." );
+            }
+        }
+
+        public void showSiteKeys() {
+            MPSite<?> site = sitesModel.getSelectedItem();
+            if (site == null)
+                return;
+
+            JTextArea resultField = Components.textArea();
+            resultField.setEnabled( false );
+
+            CollectionListModel<MPKeyPurpose> purposeModel = new CollectionListModel<>( MPKeyPurpose.values() );
+            DocumentModel                     contextModel = new DocumentModel( new PlainDocument() );
+            UnsignedIntegerModel              counterModel = new UnsignedIntegerModel( UnsignedInteger.ONE );
+            CollectionListModel<MPResultType> typeModel    = new CollectionListModel<>( MPResultType.values() );
+            DocumentModel                     stateModel   = new DocumentModel( new PlainDocument() );
+
+            Runnable trigger = () -> Res.job( () -> {
+                try {
+                    MPKeyPurpose purpose = purposeModel.getSelectedItem();
+                    MPResultType type    = typeModel.getSelectedItem();
+
+                    String result = ((purpose == null) || (type == null))? null:
+                            site.getResult( purpose, contextModel.getText(), counterModel.getNumber(), type, stateModel.getText() );
+
+                    Res.ui( () -> resultField.setText( result ) );
+                }
+                catch (final MPKeyUnavailableException | MPAlgorithmException e) {
+                    logger.err( e, "While computing site edit results." );
+                }
+            } );
+
+            purposeModel.selection( MPKeyPurpose.Authentication, p -> trigger.run() );
+            contextModel.selection( c -> trigger.run() );
+            counterModel.selection( c -> trigger.run() );
+            typeModel.selection( MPResultType.DeriveKey, t -> {
+                switch (t) {
+                    case DeriveKey:
+                        stateModel.setText( Integer.toString( site.getAlgorithm().mpw_keySize_min() ) );
+                        break;
+
+                    default:
+                        stateModel.setText( null );
+                }
+
+                trigger.run();
+            } );
+            stateModel.selection( c -> trigger.run() );
+
+            if (JOptionPane.OK_OPTION == Components.showDialog( this, site.getSiteName(), new JOptionPane( Components.panel(
+                    BoxLayout.PAGE_AXIS,
+                    Components.heading( "Key Calculator" ),
+                    Components.label( "Purpose:" ),
+                    Components.comboBox( purposeModel, MPKeyPurpose::getShortName ),
+                    Components.strut(),
+                    Components.label( "Context:" ),
+                    Components.textField( contextModel.getDocument() ),
+                    Components.label( "Counter:" ),
+                    Components.spinner( counterModel ),
+                    Components.label( "Type:" ),
+                    Components.comboBox( typeModel, this::getTypeDescription ),
+                    Components.label( "State:" ),
+                    Components.scrollPane( Components.textField( stateModel.getDocument() ) ),
+                    Components.strut(),
+                    resultField ) ) {
+                {
+                    setOptions( new Object[]{ "Copy", "Cancel" } );
+                    setInitialValue( getOptions()[0] );
+                }
+            } ))
+                copyResult( resultField.getText() );
+        }
+
        public void deleteSite() {
            MPSite<?> site = sitesModel.getSelectedItem();
            if (site == null)
@@ -626,69 +821,121 @@ public class UserContentPanel extends JPanel implements MasterPassword.Listener,
                         Joiner.on( " - " ).skipNulls().join( parameters.build() ) );
        }

-        private void useSite() {
+        private void useSite(final ActionEvent event) {
            MPSite<?> site = sitesModel.getSelectedItem();
            if (site instanceof MPNewSite) {
                if (JOptionPane.YES_OPTION == JOptionPane.showConfirmDialog(
                        this, strf( "<html>Remember the site <strong>%s</strong>?</html>", site.getSiteName() ),
                        "New Site", JOptionPane.YES_NO_OPTION )) {
                    sitesModel.setSelectedItem( user.addSite( site.getSiteName() ) );
-                    useSite();
+                    useSite( event );
                }
                return;
            }

-            showSiteResult( site, result -> {
+            boolean loginResult = (copyLoginKeyStroke.getModifiers() & event.getModifiers()) != 0;
+            showSiteResult( site, loginResult, result -> {
                if (result == null)
                    return;

                if (site instanceof MPFileSite)
                    ((MPFileSite) site).use();

-                Transferable clipboardContents = new StringSelection( result );
-                Toolkit.getDefaultToolkit().getSystemClipboard().setContents( clipboardContents, null );
-
-                Res.ui( () -> {
-                    Window window = SwingUtilities.windowForComponent( UserContentPanel.this );
-                    if (window instanceof Frame)
-                        ((Frame) window).setExtendedState( Frame.ICONIFIED );
-                } );
+                copyResult( result );
            } );
        }

        private void showSiteResult(@Nullable final MPSite<?> site) {
-            showSiteResult( site, null );
+            showSiteResult( site, false, result -> {
+            } );
        }

-        private void showSiteResult(@Nullable final MPSite<?> site, @Nullable final Consumer<String> resultCallback) {
-            if (site == null) {
-                if (resultCallback != null)
-                    resultCallback.accept( null );
-                Res.ui( () -> {
-                    passwordLabel.setText( " " );
-                    passwordField.setText( " " );
-                    settingsButton.setEnabled( false );
-                    deleteButton.setEnabled( false );
-                } );
-                return;
-            }
-
+        private void showSiteResult(@Nullable final MPSite<?> site, final boolean loginResult, final Consumer<String> resultCallback) {
            Res.job( () -> {
                try {
-                    String result = site.getResult();
-                    if (resultCallback != null)
-                        resultCallback.accept( result );
-
-                    Res.ui( () -> {
-                        passwordLabel.setText( strf( "Your password for %s:", site.getSiteName() ) );
-                        passwordField.setText( result );
-                        settingsButton.setEnabled( true );
-                        deleteButton.setEnabled( true );
-                    } );
+                    if (site != null)
+                        return loginResult? site.getLogin(): site.getResult();
                }
                catch (final MPKeyUnavailableException | MPAlgorithmException e) {
                    logger.err( e, "While resolving password for: %s", site );
                }
+
+                return null;
+            }, resultCallback.andThen( result -> Res.ui( () -> {
+                passwordLabel.setText( ((result != null) && (site != null))? strf( "Your password for %s:", site.getSiteName() ): " " );
+                passwordField.setText( (result != null)? result: " " );
+                settingsButton.setEnabled( result != null );
+                questionsButton.setEnabled( result != null );
+                editButton.setEnabled( result != null );
+                keyButton.setEnabled( result != null );
+                deleteButton.setEnabled( result != null );
+            } ) ) );
+        }
+
+        private void useQuestion(@Nullable final MPQuestion question) {
+            if (question instanceof MPNewQuestion) {
+                if (JOptionPane.YES_OPTION == JOptionPane.showConfirmDialog(
+                        this,
+                        strf( "<html>Remember the security question with keyword <strong>%s</strong>?</html>",
+                              Strings.isNullOrEmpty( question.getKeyword() )? "<empty>": question.getKeyword() ),
+                        "New Question", JOptionPane.YES_NO_OPTION )) {
+                    useQuestion( question.getSite().addQuestion( question.getKeyword() ) );
+                }
+                return;
+            }
+
+            showQuestionResult( question, result -> {
+                if (result == null)
+                    return;
+
+                if (question instanceof MPFileQuestion)
+                    ((MPFileQuestion) question).use();
+
+                copyResult( result );
+            } );
+        }
+
+        private void showQuestionResult(@Nullable final MPQuestion question) {
+            showQuestionResult( question, answer -> {
+            } );
+        }
+
+        private void showQuestionResult(@Nullable final MPQuestion question, final Consumer<String> resultCallback) {
+            Res.job( () -> {
+                try {
+                    if (question != null)
+                        return question.getAnswer();
+                }
+                catch (final MPKeyUnavailableException | MPAlgorithmException e) {
+                    logger.err( e, "While resolving answer for: %s", question );
+                }
+
+                return null;
+            }, resultCallback.andThen( answer -> Res.ui( () -> {
+                if ((answer == null) || (question == null))
+                    answerLabel.setText( " " );
+                else
+                    answerLabel.setText(
+                            Strings.isNullOrEmpty( question.getKeyword() )?
+                                    strf( "<html>Answer for site <b>%s</b>:", question.getSite().getSiteName() ):
+                                    strf( "<html>Answer for site <b>%s</b>, of question with keyword <b>%s</b>:",
+                                          question.getSite().getSiteName(), question.getKeyword() ) );
+                answerField.setText( (answer != null)? answer: " " );
+            } ) ) );
+        }
+
+        private void copyResult(final String result) {
+            Transferable clipboardContents = new StringSelection( result );
+            Toolkit.getDefaultToolkit().getSystemClipboard().setContents( clipboardContents, null );
+
+            Res.ui( () -> {
+                Window answerDialog = SwingUtilities.windowForComponent( answerField );
+                if (answerDialog instanceof Dialog)
+                    answerDialog.setVisible( false );
+
+                Window window = SwingUtilities.windowForComponent( UserContentPanel.this );
+                if (window instanceof Frame)
+                    ((Frame) window).setExtendedState( Frame.ICONIFIED );
            } );
        }

@@ -713,13 +960,11 @@ public class UserContentPanel extends JPanel implements MasterPassword.Listener,
                updateSitesJob.cancel( true );

            updateSitesJob = Res.job( () -> {
-                Collection<MPSite<?>> sites = new LinkedList<>();
-                if (!Strings.isNullOrEmpty( query )) {
-                    sites.addAll( new LinkedList<>( user.findSites( query ) ) );
+                Collection<MPSite<?>> sites = new LinkedList<>( user.findSites( query ) );

+                if (!Strings.isNullOrEmpty( query ))
                    if (sites.stream().noneMatch( site -> site.getSiteName().equalsIgnoreCase( query ) ))
                        sites.add( new MPNewSite( user, query ) );
-                }

                Res.ui( () -> sitesModel.set( sites ) );
            } );
--- a/platform-independent/java/gui/src/main/resources/media/icon_edit.png
+++ b/platform-independent/java/gui/src/main/resources/media/icon_edit.png
--- a/platform-independent/java/gui/src/main/resources/media/icon_edit@2x.png
+++ b/platform-independent/java/gui/src/main/resources/media/icon_edit@2x.png
--- a/platform-independent/java/gui/src/main/resources/media/icon_key.png
+++ b/platform-independent/java/gui/src/main/resources/media/icon_key.png
--- a/platform-independent/java/gui/src/main/resources/media/icon_key@2x.png
+++ b/platform-independent/java/gui/src/main/resources/media/icon_key@2x.png
--- a/platform-independent/java/model/src/main/java/com/lyndir/masterpassword/model/MPQuestion.java
+++ b/platform-independent/java/model/src/main/java/com/lyndir/masterpassword/model/MPQuestion.java
@@ -40,6 +40,13 @@ public interface MPQuestion extends Comparable<MPQuestion> {

    void setType(MPResultType type);

+    @Nonnull
+    default String getAnswer()
+            throws MPKeyUnavailableException, MPAlgorithmException {
+
+        return getAnswer( null );
+    }
+
    @Nonnull
    String getAnswer(@Nullable String state)
            throws MPKeyUnavailableException, MPAlgorithmException;
--- a/platform-independent/java/model/src/main/java/com/lyndir/masterpassword/model/MPSite.java
+++ b/platform-independent/java/model/src/main/java/com/lyndir/masterpassword/model/MPSite.java
@@ -18,6 +18,7 @@

 package com.lyndir.masterpassword.model;

+import com.google.common.collect.ImmutableCollection;
 import com.google.common.primitives.UnsignedInteger;
 import com.lyndir.masterpassword.*;
 import java.util.Collection;
@@ -57,28 +58,38 @@ public interface MPSite<Q extends MPQuestion> extends Comparable<MPSite<?>> {

    void setLoginType(@Nullable MPResultType loginType);

+    @Nullable
    default String getResult()
            throws MPKeyUnavailableException, MPAlgorithmException {
-
        return getResult( MPKeyPurpose.Authentication );
    }

-    @Nonnull
+    @Nullable
    default String getResult(final MPKeyPurpose keyPurpose)
            throws MPKeyUnavailableException, MPAlgorithmException {
        return getResult( keyPurpose, null );
    }

-    @Nonnull
+    @Nullable
    default String getResult(final MPKeyPurpose keyPurpose, @Nullable final String keyContext)
            throws MPKeyUnavailableException, MPAlgorithmException {
        return getResult( keyPurpose, keyContext, null );
    }

-    @Nonnull
+    @Nullable
    String getResult(MPKeyPurpose keyPurpose, @Nullable String keyContext, @Nullable String state)
            throws MPKeyUnavailableException, MPAlgorithmException;

+    @Nullable
+    String getResult(MPKeyPurpose keyPurpose, @Nullable String keyContext,
+                     @Nullable UnsignedInteger counter, MPResultType type, @Nullable String state)
+            throws MPKeyUnavailableException, MPAlgorithmException;
+
+    @Nonnull
+    String getState(MPKeyPurpose keyPurpose, @Nullable String keyContext,
+                    @Nullable UnsignedInteger counter, MPResultType type, String state)
+            throws MPKeyUnavailableException, MPAlgorithmException;
+
    @Nonnull
    default String getLogin()
            throws MPKeyUnavailableException, MPAlgorithmException {
@@ -94,10 +105,17 @@ public interface MPSite<Q extends MPQuestion> extends Comparable<MPSite<?>> {
    @Nonnull
    MPUser<?> getUser();

-    boolean addQuestion(Q question);
+    @Nonnull
+    Q addQuestion(String keyword);
+
+    @Nonnull
+    Q addQuestion(Q question);

    boolean deleteQuestion(Q question);

    @Nonnull
    Collection<Q> getQuestions();
+
+    @Nonnull
+    ImmutableCollection<Q> findQuestions(@Nullable String query);
 }
--- a/platform-independent/java/model/src/main/java/com/lyndir/masterpassword/model/MPUser.java
+++ b/platform-independent/java/model/src/main/java/com/lyndir/masterpassword/model/MPUser.java
@@ -100,6 +100,7 @@ public interface MPUser<S extends MPSite<?>> extends Comparable<MPUser<?>> {

    // - Relations

+    @Nonnull
    S addSite(String siteName);

    @Nonnull
@@ -113,9 +114,9 @@ public interface MPUser<S extends MPSite<?>> extends Comparable<MPUser<?>> {
    @Nonnull
    ImmutableCollection<S> findSites(@Nullable String query);

-    boolean addListener(Listener listener);
+    void addListener(Listener listener);

-    boolean removeListener(Listener listener);
+    void removeListener(Listener listener);

    interface Listener {

--- a/platform-independent/java/model/src/main/java/com/lyndir/masterpassword/model/impl/MPBasicQuestion.java
+++ b/platform-independent/java/model/src/main/java/com/lyndir/masterpassword/model/impl/MPBasicQuestion.java
@@ -22,6 +22,7 @@ import static com.lyndir.lhunath.opal.system.util.StringUtils.*;

 import com.lyndir.masterpassword.*;
 import com.lyndir.masterpassword.model.MPQuestion;
+import com.lyndir.masterpassword.model.MPSite;
 import java.util.Objects;
 import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
@@ -33,14 +34,23 @@ import org.jetbrains.annotations.NotNull;
 */
 public abstract class MPBasicQuestion extends Changeable implements MPQuestion {

-    private final String       keyword;
-    private       MPResultType type;
+    private final MPSite<?> site;
+    private final String    keyword;

-    protected MPBasicQuestion(final String keyword, final MPResultType type) {
+    private MPResultType type;
+
+    protected MPBasicQuestion(final MPSite<?> site, final String keyword, final MPResultType type) {
+        this.site = site;
        this.keyword = keyword;
        this.type = type;
    }

+    @Nonnull
+    @Override
+    public MPSite<?> getSite() {
+        return site;
+    }
+
    @Nonnull
    @Override
    public String getKeyword() {
@@ -55,7 +65,7 @@ public abstract class MPBasicQuestion extends Changeable implements MPQuestion {

    @Override
    public void setType(final MPResultType type) {
-        if (Objects.equals(this.type, type))
+        if (this.type == type)
            return;

        this.type = type;
@@ -70,15 +80,12 @@ public abstract class MPBasicQuestion extends Changeable implements MPQuestion {
        return getSite().getResult( MPKeyPurpose.Recovery, getKeyword(), null, getType(), state );
    }

-    @Nonnull
-    @Override
-    public abstract MPBasicSite<?, ?> getSite();
-
    @Override
    protected void onChanged() {
        super.onChanged();

-        getSite().setChanged();
+        if (site instanceof Changeable)
+            ((Changeable) site).setChanged();
    }

    @Override
--- a/platform-independent/java/model/src/main/java/com/lyndir/masterpassword/model/impl/MPBasicSite.java
+++ b/platform-independent/java/model/src/main/java/com/lyndir/masterpassword/model/impl/MPBasicSite.java
@@ -21,6 +21,9 @@ package com.lyndir.masterpassword.model.impl;
 import static com.lyndir.lhunath.opal.system.util.ObjectUtils.*;
 import static com.lyndir.lhunath.opal.system.util.StringUtils.*;

+import com.google.common.base.Strings;
+import com.google.common.collect.ImmutableCollection;
+import com.google.common.collect.ImmutableSortedSet;
 import com.google.common.primitives.UnsignedInteger;
 import com.lyndir.masterpassword.*;
 import com.lyndir.masterpassword.model.*;
@@ -35,9 +38,9 @@ import javax.annotation.Nullable;
 public abstract class MPBasicSite<U extends MPUser<?>, Q extends MPQuestion> extends Changeable
        implements MPSite<Q> {

-    private final Collection<Q> questions = new LinkedHashSet<>();
    private final U             user;
    private final String        siteName;
+    private final Collection<Q> questions = new LinkedHashSet<>();

    private MPAlgorithm     algorithm;
    private UnsignedInteger counter;
@@ -104,7 +107,7 @@ public abstract class MPBasicSite<U extends MPUser<?>, Q extends MPQuestion> ext

    @Override
    public void setResultType(final MPResultType resultType) {
-        if (Objects.equals( this.resultType, resultType ))
+        if (this.resultType == resultType)
            return;

        this.resultType = resultType;
@@ -119,14 +122,14 @@ public abstract class MPBasicSite<U extends MPUser<?>, Q extends MPQuestion> ext

    @Override
    public void setLoginType(@Nullable final MPResultType loginType) {
-        if (Objects.equals( this.loginType, loginType ))
+        if (this.loginType == loginType)
            return;

        this.loginType = ifNotNullElse( loginType, getAlgorithm().mpw_default_login_type() );
        setChanged();
    }

-    @Nonnull
+    @Nullable
    @Override
    public String getResult(final MPKeyPurpose keyPurpose, @Nullable final String keyContext, @Nullable final String state)
            throws MPKeyUnavailableException, MPAlgorithmException {
@@ -134,8 +137,10 @@ public abstract class MPBasicSite<U extends MPUser<?>, Q extends MPQuestion> ext
        return getResult( keyPurpose, keyContext, getCounter(), getResultType(), state );
    }

-    protected String getResult(final MPKeyPurpose keyPurpose, @Nullable final String keyContext,
-                               @Nullable final UnsignedInteger counter, final MPResultType type, @Nullable final String state)
+    @Nullable
+    @Override
+    public String getResult(final MPKeyPurpose keyPurpose, @Nullable final String keyContext,
+                            @Nullable final UnsignedInteger counter, final MPResultType type, @Nullable final String state)
            throws MPKeyUnavailableException, MPAlgorithmException {

        return getUser().getMasterKey().siteResult(
@@ -143,8 +148,10 @@ public abstract class MPBasicSite<U extends MPUser<?>, Q extends MPQuestion> ext
                keyPurpose, keyContext, type, state );
    }

-    protected String getState(final MPKeyPurpose keyPurpose, @Nullable final String keyContext,
-                              @Nullable final UnsignedInteger counter, final MPResultType type, final String state)
+    @Nonnull
+    @Override
+    public String getState(final MPKeyPurpose keyPurpose, @Nullable final String keyContext,
+                           @Nullable final UnsignedInteger counter, final MPResultType type, final String state)
            throws MPKeyUnavailableException, MPAlgorithmException {

        return getUser().getMasterKey().siteState(
@@ -160,13 +167,13 @@ public abstract class MPBasicSite<U extends MPUser<?>, Q extends MPQuestion> ext
        return getResult( MPKeyPurpose.Identification, null, null, getLoginType(), state );
    }

+    @Nonnull
    @Override
-    public boolean addQuestion(final Q question) {
-        if (!questions.add( question ))
-            return false;
+    public Q addQuestion(final Q question) {
+        questions.add( question );

        setChanged();
-        return true;
+        return question;
    }

    @Override
@@ -184,6 +191,17 @@ public abstract class MPBasicSite<U extends MPUser<?>, Q extends MPQuestion> ext
        return Collections.unmodifiableCollection( questions );
    }

+    @Nonnull
+    @Override
+    public ImmutableCollection<Q> findQuestions(@Nullable final String query) {
+        ImmutableSortedSet.Builder<Q> results = ImmutableSortedSet.naturalOrder();
+        for (final Q question : getQuestions())
+            if (Strings.isNullOrEmpty( query ) || question.getKeyword().startsWith( query ))
+                results.add( question );
+
+        return results.build();
+    }
+
    @Nonnull
    @Override
    public U getUser() {
--- a/platform-independent/java/model/src/main/java/com/lyndir/masterpassword/model/impl/MPBasicUser.java
+++ b/platform-independent/java/model/src/main/java/com/lyndir/masterpassword/model/impl/MPBasicUser.java
@@ -20,6 +20,7 @@ package com.lyndir.masterpassword.model.impl;

 import static com.lyndir.lhunath.opal.system.util.StringUtils.*;

+import com.google.common.base.Strings;
 import com.google.common.collect.ImmutableCollection;
 import com.google.common.collect.ImmutableSortedSet;
 import com.lyndir.lhunath.opal.system.CodeUtils;
@@ -202,22 +203,21 @@ public abstract class MPBasicUser<S extends MPBasicSite<?, ?>> extends Changeabl
    @Override
    public ImmutableCollection<S> findSites(@Nullable final String query) {
        ImmutableSortedSet.Builder<S> results = ImmutableSortedSet.naturalOrder();
-        if (query != null)
-            for (final S site : getSites())
-                if (site.getSiteName().startsWith( query ))
-                    results.add( site );
+        for (final S site : getSites())
+            if (Strings.isNullOrEmpty( query ) || site.getSiteName().startsWith( query ))
+                results.add( site );

        return results.build();
    }

    @Override
-    public boolean addListener(final Listener listener) {
-        return listeners.add( listener );
+    public void addListener(final Listener listener) {
+        listeners.add( listener );
    }

    @Override
-    public boolean removeListener(final Listener listener) {
-        return listeners.remove( listener );
+    public void removeListener(final Listener listener) {
+        listeners.remove( listener );
    }

    @Override
--- a/platform-independent/java/model/src/main/java/com/lyndir/masterpassword/model/impl/MPFileQuestion.java
+++ b/platform-independent/java/model/src/main/java/com/lyndir/masterpassword/model/impl/MPFileQuestion.java
@@ -30,16 +30,13 @@ import javax.annotation.Nullable;
 */
 public class MPFileQuestion extends MPBasicQuestion {

-    private final MPFileSite site;
-
    @Nullable
    private String answerState;

    public MPFileQuestion(final MPFileSite site, final String keyword,
                          @Nullable final MPResultType type, @Nullable final String answerState) {
-        super( keyword, ifNotNullElse( type, site.getAlgorithm().mpw_default_answer_type() ) );
+        super( site, keyword, ifNotNullElse( type, site.getAlgorithm().mpw_default_answer_type() ) );

-        this.site = site;
        this.answerState = answerState;
    }

@@ -48,6 +45,8 @@ public class MPFileQuestion extends MPBasicQuestion {
        return answerState;
    }

+    @Nonnull
+    @Override
    public String getAnswer()
            throws MPKeyUnavailableException, MPAlgorithmException {
        return getAnswer( answerState );
@@ -66,9 +65,8 @@ public class MPFileQuestion extends MPBasicQuestion {
        setChanged();
    }

-    @Nonnull
-    @Override
-    public MPFileSite getSite() {
-        return site;
+    public void use() {
+        if (getSite() instanceof MPFileSite)
+            ((MPFileSite) getSite()).use();
    }
 }
--- a/platform-independent/java/model/src/main/java/com/lyndir/masterpassword/model/impl/MPFileSite.java
+++ b/platform-independent/java/model/src/main/java/com/lyndir/masterpassword/model/impl/MPFileSite.java
@@ -93,7 +93,7 @@ public class MPFileSite extends MPBasicSite<MPFileUser, MPFileQuestion> {
        setChanged();
    }

-    @Nonnull
+    @Nullable
    @Override
    public String getResult(final MPKeyPurpose keyPurpose, @Nullable final String keyContext)
            throws MPKeyUnavailableException, MPAlgorithmException {
@@ -145,6 +145,12 @@ public class MPFileSite extends MPBasicSite<MPFileUser, MPFileQuestion> {
        setChanged();
    }

+    @Nonnull
+    @Override
+    public MPFileQuestion addQuestion(final String keyword) {
+        return addQuestion( new MPFileQuestion( this, keyword, null, null ) );
+    }
+
    @Override
    public int compareTo(@Nonnull final MPSite<?> o) {
        int comparison = (o instanceof MPFileSite)? ((MPFileSite) o).getLastUsed().compareTo( getLastUsed() ): 0;
--- a/platform-independent/java/model/src/main/java/com/lyndir/masterpassword/model/impl/MPFileUser.java
+++ b/platform-independent/java/model/src/main/java/com/lyndir/masterpassword/model/impl/MPFileUser.java
@@ -207,6 +207,7 @@ public class MPFileUser extends MPBasicUser<MPFileSite> {
        super.reset();
    }

+    @Nonnull
    @Override
    public MPFileSite addSite(final String siteName) {
        return addSite( new MPFileSite( this, siteName ) );
--- a/platform-independent/java/model/src/main/java/com/lyndir/masterpassword/model/impl/MPFileUserManager.java
+++ b/platform-independent/java/model/src/main/java/com/lyndir/masterpassword/model/impl/MPFileUserManager.java
@@ -66,6 +66,7 @@ public class MPFileUserManager {

    protected MPFileUserManager(final File path) {
        this.path = path;
+        reload();
    }

    public void reload() {
@@ -128,12 +129,13 @@ public class MPFileUserManager {
        return ImmutableSortedSet.copyOf( userByName.values() );
    }

-    public boolean addListener(final Listener listener) {
-        return listeners.add( listener );
+    public void addListener(final Listener listener) {
+        if (listeners.add( listener ))
+            listener.onFilesUpdated( getFiles() );
    }

-    public boolean removeListener(final Listener listener) {
-        return listeners.remove( listener );
+    public void removeListener(final Listener listener) {
+        listeners.remove( listener );
    }

    private void fireUpdated() {
@@ -141,7 +143,6 @@ public class MPFileUserManager {
            return;

        ImmutableSortedSet<MPFileUser> files = getFiles();
-
        for (final Listener listener : listeners)
            listener.onFilesUpdated( files );
    }
--- a/platform-independent/java/model/src/main/java/com/lyndir/masterpassword/model/impl/MPJSONFile.java
+++ b/platform-independent/java/model/src/main/java/com/lyndir/masterpassword/model/impl/MPJSONFile.java
@@ -170,7 +170,7 @@ public class MPJSONFile extends MPJSONAnyObject {
            if (fileSite.questions != null)
                for (final Map.Entry<String, Site.Question> questionEntry : fileSite.questions.entrySet()) {
                    Site.Question fileQuestion = questionEntry.getValue();
-                    MPFileQuestion question = new MPFileQuestion( site, questionEntry.getKey(),
+                    MPFileQuestion question = new MPFileQuestion( site, ifNotNullElse( questionEntry.getKey(), "" ),
                                                                  fileQuestion.type, export.redacted? fileQuestion.answer: null );

                    if (!export.redacted && (fileQuestion.answer != null))
--- a/public/site
+++ b/public/site
Author	SHA1	Message	Date
Maarten Billemont	39f6893742	2.7-java-7	2018-08-27 18:24:46 -04:00
Maarten Billemont	7bf7b8981c	Fix duplication of user names in files list.	2018-08-27 18:24:00 -04:00
Maarten Billemont	09abe21fed	Release masterpassword-gui-2.7.6	2018-08-27 13:22:03 -04:00
Maarten Billemont	6fae0fe425	2.7-java-6	2018-08-27 13:11:32 -04:00
Maarten Billemont	0558176847	Fix dependencies in native mpw DLLs.	2018-08-27 13:07:38 -04:00
Maarten Billemont	c553201cda	Small site update.	2018-08-26 20:41:27 -04:00
Maarten Billemont	665be9494b	Release masterpassword-gui-2.7.5	2018-08-26 18:19:01 -04:00
Maarten Billemont	5ca81b4aa7	Don't use daemon when setting release passwords in environment.	2018-08-25 11:57:36 -04:00
Maarten Billemont	3cbb063926	Refactor Native to try and load other architectures.	2018-08-24 16:33:14 -04:00
Maarten Billemont	d5551c8c8c	Key calculator and access to the full algorithm.	2018-08-24 13:48:53 -04:00
Maarten Billemont	9a40e52d53	2.7-java-5	2018-08-19 16:13:39 -04:00
Maarten Billemont	6f0d768e69	Saving custom passwords and logins.	2018-08-19 16:11:43 -04:00
Maarten Billemont	40fdc8d248	Fix initialization dependency cycle & load files on init.	2018-08-18 13:43:41 -04:00
Maarten Billemont	6b9e1b8cb8	Standard label font & fix warnings.	2018-08-14 12:06:04 -04:00
Maarten Billemont	f41cdb8742	Site security questions and copy login name.	2018-08-13 17:53:31 -04:00
Maarten Billemont	10c6d203b8	Implement security answers & immediate site lookup.	2018-08-07 00:07:16 -04:00
Maarten Billemont	7d1aa9c9f4	Release 2.7.4	2018-08-02 12:34:24 -04:00
Maarten Billemont	c26281e3b7	2.7-java-4	2018-08-02 12:20:45 -04:00
Maarten Billemont	f0b1f0c9e0	Build for older glibc.	2018-08-02 12:19:49 -04:00
Maarten Billemont	9682efc7c9	Release masterpassword-gui-2.7.3	2018-08-02 01:44:50 -04:00