Larry Lewis

OK after a lot of experimenting add this to a new file in the directory;

/wp-content/plugins/simple-ajax-chat/.htaccess

<FilesMatch "\.(?i:php)$">
  <IfModule !mod_authz_core.c>
    Order deny,allow
    Allow from all
  </IfModule>
  <IfModule mod_authz_core.c>
    Require all granted
  </IfModule>
</FilesMatch>

That will override the security widget, tested on my local test system.

Note: Will fail if debugging is enabled

Fixed, at the top of wp-cron.php it is sending headers which then causes the session setting to fail….

<?php // Simple Ajax Chat > Chat Form

if (!defined('ABSPATH')) exit;

if (sac_is_session_started() === false) {
	if( defined( 'DOING_CRON' ) AND DOING_CRON === TRUE )
	{
		return;
	}
	
	if (session_status() === PHP_SESSION_NONE) {
		
		$simple_ajax_chat_domain = (isset($_SERVER['HTTP_HOST']) && !empty($_SERVER['HTTP_HOST'])) ? sanitize_text_field($_SERVER['HTTP_HOST']) : null;
		
		if ($simple_ajax_chat_domain) {
			session_set_cookie_params('21600', '/', $simple_ajax_chat_domain, false, true);
			
			session_start(array('read_and_close' => true));
			
		}
		
	}
	
}

v8.2 … it is not cli as that was there when I did it from command line as a test.

It is hitting this warning every cron call, which is invoked as;

*/15 * * * * wget --delete-after https://jenolan.org/wp-cron.php >/dev/null 2>&1

Just ran it manually and it threw the warning hmmmm I will add some code to your function to see if I can work out why it is triggering ??

I would hazard a guess it is happening inside cron call, I read somewhere that current php you can not use reliably(line 25);

if (php_sapi_name() !== 'cli') {

trying to find the explanation, will post when/if found

Pushed the small changes and update to being compatible with WP 5.5, plan on doing a small spring clean of the code then see about adding the top line of years (as an option).

I like this plugin because it is free with none of these ubiquitous ‘pro’ stuff, really tired of finding plugins that say they will do something but then find that most of it is ‘pro’ with ‘yearly support’ … getting off soapbox.

Be well,
Larry

G’day I have done one fix, was looking at adding the gist you did as a shortcode option next. Do not really need to take over if you will be the manager ??

• This reply was modified 4 years, 7 months ago by Larry Lewis.

Ummm … the code says;

<?php if ($use_username && !empty($logged_username)) : ?>

Which if I am reading it right says if the setting ‘use_username’ is ON and the user is logged in then it shows the ‘Name: {name}’ there is no further ‘if’ to not display, the emit code follows immediately.

Backed off my code mod, added the css didn’t work, should be

#sac-user-info { display: none; }

I have modded the output to suit .. no worries

ok need to have the logon option set to off, but now the username is not hidden ??

• This reply was modified 4 years, 10 months ago by Larry Lewis.

Yes .. it says you need to be logged in to access chat, that is with ‘Require users to be logged in to view and use the chat box.’ on and ‘Display chat messages as read-only. So visitors can view chats, but not add their own.’ on

I have that what I would like is that the chat be visible but that the entry not be available unless logged in.

ug .. yes that fixed it bugger!

Thanks!!!
Larry

No longer coding aMember, asked them to take over but they refused. Have asked for this plugin to be removed

Sorry another the snippet description doesn’t seem to be placed above the code.