Phocagallery plugins

Phoca Gallery - image gallery extension
phproberto
Phoca Newbie
Phoca Newbie
Posts: 1
Joined: 05 Jul 2012, 14:17

Phocagallery plugins

Post by phproberto »

Hi,

first of all thank you for this great component!

We are developing an integration betweeen phocagallery and hikashop. We need to make some actions before / after a image has been saved. Digging into the code I saw that there is some commented code in administrator/components/com_phocagallery/models/phocagalleryimg.php that triggers some plugin actions. For example:

Code: Select all

		// Trigger the onContentBeforeSave event.
		/*$result = $dispatcher->trigger($this->event_before_save, array($this->option.'.'.$this->name, $table, $isNew));
		if (in_array(false, $result, true)) {
			$this->setError($table->getError());
			return false;
		}*/ 
Are you going to implement it on next releases? As we need it now we are going to add some of them. Do you have any publi repo to commit our changes? We would like to share our improvements to contribute to the component.

Thank you in advance.

Roberto Segura.
User avatar
Jan
Phoca Hero
Phoca Hero
Posts: 48694
Joined: 10 Nov 2007, 18:23
Location: Czech Republic
Contact:

Re: Phocagallery plugins

Post by Jan »

Hi, this was removed (commented) because it did conflicts with other extensions :-( :-(
If you find Phoca extensions useful, please support the project
Silwing
Phoca Newbie
Phoca Newbie
Posts: 1
Joined: 25 Apr 2013, 15:08

Re: Phocagallery plugins

Post by Silwing »

That is quite sad, such conflicts exists only because of badly written extensions not implementing proper checking of context... Have you considered implementing separate events for before and after save, instead of using the default content events? That should rule out any conflicts.
User avatar
Jan
Phoca Hero
Phoca Hero
Posts: 48694
Joined: 10 Nov 2007, 18:23
Location: Czech Republic
Contact:

Re: Phocagallery plugins

Post by Jan »

Hi, I have tested it of course with "own names" but still some extensions had have problems there - it is still under review, so I think, maybe some solution will be found for this issue :idea:

Jan
If you find Phoca extensions useful, please support the project
Post Reply