All Videos in FLEXIcontent version 2.0.3 beta r1685

bau65
Topic Author
Offline
Junior Member

12 years 9 months ago #37311 by bau65

All Videos in FLEXIcontent version 2.0.3 beta r1685 was created by bau65

Hi to all,
i've install and enable All videos plugin, so I've made a text field as follow:
Trigger content plugins YES
Specific plugins only selected jw_allvideos
Remove space YES
Prefix text of every value {youtube}
Suffix text of every value {/youtube}

So if I put in the field the ID video e.g. VJGCb1qYzuo in the front end I see {youtube}VJGCb1qYzuo{/youtube} instead the player and I've the following errors

Warning: explode() expects parameter 2 to be string, array given in /home/conca/public_html/components/com_flexicontent/classes/flexicontent.fields.php on line 562

Warning: Invalid argument supplied for foreach() in /home/conca/public_html/components/com_flexicontent/classes/flexicontent.fields.php on line 568

Whath I do wrong?

Please Log in or Create an account to join the conversation.

ggppdk
Offline
Administrator

12 years 9 months ago #37316 by ggppdk

Replied by ggppdk on topic All Videos in FLEXIcontent version 2.0.3 beta r1685

Hello,
i will commit a fix (will make a new beta package tomorrow),

this is the fix to test it:
replace lines 561-567 (the if-elseif-else statement) with:

Code:

$_plgs = $field->parameters->get('plugins');
$_plgs = $_plgs ? $_plgs : array();
$_plgs = is_array($_plgs) ? $_plgs : explode('|', $_plgs);

-- Flexicontent is Free but involves a big effort on our part.
Like the our support? (for a bug-free FC, despite having a long list of functions) Like the features? Like the ongoing development and future commitment to FLEXIcontent?
-- Add your voice to the FLEXIcontent JED listing with a 5-star review. Thanks!

Please Log in or Create an account to join the conversation.

bau65
Topic Author
Offline
Junior Member

12 years 9 months ago #37322 by bau65

Replied by bau65 on topic All Videos in FLEXIcontent version 2.0.3 beta r1685

Hi to George,
at the moment I solved with embed code in a text field for 2 reason:
because the All videos plugin don't work in the field but also in the articles so is probably there is a conflict with other component, I've try with another plugin but is the same
With embed code is possible include not only YouTube but any other video a.g. Vimeo
Of course I'll update Felxicontent when will be ready and try again with All Videos plugin:)

Please Log in or Create an account to join the conversation.

micker
Offline
Administrator

12 years 9 months ago #37324 by micker

Replied by micker on topic All Videos in FLEXIcontent version 2.0.3 beta r1685

if you fin this plugin (all video include)
share it !

FLEXIcontent is Free but involves a very big effort on our part.
Like the our support? (for a bug-free FC, despite being huge extension) Like the features? Like the ongoing development and future commitment to FLEXIcontent?
-- Add your voice to the FLEXIcontent JED listing reviews. Thanks![/size]

Please Log in or Create an account to join the conversation.

bau65
Topic Author
Offline
Junior Member

12 years 8 months ago #37357 by bau65

Replied by bau65 on topic All Videos in FLEXIcontent version 2.0.3 beta r1685

Sorry?

Please Log in or Create an account to join the conversation.

ggppdk
Offline
Administrator

12 years 8 months ago #37358 by ggppdk

Replied by ggppdk on topic All Videos in FLEXIcontent version 2.0.3 beta r1685

bau65 wrote: Hi to George,
at the moment I solved with embed code in a text field for 2 reason:
because the All videos plugin don't work in the field but also in the articles so is probably there is a conflict with other component, I've try with another plugin but is the same
With embed code is possible include not only YouTube but any other video a.g. Vimeo
Of course I'll update Felxicontent when will be ready and try again with All Videos plugin:)

Regardless of any conflict with any other extension, the fix i posted above (for v2.0.3 beta1) is needed to fix content plugin triggering in fields

Please Log in or Create an account to join the conversation.

bau65
Topic Author
Offline
Junior Member

12 years 8 months ago #37359 by bau65

Replied by bau65 on topic All Videos in FLEXIcontent version 2.0.3 beta r1685

Yes, yes, I'll replace lines 561-567

Thanks

Please Log in or Create an account to join the conversation.

bau65
Topic Author
Offline
Junior Member

12 years 8 months ago #37372 by bau65

Replied by bau65 on topic All Videos in FLEXIcontent version 2.0.3 beta r1685

I've change the line 561 - 567 of flexicontent.fields.php as follow:

Code:

if (FLEXI_J16GE) {
					$_plgs = $field->parameters->get('plugins');
				} else if $_plgs = $_plgs ? $_plgs : array(); {
					$_plgs = array($field->parameters->get('plugins'));
				} else {
					$_plgs = is_array($_plgs) ? $_plgs : explode('|', $_plgs);
				}

It's ok?

Please Log in or Create an account to join the conversation.

ggppdk
Offline
Administrator

12 years 8 months ago #37373 by ggppdk

Replied by ggppdk on topic All Videos in FLEXIcontent version 2.0.3 beta r1685

Hello,
no , you need to remove the if-elseif-else statement completly and replace it with the above given lines

Please Log in or Create an account to join the conversation.

bau65
Topic Author
Offline
Junior Member

12 years 8 months ago #37374 by bau65

Replied by bau65 on topic All Videos in FLEXIcontent version 2.0.3 beta r1685

Hi, is right now?

Code:

$_plgs = null;
				if (!@$_plgs_loaded['__ALL__']) {
					JPluginHelper::importPlugin('content', $plugin = null, $autocreate = true, $dispatcher);
					$_plgs_loaded['__ALL__'] = 1;
				}
			}
			
			    $_plgs = $field->parameters->get('plugins');
    $_plgs = $_plgs ? $_plgs : array();
    $_plgs = is_array($_plgs) ? $_plgs : explode('|', $_plgs);
		
		// Suppress some plugins from triggering for compatibility reasons, e.g.
		// (a) jcomments, jom_comment_bot plugins, because we will get comments HTML manually inside the template files
		$suppress_arr = array('jcomments', 'jom_comment_bot');
		FLEXIUtilities::suppressPlugins($suppress_arr, 'suppress' );

Please Log in or Create an account to join the conversation.