[Patch] Mo. bis Fr. vorauswählen bei Wochentagstimern

[GetAway]

Timerlist: automatic set mo.-fr. when selected weekdays. Ported from Neutrino-HD.
auto_set_mo_to_fri_when_selected_weekdays_timerlist.diff

[Mourice]

Timerlist: automatic set mo.-fr. when selected weekdays. Ported from Neutrino-HD.
auto_set_mo_to_fri_when_selected_weekdays_timerlist.diff

Hi,

getestet --> geht schon ganz gut.

Nur beim Aufrufen der Timerliste --> Neuer Timer ist ja standardmässig "einmalig" aktiv.
Da sind dann schon die Wochentage "markiert".

Schaltet man einmal durch sind dann auch bei "einmalig" wieder die Wochentage deaktiviert.

MfG

Edit:
vielleicht besser weglassen ...

.
.
.
@@ -1049,8 +1055,8 @@ int CTimerList::newTimer()
CIntInput timerSettings_repeatCount(LOCALE_TIMERLIST_REPEATCOUNT, (long&)timerNew.repeatCount,3, LOCALE_TIMERLIST_REPEATCOUNT_HELP1, LOCALE_TIMERLIST_REPEATCOUNT_HELP2);
CMenuForwarder *m5 = new CMenuForwarder(LOCALE_TIMERLIST_REPEATCOUNT, false,timerSettings_repeatCount.getValue() , &timerSettings_repeatCount);

- CTimerListRepeatNotifier notifier((int *)&timerNew.eventRepeat,m4,m5);
- strcpy(m_weekdaysStr,"-------");
+ CTimerListRepeatNotifier notifier((int *)&timerNew.eventRepeat, m4, m5, m_weekdaysStr);
+ strcpy(m_weekdaysStr,"XXXXX--");
CMenuOptionChooser* m3 = new CMenuOptionChooser(LOCALE_TIMERLIST_REPEAT, (int *)&timerNew.eventRepeat, TIMERLIST_REPEAT_OPTIONS, TIMERLIST_REPEAT_OPTION_COUNT, true, &notifier);

[GetAway]

@Mourice

Danke fürs Testen. Ich hatte nur mit einem bereits angelegten Timer geprüft.
Der Bug müsste dann auch in Neutrino-HD gefixt werden.

Patch aktualisiert:
auto_set_mo_to_fri_when_selected_weekdays_timerlist.diff

[Mourice]

@Mourice

Danke fürs Testen. Ich hatte nur mit einem bereits angelegten Timer geprüft.
Der Bug müsste dann auch in Neutrino-HD gefixt werden.

Patch aktualisiert:
auto_set_mo_to_fri_when_selected_weekdays_timerlist.diff

Hi,

jetzt haste eine Zeile zuviel wegoptimiert ...

strcpy(m_weekdaysStr,"-------"); muss schon bleiben


der Patch müsste so aussehen ...

Code: Alles auswählen

.
.
.
@@ -975,7 +981,7 @@ int CTimerList::modifyTimer()

 	CMenuForwarder *m5 = new CMenuForwarder(LOCALE_TIMERLIST_REPEATCOUNT, timer->eventRepeat != (int)CTimerd::TIMERREPEAT_ONCE ,timerSettings_repeatCount.getValue() , &timerSettings_repeatCount);

-	CTimerListRepeatNotifier notifier((int *)&timer->eventRepeat,m4,m5);
+	CTimerListRepeatNotifier notifier((int *)&timer->eventRepeat, m4, m5, m_weekdaysStr);
 	CMenuOptionChooser* m3 = new CMenuOptionChooser(LOCALE_TIMERLIST_REPEAT, (int *)&timer->eventRepeat, TIMERLIST_REPEAT_OPTIONS, TIMERLIST_REPEAT_OPTION_COUNT, true, &notifier);

 	CRecDirChooser recDirs(LOCALE_TIMERLIST_RECORDING_DIR,NEUTRINO_ICON_SETTINGS,NULL,timer->recordingDir);
@@ -1049,8 +1055,8 @@ int CTimerList::newTimer()
 	CIntInput timerSettings_repeatCount(LOCALE_TIMERLIST_REPEATCOUNT, (long&)timerNew.repeatCount,3, LOCALE_TIMERLIST_REPEATCOUNT_HELP1, LOCALE_TIMERLIST_REPEATCOUNT_HELP2);
 	CMenuForwarder *m5 = new CMenuForwarder(LOCALE_TIMERLIST_REPEATCOUNT, false,timerSettings_repeatCount.getValue() , &timerSettings_repeatCount);

-	CTimerListRepeatNotifier notifier((int *)&timerNew.eventRepeat,m4,m5);
+	CTimerListRepeatNotifier notifier((int *)&timerNew.eventRepeat, m4, m5, m_weekdaysStr);
 	CMenuOptionChooser* m3 = new CMenuOptionChooser(LOCALE_TIMERLIST_REPEAT, (int *)&timerNew.eventRepeat, TIMERLIST_REPEAT_OPTIONS, TIMERLIST_REPEAT_OPTION_COUNT, true, &notifier);

 	//bouquets

[GetAway]

Was geht denn nicht?

[Mourice]

Was geht denn nicht?

Hi,

wenn Du diese Zeile weglässt, fehlen auch die "-----" bei Wochentag beim ersten anstarten.

MfG

Edit:

Index: apps/tuxbox/neutrino/src/gui/timerlist.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/src/gui/timerlist.cpp,v
retrieving revision 1.115
diff -u -p -r1.115 timerlist.cpp
--- a/apps/tuxbox/neutrino/src/gui/timerlist.cpp 10 Apr 2012 13:03:49 -0000 1.115
+++ b/apps/tuxbox/neutrino/src/gui/timerlist.cpp 6 Jun 2012 10:21:11 -0000
@@ -158,20 +158,26 @@ private:
CMenuForwarder* m2;

int* iRepeat;
+ char * weekdays;
public:
- CTimerListRepeatNotifier( int* repeat, CMenuForwarder* a1, CMenuForwarder *a2)
+ CTimerListRepeatNotifier( int* repeat, CMenuForwarder* a1, CMenuForwarder *a2, char * wstr)
{
m1 = a1;
m2 = a2;
iRepeat=repeat;
+ weekdays = wstr;
}

bool changeNotify(const neutrino_locale_t, void *)
{
- if(*iRepeat >= (int)CTimerd::TIMERREPEAT_WEEKDAYS)
+ if(*iRepeat >= (int)CTimerd::TIMERREPEAT_WEEKDAYS) {
m1->setActive (true);
- else
+ strcpy(weekdays, "XXXXX--");
+ }
+ else {
m1->setActive (false);
+ strcpy(weekdays, "-------");
+ }
if (*iRepeat != (int)CTimerd::TIMERREPEAT_ONCE)
m2->setActive(true);
else
@@ -975,7 +981,7 @@ int CTimerList::modifyTimer()

CMenuForwarder *m5 = new CMenuForwarder(LOCALE_TIMERLIST_REPEATCOUNT, timer->eventRepeat != (int)CTimerd::TIMERREPEAT_ONCE ,timerSettings_repeatCount.getValue() , &timerSettings_repeatCount);

- CTimerListRepeatNotifier notifier((int *)&timer->eventRepeat,m4,m5);
+ CTimerListRepeatNotifier notifier((int *)&timer->eventRepeat, m4, m5, m_weekdaysStr);
CMenuOptionChooser* m3 = new CMenuOptionChooser(LOCALE_TIMERLIST_REPEAT, (int *)&timer->eventRepeat, TIMERLIST_REPEAT_OPTIONS, TIMERLIST_REPEAT_OPTION_COUNT, true, &notifier);

CRecDirChooser recDirs(LOCALE_TIMERLIST_RECORDING_DIR,NEUTRINO_ICON_SETTINGS,NULL,timer->recordingDir);
@@ -1049,8 +1055,7 @@ int CTimerList::newTimer()
CIntInput timerSettings_repeatCount(LOCALE_TIMERLIST_REPEATCOUNT, (long&)timerNew.repeatCount,3, LOCALE_TIMERLIST_REPEATCOUNT_HELP1, LOCALE_TIMERLIST_REPEATCOUNT_HELP2);
CMenuForwarder *m5 = new CMenuForwarder(LOCALE_TIMERLIST_REPEATCOUNT, false,timerSettings_repeatCount.getValue() , &timerSettings_repeatCount);

- CTimerListRepeatNotifier notifier((int *)&timerNew.eventRepeat,m4,m5);
- strcpy(m_weekdaysStr,"-------");
+ CTimerListRepeatNotifier notifier((int *)&timerNew.eventRepeat, m4, m5, m_weekdaysStr);
CMenuOptionChooser* m3 = new CMenuOptionChooser(LOCALE_TIMERLIST_REPEAT, (int *)&timerNew.eventRepeat, TIMERLIST_REPEAT_OPTIONS, TIMERLIST_REPEAT_OPTION_COUNT, true, &notifier);

//bouquets

Die rot markierte Zeile muss aus dem Diff raus, da diese Zeile in der Quelle drin bleiben muss.

[GetAway]

Stimmt. Bitte beim einchecken dran denken .

[rhabarber1848]

Steht die strcpy-Zeile vor oder nach der CTimerListRepeatNotifier-Zeile?

[GetAway]

Danach:

Code: Alles auswählen

	CTimerListRepeatNotifier notifier((int *)&timerNew.eventRepeat, m4, m5, m_weekdaysStr);
	strcpy(m_weekdaysStr,"-------");
	CMenuOptionChooser* m3 = new CMenuOptionChooser(LOCALE_TIMERLIST_REPEAT, (int *)&timerNew.eventRepeat, TIMERLIST_REPEAT_OPTIONS, TIMERLIST_REPEAT_OPTION_COUNT, true, &notifier);

[GetAway]

Patch nochmal aktualisiert:
auto_set_mo_to_fri_when_selected_weekdays_timerlist.diff

[Gaucho316]

Ich habe eine Bitte. Es wäre schön, wenn der Patch mit einer kleineren Änderung der Reihenfolge eingecheckt werden würde. Die Zeilen

Code: Alles auswählen

m1->setActive(...);

und

Code: Alles auswählen

strcpy(weekdays, ...);

sind vertauscht. Funktionstechnisch ändert sich dabei nichts. Es ist aber Voraussetzung für einen zweiten Patch, der das unnötige Neuzeichnen des gesamten Menuwidgets verhindert.

Timerlist: auto set Mon to Fri when selected weekdays
Link entfernt, da Patch im CVS

Timerlist: remove unnecessary repaint calls in notifiers
Link entfernt, da Patch im CVS

[rhabarber1848]

committed:

timerlist_auto-set-mo-to-fri-when-selected-weekdays_2012-06-25_1813.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3515

timerlist_remove-unnecessary-repaint-calls-in-notifiers_2012-06-25_1816.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3516