[3.2][RC] Glossary 3.0.8

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!
Scam Warning
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)
rockmartin
Registered User
Posts: 28
Joined: Tue Apr 26, 2016 7:00 am

Re: [RC] Glossary 1.3.20

Post by rockmartin »

hello, i got this in the ACP

Code: Select all

SQL ERROR [ mysqli ]

Invalid default value for 'lang' [1067]

SQL

ALTER TABLE phpbb3_glossary ALTER COLUMN lang SET DEFAULT 'de_x_sie'

BACKTRACE

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

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

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

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

FILE: [ROOT]/ext/lmdi/gloss/acp/gloss_module.php
LINE: 76
CALL: phpbb\db\driver\factory->sql_query()

FILE: [ROOT]/includes/functions_module.php
LINE: 674
CALL: lmdi\gloss\acp\gloss_module->main()

FILE: [ROOT]/adm/index.php
LINE: 81
CALL: p_master->load_active()

User avatar
pierredu
Registered User
Posts: 1051
Joined: Thu Nov 01, 2012 8:04 am
Location: Paris (France)

Re: [RC] Glossary 1.3.20

Post by pierredu »

I think the language constant should be a two-letter constant (de).

User avatar
canonknipser
Registered User
Posts: 2096
Joined: Thu Sep 08, 2011 4:16 am
Location: Germany
Name: Frank Jakobs
Contact:

Re: [RC] Glossary 1.3.20

Post by canonknipser »

some languages exist more than once (dialects like en_us, formal language like de_x_sie etc.) see: http://area51.phpbb.com/docs/30x/coding ... ranslation
Greetings, Frank
phpbb.de support team member
English is not my native language - no support via PM or mail
New arrival - Extensions and scripts for phpBB

User avatar
pierredu
Registered User
Posts: 1051
Joined: Thu Nov 01, 2012 8:04 am
Location: Paris (France)

Re: [RC] Glossary 1.3.20

Post by pierredu »

Indeed.

I'll check tomorrow why this happens. The lang column is wide enough to get the de-x-sie value.

User avatar
wowmatu
Registered User
Posts: 24
Joined: Fri Mar 22, 2013 10:55 pm

Re: [RC] Glossary 1.3.20

Post by wowmatu »

when the final version would be available?

User avatar
pierredu
Registered User
Posts: 1051
Joined: Thu Nov 01, 2012 8:04 am
Location: Paris (France)

Re: [RC] Glossary 1.3.20

Post by pierredu »

This extension is already in production on my boards.

User avatar
pierredu
Registered User
Posts: 1051
Joined: Thu Nov 01, 2012 8:04 am
Location: Paris (France)

Re: [RC] Glossary 1.3.20

Post by pierredu »

I installed the de_x_sie language pack on my board and tested the default language change feature.
No problem.

It must be something else.
Another extension? I doubt it.
An older version of the extension? Could you check with the latest one (on github)?

rockmartin
Registered User
Posts: 28
Joined: Tue Apr 26, 2016 7:00 am

Re: [RC] Glossary 1.3.20

Post by rockmartin »

I have download the latest version (1.3.24) from github. Same error



Code: Select all

SQL ERROR [ mysqli ]

Invalid default value for 'lang' [1067]

SQL

ALTER TABLE phpbb3_glossary ALTER COLUMN lang SET DEFAULT 'de_x_sie'

BACKTRACE

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

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

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

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

FILE: [ROOT]/ext/lmdi/gloss/acp/gloss_module.php
LINE: 81
CALL: phpbb\db\driver\factory->sql_query()

FILE: [ROOT]/includes/functions_module.php
LINE: 674
CALL: lmdi\gloss\acp\gloss_module->main()

FILE: [ROOT]/adm/index.php
LINE: 81
CALL: p_master->load_active()
Other extensions that I have installed:
  • Auto Database Backup 1.1.0
  • External Links Open in New Window 1.0.1
  • Lightbox 1.0.2
  • Pages 1.0.4
  • phpBB.de - External Image as Link 1.0.0
  • Precise Similar Topics 1.3.2
  • Private Message Box Status Bars 2.0.1

User avatar
kasimi
Extension Customisations
Extension Customisations
Posts: 4056
Joined: Sat Sep 10, 2011 7:12 pm
Location: Germany
Contact:

Re: [RC] Glossary 1.3.20

Post by kasimi »

pierredu wrote:The lang column is wide enough to get the de-x-sie value.
Are you sure? It's set to VCHAR:2 https://github.com/pierrdu/lmdi_gloss/b ... _1.php#L40

User avatar
pierredu
Registered User
Posts: 1051
Joined: Thu Nov 01, 2012 8:04 am
Location: Paris (France)

Re: [RC] Glossary 1.3.20

Post by pierredu »

Kasimi,

As always, you're right.
I'll correct this immediatly.

I was using a version which was modified since.

My apologies.

I'll correct the code. But I have to code a new migration file and this will take some time.

For the OP, the simplest solution would be to rename the language pack or to extend the column width in phpMyAdmin.

rockmartin
Registered User
Posts: 28
Joined: Tue Apr 26, 2016 7:00 am

Re: [RC] Glossary 1.3.20

Post by rockmartin »

Is there already an update? :D

User avatar
pierredu
Registered User
Posts: 1051
Joined: Thu Nov 01, 2012 8:04 am
Location: Paris (France)

Re: [RC] Glossary 1.3.20

Post by pierredu »

The current version on github is corrected.

User avatar
wowmatu
Registered User
Posts: 24
Joined: Fri Mar 22, 2013 10:55 pm

Re: [RC] Glossary 1.3.20

Post by wowmatu »

When the RTM version is available, so that it is already available in "Extensions Database Releases"

rockmartin
Registered User
Posts: 28
Joined: Tue Apr 26, 2016 7:00 am

Re: [RC] Glossary 1.3.20

Post by rockmartin »

pierredu wrote:
Thu Mar 23, 2017 12:49 pm
The current version on github is corrected.
If I activate the plugin I get the following error.
LMDI Glossary: 1.4.2
phpBB: 3.1.10

Code: Select all

Allgemeiner Fehler
SQL ERROR [ mysqli ]

Invalid default value for 'lang' [1067]

SQL

ALTER TABLE phpbb3_glossary ALTER COLUMN lang SET DEFAULT 'de_x_sie'

BACKTRACE

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

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

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

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

FILE: [ROOT]/ext/lmdi/gloss/acp/gloss_module.php
LINE: 69
CALL: phpbb\db\driver\factory->sql_query()

FILE: [ROOT]/includes/functions_module.php
LINE: 674
CALL: lmdi\gloss\acp\gloss_module->main()

FILE: [ROOT]/adm/index.php
LINE: 81
CALL: p_master->load_active()

User avatar
pierredu
Registered User
Posts: 1051
Joined: Thu Nov 01, 2012 8:04 am
Location: Paris (France)

Re: [RC] Glossary 1.3.20

Post by pierredu »

Are you using an old installed version ?

Are you using MySQL oe something else?

The width of the lang column was updated in the latest migration file (release_1_3.php) and should now be 12 byte (enough for de-x-sie). Please backup your existing phpbb3_glossary table if you have something valuable in there. Disable the extension. Delete the data. Then reinstall. If you had a backup of the glossary table, delete the newly created table and rename the backup.

Post Reply

Return to “Extensions in Development”