[Plugin: Better WP Security] File change notification sends 2-3 emails

p.s. I found “@header(‘HTTP/1.0 418 I\’m a teapot’);” in there

… which is pretty cool!

(I also added this to my plugin a while ago) (see: https://plugins.svn.www.ads-software.com/wp-favicons/trunk/includes/server/plugins/request/request_cache.php in the dofilter method)

hi

i thnik i have found where the code is , i dont see the comment feature you mention, my code looks different to yours (from what i can see)

* Email report
*
* Sends a report to site admin email address if changes have been detected
*
**/
function fileemail() {
global $logid, $bwpsoptions;

//Get the right email address.
if ( is_email( $bwpsoptions[‘id_fileemailaddress’] ) ) {

$toaddress = $bwpsoptions[‘id_fileemailaddress’];

} else {

$toaddress = get_site_option( ‘admin_email’ );

}

@art4life: dont you have an editor with Line Numbers next to them?

You are now showing the fileemail() method , so now search for the place where this method is called (via $this->fileemail(); ) , which is on line 252, right above the stuff you pasted above.

So … where you see $this->fileemail(); it actually calls the mail function. If you comment that line out so with // (// = comment in PHP) then it wont call that one.

dont you have an editor with Line Numbers next to them? – no i dont have this in my editor

using lastest Firefox browers on a mac.

i have to read each line and count them

@art4life : you are not serious are you?

Why don’t you download an editor e.g. Eclipse PDT to edit your php files? Makes it much easier incl. syntax highlightning.

i see this above the previous code

im wondering if this will also cure the problem of the lock out ?

do you charge for this service as i dont mind paying for your time , im getting in over my head a bit now , im not a programmer at all.Ive spent many hours on this now.

developer has gone very quiet…..

//set latest check time
$bwpsoptions[‘id_filechecktime’] = time();

//Update the right options
if ( is_multisite() ) {

switch_to_blog( 1 );

update_option( $this->primarysettings, $bwpsoptions );

restore_current_blog();

you are not serious are you?

yes 100% , real headache counting 250 lines of code ??

@art4life “do you charge for this service” … ? not for php scripting GRIN

SCROLL somewhat more up you are now at line 277 so you have to scroll up 25 lines more to find:

if ( $bwpsoptions['id_fileemailnotify'] == 1 ) {
						$this->fileemail();
					}

p.s. : i would not do any more wordpress projects if I were you, if you don’t even have an editor?? (this probably sounds crude but typing and reading text on a forum is different than hearing it with a different tone)

also again: commenting this out will only prevent this from being mailed it does not prevent anything else like database writes, then again… in this case probably the furthest we can get.

do you charge for this service as i dont mind paying for your time

Art4life, please be aware that when someone offers to pay, those threads are closed.

https://codex.www.ads-software.com/Forum_Welcome#Offering_to_Pay

Per that rule, I’m closing this thread.

If you need paid help (and there is nothing wrong with that) please consider posting to https://jobs.wordpress.net/ instead.