Jump to content


Photo

Update screen logic change


  • Please log in to reply
16 replies to this topic

#1 Pr2

  • PLi® Contributor
  • 6,046 posts

+256
Excellent

Posted 30 March 2015 - 21:56

Hi,

 

Few months ago was introduced a nice feature that allows to check the last commit on sourceforge from the update screen.

 

But today, when the box is already up-to-date we cannot access this menu, it leaves the update menu stating that there is nothing to update.

But the last commit information can be still useful to check.

 

For exemple if we suspect a bug into one of the latest commit, it would be nice to be able to check the last commit to see if this particular module was touched, even if there is nothing to update.

 

So would be nice that even when there is nothing to update we can still access the display last commit option.

 

Just my two cents.

 

Pr2

 


NO SUPPORT by PM, it is a forum make your question public so everybody can benefit from the question/answer.
If you think that my answer helps you, you can press the up arrow in bottom right of the answer.

Wanna help with OpenPLi Translation? Please read our Wiki Information for translators

Sat: Hotbird 13.0E, Astra 19.2E, Eutelsat5A 5.0W
VU+ Solo 4K: 2*DVB-S2 + 2*DVB-C/T/T2 (used in DVB-C) & Duo 4K: 2*DVB-S2X + DVB-C (FBC)

AB-Com: PULSe 4K 1*DVB-S2X (+ DVB-C/T/T2)
Edision OS Mio 4K: 1*DVB-S2X + 1*DVB-C/T/T2
 


Re: Update screen logic change #2 littlesat

  • PLi® Core member
  • 56,123 posts

+685
Excellent

Posted 30 March 2015 - 22:11

You can find this also on menu about your box red

WaveFrontier 28.2E | 23.5E | 19.2E | 16E | 13E | 10/9E | 7E | 5E | 1W | 4/5W | 15W


Re: Update screen logic change #3 Pr2

  • PLi® Contributor
  • 6,046 posts

+256
Excellent

Posted 30 March 2015 - 23:27

Thanks for the tip, it is not the menu I play most with.  :D


NO SUPPORT by PM, it is a forum make your question public so everybody can benefit from the question/answer.
If you think that my answer helps you, you can press the up arrow in bottom right of the answer.

Wanna help with OpenPLi Translation? Please read our Wiki Information for translators

Sat: Hotbird 13.0E, Astra 19.2E, Eutelsat5A 5.0W
VU+ Solo 4K: 2*DVB-S2 + 2*DVB-C/T/T2 (used in DVB-C) & Duo 4K: 2*DVB-S2X + DVB-C (FBC)

AB-Com: PULSe 4K 1*DVB-S2X (+ DVB-C/T/T2)
Edision OS Mio 4K: 1*DVB-S2X + 1*DVB-C/T/T2
 


Re: Update screen logic change #4 littlesat

  • PLi® Core member
  • 56,123 posts

+685
Excellent

Posted 31 March 2015 - 06:34

I know.... but your idea may be considered.... ;)


WaveFrontier 28.2E | 23.5E | 19.2E | 16E | 13E | 10/9E | 7E | 5E | 1W | 4/5W | 15W


Re: Update screen logic change #5 Taapat

  • PLi® Core member
  • 2,332 posts

+120
Excellent

Posted 1 April 2015 - 21:22

@littlesat, I see that you've made the changes, what are asked Pr2.
But it seems to me, that leave all actions, if there is no updates is not correct.
I think that all options related to the update in this case do not need show.



Re: Update screen logic change #6 littlesat

  • PLi® Core member
  • 56,123 posts

+685
Excellent

Posted 1 April 2015 - 21:26

Then hou get options on different places... That is also nit really whished....

WaveFrontier 28.2E | 23.5E | 19.2E | 16E | 13E | 10/9E | 7E | 5E | 1W | 4/5W | 15W


Re: Update screen logic change #7 littlesat

  • PLi® Core member
  • 56,123 posts

+685
Excellent

Posted 1 April 2015 - 21:27

If you remove them you get options on different location... This is also something you should not wish....

WaveFrontier 28.2E | 23.5E | 19.2E | 16E | 13E | 10/9E | 7E | 5E | 1W | 4/5W | 15W


Re: Update screen logic change #8 Taapat

  • PLi® Core member
  • 2,332 posts

+120
Excellent

Posted 1 April 2015 - 21:34

I personally do not care that the location has changed, to me it is important that is shown only that, what needs to be used.
But maybe for those people, who actively used the numeric keys, the location is important, and you are right.


Edited by Taapat, 1 April 2015 - 21:34.


Re: Update screen logic change #9 littlesat

  • PLi® Core member
  • 56,123 posts

+685
Excellent

Posted 1 April 2015 - 21:40

Yep and that is the delemma here... And it does not harm... It makes no sense to select them... But it is a choicebox so I can add fixed keys... But still the firet option is usually always offline update,,,, then it becomes latest commits.... I could put there in this case the new latest upgrade log function and then remove the other stuff...

Makes sense?

Edited by littlesat, 1 April 2015 - 21:42.

WaveFrontier 28.2E | 23.5E | 19.2E | 16E | 13E | 10/9E | 7E | 5E | 1W | 4/5W | 15W


Re: Update screen logic change #10 blzr

  • PLi® Core member
  • 2,269 posts

+118
Excellent

Posted 1 April 2015 - 22:10

agree with @Taapat, displaying all update menu entries when there's no single update available is definitely weird (and confusing).

however, I'm just wondering, is it really that necessary to change anything in this update screen 'logic'?
I, for one, choose 'software update' for a clean and simple purpose of updating the image, so I find automatic closing up the screen if there's no update quite logical and handy... /?/

in the end, if there's nothing to update you end up with only 3 sort of 'useful' entries in the SW update screen:
- show last commits on sourceforge (which you can obtain in the different place as well)
- show disclaimer
- quit
not sure if it's much added value to gain here... /?/


True sarcasm doesn't need green font...

Re: Update screen logic change #11 Taapat

  • PLi® Core member
  • 2,332 posts

+120
Excellent

Posted 1 April 2015 - 22:23

And now have an additional fourth option:

- Show latest upgrade log

 

I like this logic changes.
I think that people sometimes take a update because they do not have to do at the moment. Why, if there is no an updates, not to offer them to view commits or log? Of course, commits can also look through the info.



Re: Update screen logic change #12 blzr

  • PLi® Core member
  • 2,269 posts

+118
Excellent

Posted 1 April 2015 - 22:37

And now have an additional fourth option:

- Show latest upgrade log

which probably will show you nothing useful btw...

do you know what date mine /home/root/ipkgupgrade.log is from? March 10 ;)

(seems that's something prevents log from being overwritten with actual data, at least here /?/...)


Edited by blzr, 1 April 2015 - 22:38.

True sarcasm doesn't need green font...

Re: Update screen logic change #13 littlesat

  • PLi® Core member
  • 56,123 posts

+685
Excellent

Posted 1 April 2015 - 23:01

Mine is from april 1st???

WaveFrontier 28.2E | 23.5E | 19.2E | 16E | 13E | 10/9E | 7E | 5E | 1W | 4/5W | 15W


Re: Update screen logic change #14 blzr

  • PLi® Core member
  • 2,269 posts

+118
Excellent

Posted 1 April 2015 - 23:17

I somehow suspected it's not a common issue...

more strange is that according to the info screen last flash was on March 23rd /?/

(and I didn't flash any backup)

 

anyway, I delete the log and will be watching the situation


True sarcasm doesn't need green font...

Re: Update screen logic change #15 Taapat

  • PLi® Core member
  • 2,332 posts

+120
Excellent

Posted 2 April 2015 - 17:12

@littlesat, I see that you've made the changes for which, we are talkingt. But they is not really correct. If no updates then also option "Show packages to be upgraded" should be removed.



Re: Update screen logic change #16 Taapat

  • PLi® Core member
  • 2,332 posts

+120
Excellent

Posted 2 April 2015 - 17:20

And if we are talking about coding style, then maybe it is better to use the append as in the rest of the code rather than +=?

And append is much faster.


Edited by Taapat, 2 April 2015 - 17:24.


Re: Update screen logic change #17 littlesat

  • PLi® Core member
  • 56,123 posts

+685
Excellent

Posted 2 April 2015 - 18:46

 "Show packages to be upgraded"  :)


WaveFrontier 28.2E | 23.5E | 19.2E | 16E | 13E | 10/9E | 7E | 5E | 1W | 4/5W | 15W



1 user(s) are reading this topic

0 members, 1 guests, 0 anonymous users