[3.2][3.3][RC] Thanks for posts (and optional Tapatalk integration)

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!
Get Involved
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)
LucioTS
Registered User
Posts: 15
Joined: Wed Nov 08, 2023 11:15 am

Re: [3.2][3.3][RC] Thanks for posts (and optional Tapatalk integration)

Post by LucioTS »

Yes.
I installed your version, tested it but then the error occurred which I thought depended on Thank's.
I took the screenshot and posted but realised it didn't depend on your plugin and removed it from the post but by then it was too late.

Thanks for everything.
User avatar
Naguissa
Registered User
Posts: 502
Joined: Thu Mar 02, 2017 6:55 am
Location: Barcelona
Name: Naguissa

Re: [3.2][3.3][RC] Thanks for posts (and optional Tapatalk integration)

Post by Naguissa »

LucioTS wrote: Fri Mar 01, 2024 5:24 pm Yes.
I installed your version, tested it but then the error occurred which I thought depended on Thank's.
I took the screenshot and posted but realised it didn't depend on your plugin and removed it from the post but by then it was too late.

Thanks for everything.
One note. Lucio seems italian name (maybe argentinian?). If you are from UE, I've manage several legal requirements on Spain, and Google CMP is the most simple one to implement if you use AdSense.

If not, cookiebot and others are fine, but most of them are paid.....
leschek
Registered User
Posts: 856
Joined: Tue Jul 18, 2006 12:49 pm

Re: [3.2][3.3][RC] Thanks for posts (and optional Tapatalk integration)

Post by leschek »

I was testing this extension and run into this issue:

If I change setting (for example I switch "Informative messages:" to "Yes") in "ACP" → "Thanks Settings" and "Submit" it it shows me this message:

Code: Select all

Warning

The provided value for the setting “Number of thanks from the list shown in profile” is too low. The minimum acceptable value is 1
I add to the field value: 10 and submit it again. It looks OK, but when I go back to the settings I can see, that setting "Informative messages:" is switched back to "No" and field "Number of thanks from the list shown in profile" is without value.

I tried to disable the extension and delete data, but (after enabling it again) it behaves the same.

I tried it on another board and it again looks like it doesn't remember settings I submit - again the same issue.

I use:
Thanks for posts 4.0.6
phpBB 3.3.11
PHP 8.1.27
mysql (10.6.16-MariaDB-cll-lve) - info from cPanel
LucioTS
Registered User
Posts: 15
Joined: Wed Nov 08, 2023 11:15 am

Re: [3.2][3.3][RC] Thanks for posts (and optional Tapatalk integration)

Post by LucioTS »

Sorry to bother you again
but I have a new problem.

Version 4.0.6
I have enabled the extension, in the db there is the xxxx_thanks table with the various data.

I see the total number of thanks (in the forum index, for example, at the bottom) but the details of the thanks in the individual posts do not appear, nor does the button to make the post thank you.

I have tried disabling and re-enabling the plugin, clearing the cache but it doesn't work.
Any suggestions?

PHP version 8.2
Forum version 3.3.11
Extension: 4.0.6 (under ext/naguissa/thanksforpost)

Thanks for your helpfulness.

Bye

P.S. as you asked a few posts above.... Lucio, my name, is Italian, country where I live.
LucioTS
Registered User
Posts: 15
Joined: Wed Nov 08, 2023 11:15 am

Re: [3.2][3.3][RC] Thanks for posts (and optional Tapatalk integration)

Post by LucioTS »

Hi,
I think I have solved it.
I disabled, deleted the data, removed and then reinstalled the extension.
I then retrieved the db thanks table from a backup.

Everything is working again.
User avatar
Naguissa
Registered User
Posts: 502
Joined: Thu Mar 02, 2017 6:55 am
Location: Barcelona
Name: Naguissa

Re: [3.2][3.3][RC] Thanks for posts (and optional Tapatalk integration)

Post by Naguissa »

leschek wrote: Mon Mar 04, 2024 3:18 pm I was testing this extension and run into this issue:

If I change setting (for example I switch "Informative messages:" to "Yes") in "ACP" → "Thanks Settings" and "Submit" it it shows me this message:

Code: Select all

Warning

The provided value for the setting “Number of thanks from the list shown in profile” is too low. The minimum acceptable value is 1
I add to the field value: 10 and submit it again. It looks OK, but when I go back to the settings I can see, that setting "Informative messages:" is switched back to "No" and field "Number of thanks from the list shown in profile" is without value.

I tried to disable the extension and delete data, but (after enabling it again) it behaves the same.

I tried it on another board and it again looks like it doesn't remember settings I submit - again the same issue.

I use:
Thanks for posts 4.0.6
phpBB 3.3.11
PHP 8.1.27
mysql (10.6.16-MariaDB-cll-lve) - info from cPanel

Seems some configs are missing. I've just installed phpbb and some extensions to do some tests in a NAS and I've faced no problem. But I write here an mySQL to recheck all config values:

Code: Select all

INSERT IGNORE INTO phpbb_config (config_name, config_value, is_dynamic) VALUES
('thanks_ajax', 1, 0), 
('thanks_counters_view', 1, 0), 
('thanks_forum_reput_view', 1, 0), 
('thanks_global_announce', 1, 0), 
('thanks_global_post', 0, 0), 
('thanks_info_page', 1, 0), 
('thanks_number', 100, 0), 
('thanks_number_digits', 2, 0), 
('thanks_number_post', 10, 0), 
('thanks_number_row_reput', 5, 0), 
('thanks_only_first_post', 0, 0), 
('thanks_post_reput_view', 1, 0), 
('thanks_post_view_guests', 1, 0), 
('thanks_post_view_robots', 1, 0), 
('thanks_postlist_view', 1, 0), 
('thanks_profilelist_view', 1, 0), 
('thanks_reput_graphic', 1, 0), 
('thanks_symbol_remove', 'fa-recycle', 0), 
('thanks_symbol_thanks', 'fa-thumbs-o-up', 0), 
('thanks_time_view', 1, 0), 
('thanks_top_number', 15, 0), 
('thanks_topic_reput_view', 1, 0);
User avatar
Naguissa
Registered User
Posts: 502
Joined: Thu Mar 02, 2017 6:55 am
Location: Barcelona
Name: Naguissa

Re: [3.2][3.3][RC] Thanks for posts (and optional Tapatalk integration)

Post by Naguissa »

LucioTS wrote: Mon Mar 18, 2024 6:27 pm Hi,
I think I have solved it.
I disabled, deleted the data, removed and then reinstalled the extension.
I then retrieved the db thanks table from a backup.

Everything is working again.
No idea what could have happened.....
leschek
Registered User
Posts: 856
Joined: Tue Jul 18, 2006 12:49 pm

Re: [3.2][3.3][RC] Thanks for posts (and optional Tapatalk integration)

Post by leschek »

Naguissa wrote: Wed Mar 27, 2024 9:36 am
Seems some configs are missing. I've just installed phpbb and some extensions to do some tests in a NAS and I've faced no problem. But I write here an mySQL to recheck all config values:

Code: Select all

INSERT IGNORE INTO phpbb_config (config_name, config_value, is_dynamic) VALUES
('thanks_ajax', 1, 0), 
('thanks_counters_view', 1, 0), 
('thanks_forum_reput_view', 1, 0), 
('thanks_global_announce', 1, 0), 
('thanks_global_post', 0, 0), 
('thanks_info_page', 1, 0), 
('thanks_number', 100, 0), 
('thanks_number_digits', 2, 0), 
('thanks_number_post', 10, 0), 
('thanks_number_row_reput', 5, 0), 
('thanks_only_first_post', 0, 0), 
('thanks_post_reput_view', 1, 0), 
('thanks_post_view_guests', 1, 0), 
('thanks_post_view_robots', 1, 0), 
('thanks_postlist_view', 1, 0), 
('thanks_profilelist_view', 1, 0), 
('thanks_reput_graphic', 1, 0), 
('thanks_symbol_remove', 'fa-recycle', 0), 
('thanks_symbol_thanks', 'fa-thumbs-o-up', 0), 
('thanks_time_view', 1, 0), 
('thanks_top_number', 15, 0), 
('thanks_topic_reput_view', 1, 0);
You are correct. I looked into database and it seems I don't have there some rows. If I go one by one the following rows are missing:

thanks_counters_view
thanks_global_post
thanks_info_page
thanks_number
thanks_only_first_post
thanks_postlist_view
thanks_profilelist_view

Here is screenshot:
Image
alansh
Registered User
Posts: 34
Joined: Fri May 17, 2024 4:17 pm

Re: [3.2][3.3][RC] Thanks for posts (and optional Tapatalk integration)

Post by alansh »

I just downloaded the latest version from github and tried to enable it from ACP. I got this error:

[phpBB Debug] PHP Warning: in file [ROOT]/phpbb/db/migrator.php on line 662: Undefined array key 44

Any ideas?

Alan
User avatar
Naguissa
Registered User
Posts: 502
Joined: Thu Mar 02, 2017 6:55 am
Location: Barcelona
Name: Naguissa

Re: [3.2][3.3][RC] Thanks for posts (and optional Tapatalk integration)

Post by Naguissa »

alansh wrote: Sat May 25, 2024 11:57 am I just downloaded the latest version from github and tried to enable it from ACP. I got this error:

[phpBB Debug] PHP Warning: in file [ROOT]/phpbb/db/migrator.php on line 662: Undefined array key 44

Any ideas?

Alan
It's a warning on migration, should not affect. Is the extension working properly?
alansh
Registered User
Posts: 34
Joined: Fri May 17, 2024 4:17 pm

Re: [3.2][3.3][RC] Thanks for posts (and optional Tapatalk integration)

Post by alansh »

When I try and enable it, I get these errors (and it is not enabled).

Code: Select all

General Error
SQL ERROR [ mysqli ]

Duplicate entry 'naguissa/thanksforposts' for key 'ext_name' [1062]

SQL

INSERT INTO phpbb_ext (ext_name, ext_active, ext_state) VALUES ('naguissa/thanksforposts', 0, 's:13:\"notifications\";')

BACKTRACE

FILE: (not given by php)
LINE: (not given by php)
CALL: msg_handler()

FILE: [ROOT]/phpbb/db/driver/driver.php
LINE: 1031
CALL: trigger_error()

FILE: [ROOT]/phpbb/db/driver/mysqli.php
LINE: 202
CALL: phpbb\db\driver\driver->sql_error()

FILE: [ROOT]/phpbb/db/driver/factory.php
LINE: 353
CALL: phpbb\db\driver\mysqli->sql_query()

FILE: [ROOT]/phpbb/extension/manager.php
LINE: 179
CALL: phpbb\db\driver\factory->sql_query()

FILE: [ROOT]/phpbb/extension/manager.php
LINE: 241
CALL: phpbb\extension\manager->update_state()

FILE: [ROOT]/includes/acp/acp_extensions.php
LINE: 209
CALL: phpbb\extension\manager->enable_step()

FILE: [ROOT]/includes/functions_module.php
LINE: 676
CALL: acp_extensions->main()

FILE: [ROOT]/adm/index.php
LINE: 81
CALL: p_master->load_active()
User avatar
danieltj
Infrastructure Team Member
Infrastructure Team Member
Posts: 515
Joined: Thu May 03, 2018 9:32 pm
Location: United Kingdom
Name: Daniel James

Re: [3.2][3.3][RC] Thanks for posts (and optional Tapatalk integration)

Post by danieltj »

alansh wrote: Sat May 25, 2024 2:42 pm

Code: Select all

Duplicate entry 'naguissa/thanksforposts' for key 'ext_name' [1062]
You're getting that error because it got half way through the extension installation process and then it failed (as there was an undefined variable). You need to reset your database back to before you tried to install it the first time.

You should be able to remove the row containing naguissa/thanksforposts from the *_ext table. That should fix that issue but you may have other problems depending on at what point the installation stopped.
💷 Purchase the Awesome Payments extension today!
Monetise your forum with one off payments and subscriptions.

Need a premium extension created? Send me a PM.
alansh
Registered User
Posts: 34
Joined: Fri May 17, 2024 4:17 pm

Re: [3.2][3.3][RC] Thanks for posts (and optional Tapatalk integration)

Post by alansh »

Thanks. I'll give that a go once I can work out how to reset the database.
User avatar
danieltj
Infrastructure Team Member
Infrastructure Team Member
Posts: 515
Joined: Thu May 03, 2018 9:32 pm
Location: United Kingdom
Name: Daniel James

Re: [3.2][3.3][RC] Thanks for posts (and optional Tapatalk integration)

Post by danieltj »

When I said reset database - a poor choice of words - I meant use a backup to roll back to before the first installation.
💷 Purchase the Awesome Payments extension today!
Monetise your forum with one off payments and subscriptions.

Need a premium extension created? Send me a PM.
alansh
Registered User
Posts: 34
Joined: Fri May 17, 2024 4:17 pm

Re: [3.2][3.3][RC] Thanks for posts (and optional Tapatalk integration)

Post by alansh »

Ah, it's just a small test one. I can just delete it and start again. I have no backup of that one.

Thanks for the advice.

Alan

Return to “Extensions in Development”