Notice

The forum is in read only mode.

Support Forum

Welcome! Support Forums have been reactivated
Welcome the Technical Support section. Help us in assisting you by providing us with a concise and descriptive elaboration of your issues. Be specific and if possible, provide us with a step-by-step instruction in replicating your problem.

Bug in upload photo via front status box

9 years 9 months ago
  • Michael's Avatar
    Topic Author
  • Michael
  • Offline
  • Fresh Boarder
  • Posts: 36
  • Thank you received: 0
Licenses:
JomSocial Expired

Hi,
I bought a third party plugin from www.extly.com/ which puts a watermark on all pictures uploaded but it does not work when I upload pictures via the status box on the front end.
I contacted Extly support and they wrote reported this after examining:

We've checked JomSocial source code. It's only calling the trigger for the last uploaded photo of the set. It's a bug.
To be specific:

components/com_community/controllers/system.php, line 1097
foreach ($photoIds as $key => $photoId) {
..
 $photo->store();
}

// Trigger onPhotoCreate
... 
$apps->triggerEvent('onPhotoCreate', $params);

The call to onPhotoCreate is out of the Photo iterarion.
To solve the issue, you can report the bug and ask for fix. Meanwhile, you can just manually fix the code and wait for the final patched version.


So now I am asking JS support for a fix to correct this, please :)
Thanks.

9 years 9 months ago
  • Dimas Tekad Santosa's Avatar
  • Dimas Tekad Santosa
  • Visitor
  • Thank you received: 0
Licenses:

Hi Mike,

Thank you for reporting, I already added this to our bug list..

thank you

9 years 9 months ago
  • Michael's Avatar
    Topic Author
  • Michael
  • Offline
  • Fresh Boarder
  • Posts: 36
  • Thank you received: 0
Licenses:
JomSocial Expired

Hi Dimas,
Good to hear.
Do you already have a fix for this I can apply until this is corrected in a final release?

Thanks

9 years 9 months ago
  • Dimas Tekad Santosa's Avatar
  • Dimas Tekad Santosa
  • Visitor
  • Thank you received: 0
Licenses:

Hi,

I will let you know once this issue fixed.

thank you

9 years 8 months ago
  • Dimas Tekad Santosa's Avatar
  • Dimas Tekad Santosa
  • Visitor
  • Thank you received: 0
Licenses:

Please you download this file www.dropbox.com/s/504tea4ujw84g1q/system.php?dl=0 and copy to components/com_community/controllers/

note to the 3rd party : the trigger is an array of images object.

9 years 8 months ago
  • Michael's Avatar
    Topic Author
  • Michael
  • Offline
  • Fresh Boarder
  • Posts: 36
  • Thank you received: 0
Licenses:
JomSocial Expired

Aweome. Thanks Dimas.
I will let 3rd party now about this.

9 years 8 months ago
  • Michael's Avatar
    Topic Author
  • Michael
  • Offline
  • Fresh Boarder
  • Posts: 36
  • Thank you received: 0
Licenses:
JomSocial Expired

Well, this was a short happiness ;)
After testing negativ and talking with Extly again I got this from them:

Well, JS are nearer to fix the issue. However, they have introduced a new bug.
Instead of storing N photos in an array, they have stored N references to the latest photo in the array:

$params[] = $photo;
Instead of:
$params[] = clone $photo;
Please, ask them to fix it, we have our plugin update ready.

Thanks,
Michael

9 years 8 months ago
  • Dimas Tekad Santosa's Avatar
  • Dimas Tekad Santosa
  • Visitor
  • Thank you received: 0
Licenses:

Hi Michael,

Ok I will ask to the developer again..

thank you

9 years 8 months ago
  • Dimas Tekad Santosa's Avatar
  • Dimas Tekad Santosa
  • Visitor
  • Thank you received: 0
Licenses:

Hi Michael,

Can you try to full upgrading your jomsocial to the latest version 3.2.1.4, please? our developer said he didint use "references" method.. I hope it will be fixed after you do full upgrading.

thank you

9 years 8 months ago
  • Dimas Tekad Santosa's Avatar
  • Dimas Tekad Santosa
  • Visitor
  • Thank you received: 0
Licenses:

some update from our dev, please you re apply this patch www.dropbox.com/s/504tea4ujw84g1q/system.php?dl=0

thank you

9 years 8 months ago
  • Michael's Avatar
    Topic Author
  • Michael
  • Offline
  • Fresh Boarder
  • Posts: 36
  • Thank you received: 0
Licenses:
JomSocial Expired

Hi Dimas,
Thank you for your very fast solution. I will test this together with Extly.

Michael

9 years 8 months ago
  • Michael's Avatar
    Topic Author
  • Michael
  • Offline
  • Fresh Boarder
  • Posts: 36
  • Thank you received: 0
Licenses:
JomSocial Expired

Hi again,
I have upgraded to 3.2.1.4 and uploaded the new system.php file.
Now when I upload pictures via the status box now, nothing is saved. It looks like it is uploaded but I cannot see any pictures.

Thanks,
Michael

9 years 8 months ago
  • Dimas Tekad Santosa's Avatar
  • Dimas Tekad Santosa
  • Visitor
  • Thank you received: 0
Licenses:

Hi Michael,

how about if you revert that code? I think you applied it wrong.. I suggest you wait newest package from us, which will be released soon.. now newest package under the automation test.

thank you

9 years 8 months ago
  • Michael's Avatar
    Topic Author
  • Michael
  • Offline
  • Fresh Boarder
  • Posts: 36
  • Thank you received: 0
Licenses:
JomSocial Expired

Hi Dimas,
I think that is a good solution :)

I will await your next release.
Thanks

Moderators: Piotr Garasiński
Powered by Kunena Forum

Join 180,000 websites creating Amazing communities

JomSocial is the most complete, easy-to-use addon that turns Joomla CMS into a
full -fledged, social networking site

TRY NOW BUY NOW