jondaley
Forum Replies Created
-
We aren’t using a mobile plugin (yet). All I am trying to do now is get my-calendar to not break for desktop users.
I don’t know what the my-calendar desktop version looks like on a mobile device, it might not be that useful, but I’m more concerned about the desktop users currently.
Yes, supercache is a pain sometimes. There is a way for your plugin to register with supercache, or otherwise tell supercache that you are serving up mobile content, so then it caches properly. But I’ve never done that myself, so I don’t know how tricky it is.
Forum: Plugins
In reply to: [Post Ratings] [Plugin: Post Ratings] Numbers on left sideWorks for me as well. Any comment from the plugin author?
Yeah, it looks like the plugin has been hacked or something.
The “headtwitter” function can be removed easily enough, but who knows if this plugin is doing what it is supposed to do.
The twitter.swf file it is trying to load isn’t available.
I’d think that “missing an email or two” from your paying customers would be a pretty bad thing. I know I couldn’t get away with that with my customers at least.
I only needed an importing plugin for a one-time import from an excel spreadsheet.
I didn’t say I was pestering you since then. I bought your plugin, saw how it had a bunch of bugs in it, emailed you, didn’t get a response, worked on editing your plugin to make it work, gave up, and then re-searched for a CSV importer, and found
https://www.ads-software.com/extend/plugins/easy-csv-importer-2/
And off I went, happy as a clam, though $30 poorer. I figured since you didn’t respond to support questions, you definitely wouldn’t respond to refund questions, and figured it wasn’t worth my time to reverse the charges, etc.
Marking as resolved.
Oh – I forgot to reply to your last sentence. I definitely will not be contacting you later for your new plugins. I already wasted $30 on your first plugin that was junk.
So by “support” you mean no email to my requests after I sent you money, and then three months later you reply to say “yes there is support”?
I would call a three month’s response time incredibly bad for a free plugin, and completely unacceptable for someone who says he supports his plugin full-time, and charges for it.
Fortunately, everyone else can see the time stamps on these emails, and see that you didn’t respond for a long time and know that it isn’t worth paying you any money.
The errors weren’t in your code, so I fixed the errors in my code – they looked like the sort of errors that come from a php upgrade, and I hadn’t noticed the errors in the error log before. (so, thanks for making me see those errors…)
And thanks for uploading the new version so quickly.
I’m getting a really weird error that I can’t figure out what the problem is.
As soon as I enable the “make pages anonymous” check box, I get a 500 error, and nothing is reported in the apache logs, which I’ve never seen before. I see the 500 error in the access.log, but nothing in error.log, and nothing on the website, even if I turn on display_errors.
Just content-length: 0.
I tried replacing the “true” in the apply_filters call with $notused, and setting $notused to ” prior to calling apply_filters, but that didn’t help (I was thinking that maybe since the function expects to be able to modify that parameter, maybe you aren’t allowed to pass in such base constructs like a boolean, though I see you do that one other in a filter.
I removed the if() statement, and just had a line that said
apply_filters('...', true);
and I can reliably generate the 500 error by uncommenting that line, and fixing it by commenting. I am at a loss as to what is happening, and I’m working on a semi-live/beta site, so I’m trying to not have it down for too much.
I did write a test.php that includes wp-config.php and called it from the command line, and don’t see any problems, and I’ve since removed all of the code that I wrote: add_filter(), and the associated remove_cookies function that was called as a filter.
I’m not sure where to go from here. It seems like it is a syntax error in the apply_filter line, though I don’t know why the php engine isn’t outputting an error. Maybe I’ll try manually retyping those couple lines, if there is a funny character in it or something.
Thanks for getting back to me so quickly – since it was mostly working without the feature, I was no longer in super-critical mode, but getting back to it now.
I’m checking out your changes now, and I’m happy to see the change from HTTP_HOST to SERVER_NAME, as I occasionally got weird cached files due to that (mostly due to case-sensitivity problems). I see the stuff about multi-site, and so in the places that you still use HTTP_HOST, you might considering running a strtolower() on it.
I’ll check out the filter feature shortly.
matches, though you are right “$output =” could be removed – that is just an int saying whether it matched or not.
I suppose the code could be made better like:
$ret = preg_match_all('/<img.+src=[\'"]([^\'"]+)[\'"].*>/i', get_the_content(), $matches); if($ret && isset($matches[1][0])) $sThumbUrl = $matches[1][0];
(I wrote that code here in this comment box, but it should be straightforward enough that I typed it correctly [insert famous last words])
I actually didn’t type that block on my own – it was pasted from somewhere. preg_match would probably be more efficient, since we only want one.
I could rewrite that part and test it for you if you wanted. My guess is that it would just be this:
$ret = preg_match('/<img.+src=[\'"]([^\'"]+)[\'"].*>/i', get_the_content(), $matches); if($ret && isset($matches[1])) $sThumbUrl = $matches[1];
But, if it is there, someone can just erase it (or add to it), right?
I don’t think there is a downside of including it. Unless you are saying that site owners want to be able to customize that text?
And here is how I did it so I don’t have to edit the plugin code directly (note the #body hack is to make the CSS more specific than the plugin’s own CSS)
#body #youtube-sidebar-widget ul li div.play_arrow {
float: none;
}Ah, just removing the float: left from the play_arrow class fixes it. Negative margins with floats aren’t as well supported as just negative margins.