redwallhp

Go ahead and keep the trunk. Just be sure to run the updater once I release the “official” patch.

@ckohrman Do you want to try out the trunk version to make sure I got it right? https://downloads.www.ads-software.com/plugin/wp125.zip

It’s probably fixable. I will work on a patch soon.

Wait…how many *active* ads do you have right now? Do you not have any? Because that might be the source of the problem. If you add one, does the error go away?

Hmm. My test box is running 5.2.5. We both have 5.2.x, so I don’t think the version itself is the problem…

Well, I’ll look into it… It’s strange though that some people get the error, and others don’t. Unfortunately, I’m the latter group.

My guess is some PHP configurations respond differently to some part of the plugin source.

What version of PHP are you running?

@edyrio and @ckohrman: Try deleting the plugin directory and re-uploading the files.

Do you have a link where I can see it in action?

Well, I’m flying blind here. I can’t reproduce the error myself, so I made some educated guesses and committed a 1.3.2 release that will *hopefully* fix it.

I just tagged release 1.3.1. It should be available in 15 minutes or so.

Not quite a typo, but close enough. Forgot to account for the possibility that the wp125_write_ads() functions probably wouldn’t have a new optional argument passed to it 99% of the time… ??

I will look into it.

It will be fixed in version 1.3.0, which should be released any minute now.

The current version of WP125 pulls the “ad name” field and displays it in the alt attribute. So in a way, there already is such a function.

Open wp125.php and fine the line

define(“MANAGEMENT_PERMISSION”, “edit_themes”);

Change “edit_themes” to the proper roll. You probably want “edit_others_posts”

It’s probably because your theme has defined a class of .even somewhere. If you add to your CSS

#wp125adwrap_2c .even { border:0; }

you should be able to reverse the effects. I use border as an example, but your CSS may vary.