Skip to content
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

Pause stays on for Log Off screen #9

Closed
wwiv opened this issue Apr 22, 2005 · 8 comments
Closed

Pause stays on for Log Off screen #9

wwiv opened this issue Apr 22, 2005 · 8 comments
Assignees
Milestone

Comments

@wwiv
Copy link
Contributor

wwiv commented Apr 22, 2005

The pause stays on for the Log Off screen during the log
off process, and causes the node to stay online even
though the telenet person has disconnected.

Original comment by: seacrest

@wwiv
Copy link
Contributor Author

wwiv commented Jun 18, 2014

recommended patch:

Index: menusupp.cpp
===================================================================
--- menusupp.cpp    (revision 581)
+++ menusupp.cpp    (working copy)
@@ -306,7 +306,9 @@
       write_inst(INST_LOC_LOGOFF, 0, INST_FLAGS_NONE);
       GetSession()->bout.ClearScreen();
       GetSession()->bout << "Time on   = " << ctim(timer() - timeon) << wwiv::endl;
+      tmp_disable_pause(true);
       printfile(LOGOFF_NOEXT);
+      tmp_disable_pause(false);
       GetSession()->GetCurrentUser()->SetLastSubNum(GetSession()->GetCurrentMessageArea());
       GetSession()->GetCurrentUser()->SetLastDirNum(GetSession()->GetCurrentFileArea());
       if (okconf(GetSession()->GetCurrentUser())) {

Original comment by: ericpareja

@wwiv
Copy link
Contributor Author

wwiv commented Jun 18, 2014

  • status: open --> fixed
  • assigned_to: Eric Pareja
  • Group: --> 5.0.0
  • Priority: 5 --> 1

Original comment by: wwiv

@wwiv
Copy link
Contributor Author

wwiv commented Jun 18, 2014

Applied Xenos's patch (add tmp_pause_disable around the printfile call) just like above if we have matrix logoff enabled.

Original comment by: wwiv

@wwiv
Copy link
Contributor Author

wwiv commented Jun 18, 2014

Fixed in r581

Original comment by: wwiv

@wwiv
Copy link
Contributor Author

wwiv commented Jul 19, 2014

Replacement function TempDisablePause introduced in r701 doesn't work as expected. User's PauseOnPage flag gets unset permanently after log off but doesn't take effect before (?) logoff causing logoff.msg screen (if more than a page long) to display the pause prompt.

Original comment by: ericpareja

@wwiv
Copy link
Contributor Author

wwiv commented Jul 19, 2014

  • status: fixed --> open

Original comment by: ericpareja

@wwiv
Copy link
Contributor Author

wwiv commented Jul 20, 2014

  • status: open --> fixed

Original comment by: ericpareja

@wwiv
Copy link
Contributor Author

wwiv commented Jul 20, 2014

fixed by Rushfan in r728

Original comment by: ericpareja

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

No branches or pull requests

4 participants
@ericpareja @askpatrickw @wwiv and others