Skip to content

minor fixes on sfFormField and sfSessionStorage #386

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

pedromellofh
Copy link

sfFormField lacks defaults in __construct and sfSessionRestore throws a warning when calls session_regenerate_id uppon an already started

@alquerci
Copy link
Contributor

@thePanz
Copy link
Member

thePanz commented Apr 22, 2025

Thanks @alquerci for the pointer to SF 7.2 implementation.

@pedromellofh I would follow that example here, and NOT silence the call triggering the warning.

Would you adapt the code, by taking in the example from SF 7.2 linked by @alquerci ? thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants