salesagility / SuiteCRM-Portal-Joomla

Joomla Plugin for the SuiteCRM Portal
GNU General Public License v2.0
22 stars 21 forks source link

Updating password only on AOP settings page in Joomla doesn't do anything #28

Open mruivo opened 7 years ago

mruivo commented 7 years ago

Issue

On the Advanced Open Portal settings page on Joomla, after updating only the password field, the MD5 hash in the Joomla database does not change.

Expected Behavior

If updating password only, the password should actually update

Actual Behavior

After updating password only, nothing happens in database

Possible Fix

Steps to Reproduce

  1. Set up AOP on Joomla
  2. Look at sugar_pass field on joomla_advancedopenportal
  3. On AOP settings page, change only password field, save
  4. Go back to database, the hash will stay the same, even though it should have changed

Context

High priority. Trying to make AOP work. I did everything properly, but could not create a portal user. Wasted many hours trying to get it to work.

Your Environment

greenlanegreb commented 5 years ago

I actually have a funny feeling that this is STILL an issue. I had an issue whereby a "0" error displayed when I changed my Suite CRM password.

I tried to update the Suite CRM instance password in the Joomla Plugin and it would not do anything. When I changed my password back within Suite CRM, the portal worked again (before it through a PHP 7.1 wobbly) -

The error thrown for me then was: 0 Too few arguments to function SugarCase::__construct(), 1 passed in /var/www/clients/client1/web12/web/croydonfix/repairtool/portal/components/com_advancedopenportal/models/SugarCasesConnection.php on line 277 and exactly 2 expected" - The good news is the proposed fix on Github sorts that out but not the password issue. It may just have been bad luck to have the two issues throw at the same time.

The wider topic referencing the fix is at: https://suitecrm.com/suitecrm/forum/advanced-opensales-workflow-portal-support/14819-simple-fix-for-advanced-openportal-with-php-7-1 and the fix itself is at: https://github.com/salesagility/SuiteCRM-Portal-Joomla/pull/32/files

With the zero error, it'd be helpful to have it display some more human readable error messages though please...

To the OP, Ashley is going to take a look at this page and has kindly agreed to take some high profile bug issues and prioritise them.

greenlanegreb commented 5 years ago

Hi, are there any updates on this? Thanks ever so much.

greenlanegreb commented 4 years ago

Any updates on this please?