Rated 5 out of 5 stars

Simple and brilliant. The author also supports on http://kde-apps.org/content/show.php?content=116886 and responds to questions and comments.

This review is for a previous version of the add-on (2.2.1-signed.1-signed). 

Rated 5 out of 5 stars

I really like this add-on but I had to disable it after the FF26 upgrade. However, the 2.2 version worked once I disabled baselinejit option per Guillermo's recommendation (below).

This review is for a previous version of the add-on (2.2.1-signed.1-signed). 

It seems that the crashes experienced by the users are related to the new Javascript JIT Compiler Called ION Baseline. Disabling it seems to solve the problem. You can disable it by setting javascript.options.baselinejit.* in about:config to false

Rated 4 out of 5 stars

It has been working for me too, until upgraded to Firefox 26. Solution provided by tbali didn't work for me (Kubuntu 13.10 AMD64).

This review is for a previous version of the add-on (2.1.1-signed.1-signed). 

Rated 5 out of 5 stars

I like this addon! But after upgrade to FF 26, it crash almost every 2 minutes.
Could you try to fix it, please?

This review is for a previous version of the add-on (2.1.1-signed.1-signed). 

Rated 5 out of 5 stars

I also have crashes with firefox 26 on Ubuntu 12.10.

This review is for a previous version of the add-on (2.1.1-signed.1-signed). 

Rated 5 out of 5 stars

tbali, thanks for the fix! Will be good to see it in next version.

Also I want report that after upgrade Firefox from 25 to 26 (on Ubuntu 13.10 AMD64), I got segfault crashes very often (in crash report I see segfault 11 and KWallet plugin). Other users have this problem too?

How can I debug this error?

This review is for a previous version of the add-on (2.1.1-signed.1-signed).  This user has 15 previous reviews of this add-on.

Rated 5 out of 5 stars

I found the problem with firefox 24+. The api changed.
In the file ~/.mozilla/firefox/XXXXXXX.default/extensions/kwallet@guillermo.molina/components/KDEWalletStorage.js add the following lines:

get isLoggedIn() {
return true;
}

So it will look like this:

------------------
countLogins: function countLogins(hostname, submitURL, realm) {
this.log( "countLogins() Start" );
var count = this._lib.countLogins(hostname, submitURL, realm);
this.log( "countLogins() counted = " + count );
return count;
},
get isLoggedIn() {
return true;
}
};

const NSGetFactory = XPCOMUtils.generateNSGetFactory([KDEWalletStorage]);
------------------

Dear Guillermo Molina, please support is in the future too! Thanks for your work!

This review is for a previous version of the add-on (2.1.1-signed.1-signed). 

Rated 1 out of 5 stars

It seems the addon is dead already. It's a great pity. Switched back to firefox native pwd manager.

This review is for a previous version of the add-on (2.1.1-signed.1-signed).  This user has a previous review of this add-on.

Rated 2 out of 5 stars

Recently it started crashing so often that it became unusable.

This review is for a previous version of the add-on (2.1.1-signed.1-signed). 

Rated 4 out of 5 stars

but i can't find a way to change wallet.

This review is for a previous version of the add-on (2.1.1-signed.1-signed). 

Rated 5 out of 5 stars

As a follow up to my previous comment about firefox sync not working, I have observed that the passwords which are already present in the password manager before setting up sync, are sync'ed successfully as soon as sync is set up. However, any subsequent passwords added to the password manager don't seem to be sync'ed. Now, I am not sure whether this is a bug related to this plugin or firefox sync server (I'm running my own sync server). However, the simple fix for this is that whenever I want my passwords to be sync'ed I just select "Preferences" option, go to "Sync" tab and do a "Reset Sync ..." -> "Recommended: Merge this device's data with my Sync data" on my laptop. I then wait for a few minutes and all my passwords are sync'ed from my laptop to my sync server successfully. This then automatically gets sync'ed to my other devices where I am using Firefox.

This review is for a previous version of the add-on (2.1.1-signed.1-signed).  This user has a previous review of this add-on.

Rated 2 out of 5 stars

The addon installs gracefully and seems to works fine. As of the feature itself, I sadly think it should be part of the browser and not require an addon… But I'm very happy to see someone paying attention to this :)
Well, I'm using 2.1, the changelog mentions Sync integration, and that's great because I'm used to have a brunch of passwords saved/synchronized this way. But as soon as I enable the addon, my passwords list appears empty. Should I find a way to import/export my passwords from the former pw storage to the new one ?
Waiting a couple of minutes… Now Firefox complains with my sync credentials being wrong. Ok, let's enter my login + sync key to pair this device again.
Waiting a couple of minutes hoping for my passwords list to be filled again, being fetched from sync server… nothing happens, :/
Did I missed something ? Did I broke it and lost data ?
All this Sync integration should be made easier for the user to get in IMO…

This review is for a previous version of the add-on (2.1.1-signed.1-signed). 

Rated 5 out of 5 stars

Nice Addo that works fine with Firefox 16 (with KDE patch on Arch Linux x86_64).

PS: The homepage link in the addon don't works

This review is for a previous version of the add-on (2.0.1-signed.1-signed). 

Thank you very much

Rated 5 out of 5 stars

I want to salute you for this great extensions, but I also need to report a bug!!
Is there an open bug tracker in your site ?
I found none, tried to use the contact form, but not sure if it's working.
On my machine, red hat workstation 6.3 x64 extensions version 2 now crashes!

** (firefox:8295): DEBUG: NP_GetMIMEDescription
** (firefox:8295): DEBUG: NP_GetMIMEDescription
** (firefox:8295): DEBUG: NP_GetMIMEDescription
*** NSPlugin Viewer *** ERROR: detected overflow of 1 bytes
for block allocated at ../src/npw-viewer.c:1352
and released at ../src/npw-viewer.c:1370
NOTE: child process received `Goodbye', closing down
NOTE: child process received `Goodbye', closing down
** (firefox:8295): DEBUG: NP_GetMIMEDescription
/usr/lib64/firefox/firefox: symbol lookup error: /home/israelm/.mozilla/firefox/z09lwsl4.default/extensions/kwallet@guillermo.molina/components/libkdewallet_x86_64.so: undefined symbol: _ZN6KDebug13hasNullOutputE9QtMsgTypebib

[1]+ Exit 127 firefox

can you check this please ?
Thanks!

This review is for a previous version of the add-on (2.0.1-signed.1-signed). 

Rated 5 out of 5 stars

Very good add-on, it has just two minor issues.
The wallet is not reopened (if closed) and Firefox and Thunderbird password windows appear with the same message.

I managed to fix both problems in my forked repository:
https://github.com/maciex/mozilla-kwallet

Guillermo please merge the changes and/or contact me (Support site link for the add-on is not working).
Thanks !

This review is for a previous version of the add-on (1.14). 

Which distro are you using? in ubuntu libxul is no longer available.
Cheers

Rated 5 out of 5 stars

One of the main needed Firefox Add-Ons if someone uses KDE!
Many thanks to Guillermo for this nice and helpful tool and I would spend some money to him - but I never create a PayPal account while discontinuing the license agreements.

This review is for a previous version of the add-on (1.13).  This user has a previous review of this add-on.

Rated 3 out of 5 stars

Hi Guillermo,

Thanks for updating that module for the newest version of Firefox. You rock :D. Still think this is a must have module for all KDE users.

But, i've noticed few bugs with it that I didn't have with previous versions of this module. Don't know if anybody else has the same problems.

1 - For some reasons, it doesn't store all passwords. When the module is disabled (for example on this website "https://addons.mozilla.org/", Firefox asks to save the password, but with the kwallet module enabled it doesn't). There are many websites like this one.

2 - Also, I noticed that randomly, after using Firefox for a while, it doesn't fill automatically the fields for my usernames/passwords (but the credentials are correctly saved into Kwallet and the wallet is open). So I have to fill the fields myself as if nothing were saved.

Did anybody else noticed these bugs? or I am the only one?

This review is for a previous version of the add-on (1.0.10).  This user has a previous review of this add-on.

For number 1, it works for me, sorry about that
Number 1 and 2, you can debug it yourself, use:
$ NSPR_LOG_MODULES=KDEWalletLog:5 firefox
and take a look at the output. Sometimes the password backend never asks the plugin for a password, I don't know why, sorry about that.

Rated 5 out of 5 stars

Good Addon! It really works.

This review is for a previous version of the add-on (1.0.10). 

Rated 5 out of 5 stars

Since the upgrade to Firefox 10 this wonderful extension no longer works. I tried to tweak maxVersion in install.rdf without success. Any idea how I can make it working again? Will there be an update for Firefox 10? Thanks

Update: I downloaded the sources. The recompile showed that the firefox 10 interfaces changed. Adjusting types did the job. The addon works again. Here is the diff:

--- kwallet@guillermo.molina/src/KDEWallet.cpp 2012-01-09 09:21:59.000000000 +0100
+++ /home/user/make/kwallet@guillermo.molina/src/KDEWallet.cpp 2012-02-05 14:36:40.000000000 +0100
@@ -230,7 +230,7 @@
return Init();
}

-NS_IMETHODIMP KDEWallet::GetUiBusy(int *) {
+NS_IMETHODIMP KDEWallet::GetUiBusy(bool *) {
PR_LOG( gKDEWalletLog, PR_LOG_DEBUG, ( "KDEWallet::GetUiBusy() Called") );
return NS_OK;
}
@@ -370,7 +370,7 @@
nsCOMPtr<nsISupports> sup;
nsCOMPtr<nsIProperty> prop;
nsAutoString propName;
- PRBool hasMoreElements;
+ bool hasMoreElements;

rv = enumerator->HasMoreElements(&hasMoreElements);
NS_ENSURE_SUCCESS(rv, rv);
@@ -615,7 +615,7 @@
nsCOMPtr<nsISupports> sup;
nsCOMPtr<nsIProperty> prop;
nsAutoString propName;
- PRBool hasMoreElements;
+ bool hasMoreElements;

rv = enumerator->HasMoreElements(&hasMoreElements);
NS_ENSURE_SUCCESS(rv, rv);
@@ -693,7 +693,7 @@
}

NS_IMETHODIMP KDEWallet::GetLoginSavingEnabled(const nsAString & aHost,
- PRBool *_retval) {
+ bool *_retval) {
PR_LOG( gKDEWalletLog, PR_LOG_DEBUG, ( "KDEWallet::GetLoginSavingEnabled() Called") );

nsresult res = checkWallet();
@@ -716,7 +716,7 @@
}

NS_IMETHODIMP KDEWallet::SetLoginSavingEnabled(const nsAString & aHost,
- PRBool isEnabled) {
+ bool isEnabled) {
PR_LOG( gKDEWalletLog, PR_LOG_DEBUG, ( "KDEWallet::SetLoginSavingEnabled() Called") );

nsresult res = checkWallet();

This review is for a previous version of the add-on (1.0.9). 

Hi, I wish it was so easy, this plugin must be recompiled for each new version of firefox for both platforms (x64 and i586)... It's a mozilla policy. After I upload the new version, It has to go through a revision process by mozilla stuff, that's why a new version is always available earlier at my web site. (http://www.guillermomolina.com.ar/index.php/en/files/category/1-firefox-kwallet-extension). Sorry

Rated 5 out of 5 stars

Hi Guillermo,

Thank you for this great extension...

I've translated it in French (very small task, but it's done now ;-) ). How (and in which format) can I send it to you ?

Sorry for my so poor english...

This review is for a previous version of the add-on (1.0.8). 

Thankyou, I sent you an email with instructions