Skip to content

Conversation

@mabar
Copy link
Contributor

@mabar mabar commented Jun 18, 2019

Closes #155

Should be the only option which is configurable only in session_start.

@mabar mabar force-pushed the session-read-close branch 2 times, most recently from f54f2cf to d1338e5 Compare June 18, 2019 13:10
@dg
Copy link
Member

dg commented Jul 3, 2019

Because session is configured via setOptions(), it would seem better to me to use setOptions('['read_and_close' => true]) than use constructor. What do you think?

@mabar
Copy link
Contributor Author

mabar commented Jul 4, 2019

Okay. I will add check that read_and_close is only configured before session started

@mabar mabar force-pushed the session-read-close branch from d1338e5 to 24f3545 Compare July 6, 2019 13:39
@mabar mabar force-pushed the session-read-close branch from 24f3545 to 4000f7f Compare July 6, 2019 13:44
@mabar
Copy link
Contributor Author

mabar commented Jul 6, 2019

@dg Ready.

@dg
Copy link
Member

dg commented Jul 7, 2019

Great, thanks!

@dg dg merged commit 8997102 into nette:master Jul 7, 2019
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.

session - read_and_close

2 participants