acis

About me

Developer Information
Name acis
User since Feb. 5, 2012
Number of add-ons developed 0 add-ons
Average rating of developer's add-ons Not yet rated

My Reviews

KDE Wallet password integration

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).