home
products
contribute
download
documentation
forum
Home
Forums
New posts
Search forums
What's new
New posts
All posts
Latest activity
Members
Registered members
Current visitors
Donate
Log in
Register
What's new
Search
Search
Search titles only
By:
New posts
Search forums
Search titles only
By:
Menu
Log in
Register
Navigation
Install the app
Install
More options
Contact us
Close Menu
Forums
MediaPortal 1
Development
Submit: code patches (MediaPortal/TV-Server/etc.)
Restart MP when exit Config
Contact us
RSS
JavaScript is disabled. For a better experience, please enable JavaScript in your browser before proceeding.
You are using an out of date browser. It may not display this or other websites correctly.
You should upgrade or use an
alternative browser
.
Reply to thread
Message
<blockquote data-quote="tourettes" data-source="post: 1013603" data-attributes="member: 10858"><p><strong>About the feature:</strong></p><p> </p><p>I don't think we should include such feature since it is a really niche use case - no options should be added for the really limited use cases, otherwise we will end up in a really complex configuration (MP already is a monster when it comes to the user config). Also it is possible to have such functionality already:</p><ol> <li data-xf-list-type="ol">With MCE remote you just need to press GREEN button and you can open MP process. This probably is possible with other remotes that allow some configuration. And for non-programmable remotes it is possible to use some 3rd party tool.</li> <li data-xf-list-type="ol">You could rename configuration.exe to something else and then create a .bat file that opens first config and then after it closes MP.</li> </ol><p>Also you have the mouse most likely at the hand when you are about to modify settings in the non-UI tool, so probably the keyboard is as well avilable so - press CTRL-ALT-DEL and run taskmanager from there.</p><p> </p><p><strong>About the patch:</strong></p><p> </p><ol> <li data-xf-list-type="ol">Lot ofcode formatting present in the patch - ionly fucntional changes should be in. Formatting changes should be provided in a separate patch if they really are wanted. This reduces the possible conflicts with the other patches, features, fixes (== commits <img src="data:image/gif;base64,R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7" class="smilie smilie--sprite smilie--sprite1" alt=":)" title="Smile :)" loading="lazy" data-shortname=":)" />)</li> <li data-xf-list-type="ol">Some unnescessary code movement - these shouldn't be done, not even a separate patch as there is no reason to do those.</li> <li data-xf-list-type="ol">Added useless empty lines</li> <li data-xf-list-type="ol">Non-safe code with the sleep waits. A proper waiting should be used - there is no quarantee that one second is something magic and allows the configuration process to exit before MP process is started.</li> <li data-xf-list-type="ol">Copy & paste code - <strong>cbRestartMPWhenExit.Checked </strong>and the exit functionality should be a new method.</li> <li data-xf-list-type="ol"><strong>RestartMediaPortalWhenExit() </strong>method is a thread starting point. It must be handling all the exeptions or otherwise it will be possible to crash to desk to (not a big issue since application is already about to exit - but we dont want to have any system event entries of configuration crashing)..</li> <li data-xf-list-type="ol">SettingsForm.resx was changed, why?</li> </ol><p></p><p> </p><p>sorry for the naging, I just woke up <img src="data:image/gif;base64,R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7" class="smilie smilie--sprite smilie--sprite7" alt=":p" title="Stick Out Tongue :p" loading="lazy" data-shortname=":p" /></p></blockquote><p></p>
[QUOTE="tourettes, post: 1013603, member: 10858"] [B]About the feature:[/B] I don't think we should include such feature since it is a really niche use case - no options should be added for the really limited use cases, otherwise we will end up in a really complex configuration (MP already is a monster when it comes to the user config). Also it is possible to have such functionality already: [LIST=1] [*]With MCE remote you just need to press GREEN button and you can open MP process. This probably is possible with other remotes that allow some configuration. And for non-programmable remotes it is possible to use some 3rd party tool. [*]You could rename configuration.exe to something else and then create a .bat file that opens first config and then after it closes MP. [/LIST] Also you have the mouse most likely at the hand when you are about to modify settings in the non-UI tool, so probably the keyboard is as well avilable so - press CTRL-ALT-DEL and run taskmanager from there. [B]About the patch:[/B] [LIST=1] [*]Lot ofcode formatting present in the patch - ionly fucntional changes should be in. Formatting changes should be provided in a separate patch if they really are wanted. This reduces the possible conflicts with the other patches, features, fixes (== commits :)) [*]Some unnescessary code movement - these shouldn't be done, not even a separate patch as there is no reason to do those. [*]Added useless empty lines [*]Non-safe code with the sleep waits. A proper waiting should be used - there is no quarantee that one second is something magic and allows the configuration process to exit before MP process is started. [*]Copy & paste code - [B]cbRestartMPWhenExit.Checked [/B]and the exit functionality should be a new method. [*][B]RestartMediaPortalWhenExit() [/B]method is a thread starting point. It must be handling all the exeptions or otherwise it will be possible to crash to desk to (not a big issue since application is already about to exit - but we dont want to have any system event entries of configuration crashing).. [*]SettingsForm.resx was changed, why? [/LIST] sorry for the naging, I just woke up :p [/QUOTE]
Insert quotes…
Verification
Post reply
Forums
MediaPortal 1
Development
Submit: code patches (MediaPortal/TV-Server/etc.)
Restart MP when exit Config
Contact us
RSS
Top
Bottom
