fudforum / FUDforum

FUDforum is a super fast and scalable discussion forum. It is highly customizable and supports unlimited members, forums, posts, topics, polls & attachments. It can import XML Feeds and sync with USENET groups and Mailing Lists (bi-directional).
http://fudforum.org/
Other
37 stars 10 forks source link

Email registration confirmation fails #32

Closed JennekeGruen closed 1 year ago

JennekeGruen commented 1 year ago

When a new member gets an Email registration confirmation email, the confirmation/activate account link gives an error. 91E62B68-2D3D-4BD7-84B3-D9AF5DB8C2A6

Nothing in the error log.

A new user cant activate his account, after sending us an email, we can toggle email confirmation manually in the admin panel (workaround).

Not sure if related. Under datadir is the user table filled with the members. If I select it, it shoots the preg_match and htmlspecialchar deprecated error.

JennekeGruen commented 1 year ago

English error 😅: DC9A9907-90F4-46E4-9229-B2DCDBC749FD

naudefj commented 1 year ago

Fixed - please apply patch @ https://github.com/fudforum/FUDforum/commit/b18da7a2fedd38801055ca2fcf35649e943f7d06

PS: I think we need to get a new release out ASAP!

JennekeGruen commented 1 year ago

Thanx. An new version would be nice. 😊

To apply those fixes we cant use the admin panel (edit file in src) option because it shoots a 500 internal server error. We have to download, change, upload and chmod the file…

JennekeGruen commented 1 year ago

Ok. This might be a stupid question, but I just dont fully understand how patches work.

for example, fix c1626d7 (deprec errors) was in a .php file. Under the webdir. We managed to change the file and it worked like a charm.

But this change b18da7a (user confirmation) was in a php.t file. Under the fudforum scr folder. Changing that didnt solve the issue.

question is:

naudefj commented 1 year ago

The first option should work - just recompile the theme. Option two can also work, but is not ready yet. I hope it's not option 3!!!

JennekeGruen commented 1 year ago

🥳 option 1 did the trick.