wondering if you have any thoughts on what to do when flash content comes up in search results. its a bit annoying. check the attachment. im going to look into just outputting the node title but it'll be don't know if that'll be just a drupal fix for my site only or if site engines like google are reading everything as well.

CommentFileSizeAuthor
searchresults.gif7.45 KBaterchin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

douggreen’s picture

douggreen CreditAttribution: douggreen commented

This is really an issue with the search engine and we should filter these out.

But the solution is to put all your script code inside an HTML comment. This is what we had to do in the 1990's when some browsers didn't support script. And I think it's still good practice.

<script><!-- --></script>

See #257196

aterchin’s picture

aterchin CreditAttribution: aterchin commented

doug: thanks. i still don't know what i'm doing though.
wrapping the $content variable in: 

<div class="content">
	  <script><!--
		<?php print $content; ?>		
	  --></script>
</div>

isn't going to output my player.

the player should be the only thing in $content. everything else is taken care of with cck and output in panels2. so....

douggreen’s picture

douggreen CreditAttribution: douggreen commented
Project: FLV Media Player » SWFObject API
Version: 5.x-0.11 » 5.x-2.0-beta1
Component: Miscellaneous » Code
Category: support » bug

I think that the bug is in swfobject_api.module on line 158, they need to put the javascript inside a comment.

This also seems to be a bug in Drupal core, and I'm addressing that in #257196.

arthurf’s picture

arthurf CreditAttribution: arthurf commented

I'll take care of this and roll a new release

douggreen’s picture

douggreen CreditAttribution: douggreen commented

Please follow the other thread and make sure that this works, before rolling it in here. Thanks!

douggreen’s picture

douggreen CreditAttribution: douggreen commented

OK, sorry, ignore comment #5, that is the solution. #257196 is now CNR.

arthurf’s picture

arthurf CreditAttribution: arthurf commented

I've added a fix to DRUPAL-5 version of swfobject- can you give that a try and see if that fixes your problem?

aterchin’s picture

aterchin CreditAttribution: aterchin commented

yeah, dl'ed the imageapi 5.x-1.x-dev release. seems to be working fine now when i do a search. the js code is gone. thanks.

the video doesnt show up though with this version, so i'll still use swfobject_api-5.x-1.2-15.tar.gz for now.

arthurf’s picture

arthurf CreditAttribution: arthurf commented
Status: Active » Closed (fixed)

This should be resolved with the new approach to rendering the player