Bug 654524

Summary:	current KDE : Factory version of Yast2 GUI does not allow enable/disable of repos
Product:	[openSUSE] openSUSE 11.2	Reporter:	j xavier <junkDawgie>
Component:	YaST2	Assignee:	Ladislav Slezák <lslezak>
Status:	RESOLVED INVALID	QA Contact:	Jiri Srain <jsrain>
Severity:	Minor
Priority:	P4 - Low	CC:	tgoettlicher
Version:	Final
Target Milestone:	---
Hardware:	i586
OS:	openSUSE 11.2
Whiteboard:
Found By:	---	Services Priority:
Business Priority:		Blocker:	---
Marketing QA Status:	---	IT Deployment:	---

Description j xavier 2010-11-18 04:28:40 UTC

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.2.3) Gecko/20100401 Firefox/3.6.3

Yast2 version from KDE:Factory  yast2-qt-2.18.7-2.1.i586

GUI for Configure/Repositories has checkboxes (enable/disable) located lower left to change status of repo highlighted in list above. These boxes do not make the appropriate changes when toggled, the checkbox appears checked or unchecked but the corresponding indicator in the list does not (using zypper verifies that it hasn't changed, nor will it be changed if you close and restart the program).

clicking the appropriate place on the list is successful in toggling the status.

Reproducible: Always

Steps to Reproduce:
1.start Configuration/Repositories GUI
2.highlight a disabled repo
3.check the box lower left to enable, close normally.
Actual Results:  
repo just activated is not enabled.

Expected Results:  
repo should be enabled.

as stated above, the list of repo's can be clicked in the appropriate place to enable/disable successfully.

Comment 1 Martin Kudlvasr 2010-11-18 15:20:07 UTC

In my opinion, this is an invalid candidate (Factory yast2 on OS11.2). Maintainer of the yast2-qt should have the final saying. Thomas?

Comment 2 Thomas Göttlicher 2010-11-18 16:37:46 UTC

Tested the repo view with openSUSE 11.3. Double clicking the Autorefresh column doesn't influence the value of Autorefresh but the value of Enabled. 

Reassigning to maintainer of yast2-packager.

Comment 3 Ladislav Slezák 2010-11-19 09:58:58 UTC

To comment #2: the problem is that Yast UI event doesn't contain the table column which has been clicked:

2010-11-19 08:43:28 <5> muffin(29802) [YCP] clients/repositories.ycp:808   Event: $[
     "EventReason":"Activated",
     "EventSerialNo":4,
     "EventType":"WidgetEvent",
     "ID":`table,
     "WidgetClass":`Table,
     "WidgetID":`table
]

Yast just simply enables/disables the after double-click everywhere in the row (regardless the column clicked).

To fix it UI::WaitForEvent() has to return extra value (column number in the table) to the calling script so it can react accordingly. Thomas?

(BTW I could not reproduce the original problem in Factory, it's OK for me.)

Comment 4 Thomas Göttlicher 2010-11-19 14:26:00 UTC

(In reply to comment #3)
> To fix it UI::WaitForEvent() has to return extra value (column number in the
> table) to the calling script so it can react accordingly. Thomas?
You are right. The UI reports an activation event only and doesn't distinguish between columns. We don't want to change this because most UIs don't use double clicks for toggling columns but selection items.

Comment 5 j xavier 2010-11-24 19:28:16 UTC

i'm just a user and perhaps my understanding of these comments is lacking, but my original bug report indicates that the checkboxes lower left (enable, autorefresh and KeepDlPackages) simply do not cause any changes if used... only the list of repo's respond successfully in enable/disable using double-clicks.

Comment 6 Ladislav Slezák 2010-11-26 16:23:41 UTC

Sorry for the off-topic discussion...

I guess that KDE:Factory installed a newer libqt4 package which is not compatible with yast2-qt. Try installing original libqt4 and yast2-qt from 11.2.
(Well, that might not be possible if the latest KDE needs newer Qt...)

Mixing factory and 11.2 is not supported (as written in comment #1)...