cogmios
Forum Replies Created
-
I think that it does not receive a correct reply from the server and therefore somehow stops there. Do you have any more information? When you view the source of that page what does it show as “the last line” ?
@djavet maybe its an idea to first disable all of them to see which one conflicts instead bit51 having to run through all of them?
sure, yes, however im quite busy now on a project so this will probably only be the end of august before i give the feedback.
i have put in my @todo list.
Hi Bit51,
Sorry might be confusing, there are 2 notices and 1 undesired behaviour, they are unrelated and seperated issues:
Notice 1 =
Notice: Undefined property: log_content_6_table::$hook in /wp-content/plugins/better-wp-security/inc/admin/tables.php on line 520 N/ANotice 2 =
Trying to get property of non-object in /.../wp-content/plugins/better-wp-security/inc/secure.php on line 791And 3rd the undesired behaviour is:
a) you set a custom login url e.g. “/whateverloginlanding”
b) you active it and add the stuff to nginx.conf / nginx stop / nginx start
c) you go to /login and it will display the login page, i believe this is because of the canonical implementation in WP.+ if a user has already changed the prefix of the database during the install then grey out the option
@esmi : ok thanks , i will contact the dev to set it to unresolved
This has been marked as “resolved” but it is not , I also experienec it.
Is there a MOD who can change it back to unresolved?
@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.
@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.
@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.
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: http://plugins.svn.wordpress.org/wp-favicons/trunk/includes/server/plugins/request/request_cache.php in the dofilter method)
@art4life: just replace :
if ( $bwpsoptions['id_fileemailnotify'] == 1 ) { $this->fileemail(); }with
if ( $bwpsoptions['id_fileemailnotify'] == 1 ) { //$this->fileemail(); }(so place // (comment) in front of call to the email function/method )
again: this will still fill up the log with these messages but at least it wont send them out.
@art4life : there are several places where the plugin sends out mails. I wonder if it would not have been nicer to use a central mail class so it would be easier to have “send no mail” or have a filter on outgoing mails.
I think this would a good suggestion for the dev as an improvement.
@art4life : in the plugin of course GRIN …
so /plugins/better-wp-security/inc/filecheck.php
Maybe just read through all the php code of the better-wp-security plugin to get a feeling for what it does?
I’m also wondering about ‘starttime’ => $currtime in the plugin. I have not checked if it compares the stored time with a time from the plugin, I usually get the time from mysql:
try { // takes about 0.2435 seconds $sql = "SELECT UPDATE_TIME, NOW() FROM information_schema.TABLES WHERE <code>TABLE_NAME</code> = '$table_name_2';"; $row_uri = $wpdb->get_results( $sql, ARRAY_A); } catch (Exception $e) { Log::F($debug,'Could not Get Response ' . $e->getMessage()); }and then (if it is not null):
$latest_timestamp_in_database = date_create($row_uri[0]["UPDATE_TIME"]); $latest_timestamp_in_database = $latest_timestamp_in_database->getTimeStamp(); $current_date_timestamp_in_database = date_create($row_uri[0]["NOW()"]); $current_date_timestamp = $current_date_timestamp_in_database->getTimestamp(); $difference = $current_date_timestamp - $latest_timestamp_in_database;Disadvantage is that the user needs to have read access to the information schema
What I mean to say is : could be used to prevent in all possible cases sending multiple mails out, if after a send mail the send mail was hashed and logged with its timestamp in the database (just to be sure).
So whatever happens as long as this database check can take place it will not send out multiple mails if we store the send mails in the db.