[DEV] Post Love

A place for Extension Authors to post and receive feedback on Extensions still in development. No Extensions within this forum should be used within a live environment!
Anti-Spam Guide
Forum rules
READ: phpBB.com Board-Wide Rules and Regulations

IMPORTANT: Extensions Development rules

IMPORTANT FOR NEEDED EVENTS!!!
If you need an event for your extension please read this for the steps to follow to request the event(s)
EmmaL
Registered User
Posts: 20
Joined: Tue Feb 28, 2017 5:23 pm

Re: [DEV] Post Love

Post by EmmaL » Wed Jul 26, 2017 11:51 am

Alas:
[Wed Jul 26 13:46:47.236369 2017] [php7:error] [pid 7920] [client 145.129.28.51:49768] PHP Fatal error: Uncaught TypeError: Argument 1 passed to phpbb\\user_loader::load_users() must be of the type array, null given, called in /home/t-nederland-nl/www/forum/ext/anavaro/postlove/notification/postlove.php on line 100 and defined in /home/t-nederland-nl/www/forum/phpbb/user_loader.php:68\nStack trace:\n#0 /home/t-nederland-nl/www/forum/ext/anavaro/postlove/notification/postlove.php(100): phpbb\\user_loader->load_users(NULL)\n#1 /home/t-nederland-nl/www/forum/phpbb/notification/manager.php(273): anavaro\\postlove\\notification\\postlove->find_users_for_notification(Array, Array)\n#2 /home/t-nederland-nl/www/forum/ext/anavaro/postlove/controller/notifyhelper.php(68): phpbb\\notification\\manager->add_notifications('notification.ty...', Array)\n#3 /home/t-nederland-nl/www/forum/ext/anavaro/postlove/controller/ajaxify.php(87): anavaro\\postlove\\controller\\notifyhelper->notify('add', '5067', 118764, 'Re: ...', '1963', '1992')\n#4 [internal function]: anavaro\\postlove\\controller\\ajaxify->base('toggle', '118764')\n#5 /home/t in /home/t-nederland-nl/www/forum/phpbb/user_loader.php on line 68, referer: https://t-nederland.nl/forum/viewtopic. ... 7&start=10

Satanasov
Registered User
Posts: 959
Joined: Sun Jul 29, 2012 5:48 am

Re: [DEV] Post Love

Post by Satanasov » Thu Jul 27, 2017 5:09 am

Did you update and cleaned your cache?

EmmaL
Registered User
Posts: 20
Joined: Tue Feb 28, 2017 5:23 pm

Re: [DEV] Post Love

Post by EmmaL » Thu Jul 27, 2017 12:48 pm

Satanasov wrote:
Thu Jul 27, 2017 5:09 am
Did you update and cleaned your cache?
Yes.

Jef Horians
Registered User
Posts: 22
Joined: Tue Jul 25, 2017 12:32 pm

Re: [DEV] Post Love

Post by Jef Horians » Wed Aug 02, 2017 8:00 am

Same error in 1.2.1 version with phpBB 3.2.1

fliper4o
Registered User
Posts: 91
Joined: Wed Mar 23, 2011 8:15 pm
Contact:

Re: [DEV] Post Love

Post by fliper4o » Fri Aug 04, 2017 9:23 am

Здравей,
I'm so glad for that simple & l<3vely ext, I'm gonna try it but probably wait for a stable release of it for 3.2.1 !
Just wanted to say thanks & keep up the good work !
Успех :P

Satanasov
Registered User
Posts: 959
Joined: Sun Jul 29, 2012 5:48 am

Re: [DEV] Post Love

Post by Satanasov » Sat Aug 05, 2017 8:20 am

Notification issue should be fixed now.

Migrated images to FontAwsome.

EmmaL
Registered User
Posts: 20
Joined: Tue Feb 28, 2017 5:23 pm

Re: [DEV] Post Love

Post by EmmaL » Sat Aug 05, 2017 10:10 am

I'm still getting the same notification error, now in line 99 instead of 100... deactivated plugin, updated, activated plugin, oh, and cleared cache, of course.

Issue appears on both my test forum (running 3.2.0 on php 7.0.16) as well as the main forum (3.2.1 on php 7.1.7)

Is there any way I can help troubleshooting?

Satanasov
Registered User
Posts: 959
Joined: Sun Jul 29, 2012 5:48 am

Re: [DEV] Post Love

Post by Satanasov » Sat Aug 05, 2017 12:24 pm

Please try again. Apparently there is a change in the notifications system since 3.1 that I was unaware.

EmmaL
Registered User
Posts: 20
Joined: Tue Feb 28, 2017 5:23 pm

Re: [DEV] Post Love

Post by EmmaL » Sat Aug 05, 2017 3:38 pm

👍

Working now 😃

User avatar
pszyman
Registered User
Posts: 89
Joined: Tue Feb 25, 2003 10:52 am
Location: Dublin

Re: [DEV] Post Love

Post by pszyman » Sun Aug 06, 2017 8:58 am

Latest changes are simply great! Thanks a lot for such a fantastic mod.

Just a small request. Would that be possible to customize the like symbol? E.g. change heart to thumb up?

Many thanks in advance!

Satanasov
Registered User
Posts: 959
Joined: Sun Jul 29, 2012 5:48 am

Re: [DEV] Post Love

Post by Satanasov » Sun Aug 06, 2017 3:59 pm


User avatar
pszyman
Registered User
Posts: 89
Joined: Tue Feb 25, 2003 10:52 am
Location: Dublin

Re: [DEV] Post Love

Post by pszyman » Sun Aug 06, 2017 5:03 pm

Thanks for prompt answer! Mission accomplished :D.

P.S Are you aware links from the users likes page are not working? Link gets app.php before viewtopic part.

http://www.domain.com/forum/app.php/viewtopic.php?t=18183

Satanasov
Registered User
Posts: 959
Joined: Sun Jul 29, 2012 5:48 am

Re: [DEV] Post Love

Post by Satanasov » Mon Aug 07, 2017 4:53 am

pszyman wrote:
Sun Aug 06, 2017 5:03 pm
Thanks for prompt answer! Mission accomplished :D.

P.S Are you aware links from the users likes page are not working? Link gets app.php before viewtopic part.

http://www.domain.com/forum/app.php/viewtopic.php?t=18183
Yes ... this functionality is in testing stage ... I will try and fix it when I have time.

EmmaL
Registered User
Posts: 20
Joined: Tue Feb 28, 2017 5:23 pm

Re: [DEV] Post Love

Post by EmmaL » Sun Aug 13, 2017 2:25 pm

Somehow I made a wrong assumption in my translation, I assumed the post title would be there in the notification list. I'll have to update again.

That assumption doesn't appear to originate from my imagination entirely. In the code, there's this:

Code: Select all

	/**
	 * Get the HTML formatted reference of the notification
	 *
	 * @return string

	public function get_reference()
	{
		return censor_text($this->get_data('post_subject'));
	}*/
	
What's the reason for this to be commented out?

Satanasov
Registered User
Posts: 959
Joined: Sun Jul 29, 2012 5:48 am

Re: [DEV] Post Love

Post by Satanasov » Mon Aug 14, 2017 8:06 am

EmmaL wrote:
Sun Aug 13, 2017 2:25 pm
Somehow I made a wrong assumption in my translation, I assumed the post title would be there in the notification list. I'll have to update again.

That assumption doesn't appear to originate from my imagination entirely. In the code, there's this:

Code: Select all

	/**
	 * Get the HTML formatted reference of the notification
	 *
	 * @return string

	public function get_reference()
	{
		return censor_text($this->get_data('post_subject'));
	}*/
	
What's the reason for this to be commented out?
This is code left from development ... Probably I have used it at some point and removed it ...

Post Reply

Return to “Extensions in Development”

Who is online

Users browsing this forum: AbaddonOrmuz, Froddelaar, joeyb+1, m82a1pa, maximuspoder, Meis2M, Midgardo, Steve and 28 guests