FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Redhat > Fedora Development

 
 
LinkBack Thread Tools
 
Old 02-03-2010, 06:36 PM
Chris Lumens
 
Default Rename the WWID column to Identifier.

Since we're stuffing more things into this column than just WWIDS (paths on
iscsi and s390 stuff soon, at least) the column header needs to be changed
to match.
---
iw/filter_gui.py | 28 ++++++++++++++--------------
ui/filter.glade | 18 +++++++++---------
2 files changed, 23 insertions(+), 23 deletions(-)

diff --git a/iw/filter_gui.py b/iw/filter_gui.py
index 8490c26..5f3e3a6 100644
--- a/iw/filter_gui.py
+++ b/iw/filter_gui.py
@@ -44,7 +44,7 @@ CAPACITY_COL = 5
VENDOR_COL = 6
INTERCONNECT_COL = 7
SERIAL_COL = 8
-WWID_COL = 9
+ID_COL = 9
PATHS_COL = 10
PORT_COL = 11
TARGET_COL = 12
@@ -181,12 +181,12 @@ class MPathCallbacks(FilteredCallbacks):

self.vendorEntry = self.xml.get_widget("mpathVendorEntry")
self.interconnectEntry = self.xml.get_widget("mpathInterconnectEntry")
- self.WWIDEntry = self.xml.get_widget("mpathWWIDEntry")
+ self.IDEntry = self.xml.get_widget("mpathIDEntry")

self.vendorEntry.connect("changed", lambda entry: self.model.refilter())
self.vendorEntry.connect("realize", self._populateUI)
self.interconnectEntry.connect("changed", lambda entry: self.model.refilter())
- self.WWIDEntry.connect("changed", lambda entry: self.model.refilter())
+ self.IDEntry.connect("changed", lambda entry: self.model.refilter())

def addToUI(self, tuple):
if not tuple[VENDOR_COL] in self._vendors:
@@ -233,9 +233,9 @@ class MPathCallbacks(FilteredCallbacks):

def _visible_by_wwid(self, model, iter, view):
# FIXME: make this support globs, etc.
- entered = self.WWIDEntry.get_text()
+ entered = self.IDEntry.get_text()

- return entered != "" and model.get_value(iter, WWID_COL).find(entered) != -1
+ return entered != "" and model.get_value(iter, ID_COL).find(entered) != -1

class OtherCallbacks(MPathCallbacks):
def __init__(self, *args, **kwargs):
@@ -249,12 +249,12 @@ class OtherCallbacks(MPathCallbacks):

self.vendorEntry = self.xml.get_widget("otherVendorEntry")
self.interconnectEntry = self.xml.get_widget("otherInterconnectEntry")
- self.WWIDEntry = self.xml.get_widget("otherWWIDEntry")
+ self.IDEntry = self.xml.get_widget("otherIDEntry")

self.vendorEntry.connect("changed", lambda entry: self.model.refilter())
self.vendorEntry.connect("realize", self._populateUI)
self.interconnectEntry.connect("changed", lambda entry: self.model.refilter())
- self.WWIDEntry.connect("changed", lambda entry: self.model.refilter())
+ self.IDEntry.connect("changed", lambda entry: self.model.refilter())

def isMember(self, info):
return info and isOther(info)
@@ -273,14 +273,14 @@ class SearchCallbacks(FilteredCallbacks):
self.portEntry = self.xml.get_widget("searchPortEntry")
self.targetEntry = self.xml.get_widget("searchTargetEntry")
self.LUNEntry = self.xml.get_widget("searchLUNEntry")
- self.WWIDEntry = self.xml.get_widget("searchWWIDEntry")
+ self.IDEntry = self.xml.get_widget("searchIDEntry")

# When these entries are changed, we need to redo the filtering.
# If we don't do filter-as-you-type, we'd need a Search/Clear button.
self.portEntry.connect("changed", lambda entry: self.model.refilter())
self.targetEntry.connect("changed", lambda entry: self.model.refilter())
self.LUNEntry.connect("changed", lambda entry: self.model.refilter())
- self.WWIDEntry.connect("changed", lambda entry: self.model.refilter())
+ self.IDEntry.connect("changed", lambda entry: self.model.refilter())

def isMember(self, info):
return True
@@ -312,9 +312,9 @@ class SearchCallbacks(FilteredCallbacks):

def _visible_by_wwid(self, model, iter, view):
# FIXME: make this support globs, etc.
- entered = self.WWIDEntry.get_text()
+ entered = self.IDEntry.get_text()

- return entered != "" and model.get_value(iter, WWID_COL).find(entered) != -1
+ return entered != "" and model.get_value(iter, ID_COL).find(entered) != -1

class NotebookPage(object):
def __init__(self, store, name, xml, cb):
@@ -439,7 +439,7 @@ class FilterWindow(InstallWindow):
def _makeMPath(self):
np = NotebookPage(self.store, "mpath", self.xml, MPathCallbacks(self.xml))

- np.ds.addColumn(_("WWID"), WWID_COL)
+ np.ds.addColumn(_("Identifier"), ID_COL)
np.ds.addColumn(_("Capacity"), CAPACITY_COL)
np.ds.addColumn(_("Vendor"), VENDOR_COL)
np.ds.addColumn(_("Interconnect"), INTERCONNECT_COL)
@@ -449,7 +449,7 @@ class FilterWindow(InstallWindow):
def _makeOther(self):
np = NotebookPage(self.store, "other", self.xml, OtherCallbacks(self.xml))

- np.ds.addColumn(_("WWID"), WWID_COL)
+ np.ds.addColumn(_("Identifier"), ID_COL)
np.ds.addColumn(_("Capacity"), CAPACITY_COL)
np.ds.addColumn(_("Vendor"), VENDOR_COL)
np.ds.addColumn(_("Interconnect"), INTERCONNECT_COL)
@@ -464,7 +464,7 @@ class FilterWindow(InstallWindow):
np.ds.addColumn(_("Vendor"), VENDOR_COL)
np.ds.addColumn(_("Interconnect"), INTERCONNECT_COL, displayed=False)
np.ds.addColumn(_("Serial Number"), SERIAL_COL, displayed=False)
- np.ds.addColumn(_("WWID"), WWID_COL)
+ np.ds.addColumn(_("Identifier"), ID_COL)
np.ds.addColumn(_("Port"), PORT_COL)
np.ds.addColumn(_("Target"), TARGET_COL)
np.ds.addColumn(_("LUN"), LUN_COL)
diff --git a/ui/filter.glade b/ui/filter.glade
index 67c8264..22ab102 100644
--- a/ui/filter.glade
+++ b/ui/filter.glade
@@ -187,7 +187,7 @@
<property name="visible">True</property>
<property name="items" translatable="yes">Interconnect
Vendor
-WWID</property>
+Identifier</property>
<property name="add_tearoffs">False</property>
<property name="has_frame">True</property>
<property name="focus_on_click">True</property>
@@ -365,7 +365,7 @@ WWID</property>
<child>
<widget class="GtkLabel" id="label32">
<property name="visible">True</property>
- <property name="label" translatable="yes">Show WWIDs that Include:</property>
+ <property name="label" translatable="yes">Show Identifiers that Include:</property>
<property name="use_underline">False</property>
<property name="use_markup">False</property>
<property name="justify">GTK_JUSTIFY_LEFT</property>
@@ -388,7 +388,7 @@ WWID</property>
</child>

<child>
- <widget class="GtkEntry" id="mpathWWIDEntry">
+ <widget class="GtkEntry" id="mpathIDEntry">
<property name="visible">True</property>
<property name="can_focus">True</property>
<property name="editable">True</property>
@@ -540,7 +540,7 @@ WWID</property>
<property name="visible">True</property>
<property name="items" translatable="yes">Interconnect
Vendor
-WWID</property>
+Identifier</property>
<property name="add_tearoffs">False</property>
<property name="has_frame">True</property>
<property name="focus_on_click">True</property>
@@ -718,7 +718,7 @@ WWID</property>
<child>
<widget class="GtkLabel" id="label40">
<property name="visible">True</property>
- <property name="label" translatable="yes">Show WWIDs that Include:</property>
+ <property name="label" translatable="yes">Show Identifiers that Include:</property>
<property name="use_underline">False</property>
<property name="use_markup">False</property>
<property name="justify">GTK_JUSTIFY_LEFT</property>
@@ -741,7 +741,7 @@ WWID</property>
</child>

<child>
- <widget class="GtkEntry" id="otherWWIDEntry">
+ <widget class="GtkEntry" id="otherIDEntry">
<property name="visible">True</property>
<property name="can_focus">True</property>
<property name="editable">True</property>
@@ -892,7 +892,7 @@ WWID</property>
<widget class="GtkComboBoxEntry" id="searchFilterBy">
<property name="visible">True</property>
<property name="items" translatable="yes">Port / Target / LUN
-Target WWID</property>
+Target Identifier</property>
<property name="add_tearoffs">False</property>
<property name="has_frame">True</property>
<property name="focus_on_click">True</property>
@@ -1092,7 +1092,7 @@ Target WWID</property>
<child>
<widget class="GtkLabel" id="label21">
<property name="visible">True</property>
- <property name="label" translatable="yes">WWID:</property>
+ <property name="label" translatable="yes">Identifier:</property>
<property name="use_underline">False</property>
<property name="use_markup">False</property>
<property name="justify">GTK_JUSTIFY_LEFT</property>
@@ -1115,7 +1115,7 @@ Target WWID</property>
</child>

<child>
- <widget class="GtkEntry" id="searchWWIDEntry">
+ <widget class="GtkEntry" id="searchIDEntry">
<property name="visible">True</property>
<property name="can_focus">True</property>
<property name="editable">True</property>
--
1.6.5.1

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
 
Old 02-03-2010, 07:56 PM
David Lehman
 
Default Rename the WWID column to Identifier.

This whole set looks okay to me, although I am largely taking your word
for it when it comes to patch 4.

Dave

On Wed, 2010-02-03 at 14:36 -0500, Chris Lumens wrote:
> Since we're stuffing more things into this column than just WWIDS (paths on
> iscsi and s390 stuff soon, at least) the column header needs to be changed
> to match.
> ---
> iw/filter_gui.py | 28 ++++++++++++++--------------
> ui/filter.glade | 18 +++++++++---------
> 2 files changed, 23 insertions(+), 23 deletions(-)
>
> diff --git a/iw/filter_gui.py b/iw/filter_gui.py
> index 8490c26..5f3e3a6 100644
> --- a/iw/filter_gui.py
> +++ b/iw/filter_gui.py
> @@ -44,7 +44,7 @@ CAPACITY_COL = 5
> VENDOR_COL = 6
> INTERCONNECT_COL = 7
> SERIAL_COL = 8
> -WWID_COL = 9
> +ID_COL = 9
> PATHS_COL = 10
> PORT_COL = 11
> TARGET_COL = 12
> @@ -181,12 +181,12 @@ class MPathCallbacks(FilteredCallbacks):
>
> self.vendorEntry = self.xml.get_widget("mpathVendorEntry")
> self.interconnectEntry = self.xml.get_widget("mpathInterconnectEntry")
> - self.WWIDEntry = self.xml.get_widget("mpathWWIDEntry")
> + self.IDEntry = self.xml.get_widget("mpathIDEntry")
>
> self.vendorEntry.connect("changed", lambda entry: self.model.refilter())
> self.vendorEntry.connect("realize", self._populateUI)
> self.interconnectEntry.connect("changed", lambda entry: self.model.refilter())
> - self.WWIDEntry.connect("changed", lambda entry: self.model.refilter())
> + self.IDEntry.connect("changed", lambda entry: self.model.refilter())
>
> def addToUI(self, tuple):
> if not tuple[VENDOR_COL] in self._vendors:
> @@ -233,9 +233,9 @@ class MPathCallbacks(FilteredCallbacks):
>
> def _visible_by_wwid(self, model, iter, view):
> # FIXME: make this support globs, etc.
> - entered = self.WWIDEntry.get_text()
> + entered = self.IDEntry.get_text()
>
> - return entered != "" and model.get_value(iter, WWID_COL).find(entered) != -1
> + return entered != "" and model.get_value(iter, ID_COL).find(entered) != -1
>
> class OtherCallbacks(MPathCallbacks):
> def __init__(self, *args, **kwargs):
> @@ -249,12 +249,12 @@ class OtherCallbacks(MPathCallbacks):
>
> self.vendorEntry = self.xml.get_widget("otherVendorEntry")
> self.interconnectEntry = self.xml.get_widget("otherInterconnectEntry")
> - self.WWIDEntry = self.xml.get_widget("otherWWIDEntry")
> + self.IDEntry = self.xml.get_widget("otherIDEntry")
>
> self.vendorEntry.connect("changed", lambda entry: self.model.refilter())
> self.vendorEntry.connect("realize", self._populateUI)
> self.interconnectEntry.connect("changed", lambda entry: self.model.refilter())
> - self.WWIDEntry.connect("changed", lambda entry: self.model.refilter())
> + self.IDEntry.connect("changed", lambda entry: self.model.refilter())
>
> def isMember(self, info):
> return info and isOther(info)
> @@ -273,14 +273,14 @@ class SearchCallbacks(FilteredCallbacks):
> self.portEntry = self.xml.get_widget("searchPortEntry")
> self.targetEntry = self.xml.get_widget("searchTargetEntry")
> self.LUNEntry = self.xml.get_widget("searchLUNEntry")
> - self.WWIDEntry = self.xml.get_widget("searchWWIDEntry")
> + self.IDEntry = self.xml.get_widget("searchIDEntry")
>
> # When these entries are changed, we need to redo the filtering.
> # If we don't do filter-as-you-type, we'd need a Search/Clear button.
> self.portEntry.connect("changed", lambda entry: self.model.refilter())
> self.targetEntry.connect("changed", lambda entry: self.model.refilter())
> self.LUNEntry.connect("changed", lambda entry: self.model.refilter())
> - self.WWIDEntry.connect("changed", lambda entry: self.model.refilter())
> + self.IDEntry.connect("changed", lambda entry: self.model.refilter())
>
> def isMember(self, info):
> return True
> @@ -312,9 +312,9 @@ class SearchCallbacks(FilteredCallbacks):
>
> def _visible_by_wwid(self, model, iter, view):
> # FIXME: make this support globs, etc.
> - entered = self.WWIDEntry.get_text()
> + entered = self.IDEntry.get_text()
>
> - return entered != "" and model.get_value(iter, WWID_COL).find(entered) != -1
> + return entered != "" and model.get_value(iter, ID_COL).find(entered) != -1
>
> class NotebookPage(object):
> def __init__(self, store, name, xml, cb):
> @@ -439,7 +439,7 @@ class FilterWindow(InstallWindow):
> def _makeMPath(self):
> np = NotebookPage(self.store, "mpath", self.xml, MPathCallbacks(self.xml))
>
> - np.ds.addColumn(_("WWID"), WWID_COL)
> + np.ds.addColumn(_("Identifier"), ID_COL)
> np.ds.addColumn(_("Capacity"), CAPACITY_COL)
> np.ds.addColumn(_("Vendor"), VENDOR_COL)
> np.ds.addColumn(_("Interconnect"), INTERCONNECT_COL)
> @@ -449,7 +449,7 @@ class FilterWindow(InstallWindow):
> def _makeOther(self):
> np = NotebookPage(self.store, "other", self.xml, OtherCallbacks(self.xml))
>
> - np.ds.addColumn(_("WWID"), WWID_COL)
> + np.ds.addColumn(_("Identifier"), ID_COL)
> np.ds.addColumn(_("Capacity"), CAPACITY_COL)
> np.ds.addColumn(_("Vendor"), VENDOR_COL)
> np.ds.addColumn(_("Interconnect"), INTERCONNECT_COL)
> @@ -464,7 +464,7 @@ class FilterWindow(InstallWindow):
> np.ds.addColumn(_("Vendor"), VENDOR_COL)
> np.ds.addColumn(_("Interconnect"), INTERCONNECT_COL, displayed=False)
> np.ds.addColumn(_("Serial Number"), SERIAL_COL, displayed=False)
> - np.ds.addColumn(_("WWID"), WWID_COL)
> + np.ds.addColumn(_("Identifier"), ID_COL)
> np.ds.addColumn(_("Port"), PORT_COL)
> np.ds.addColumn(_("Target"), TARGET_COL)
> np.ds.addColumn(_("LUN"), LUN_COL)
> diff --git a/ui/filter.glade b/ui/filter.glade
> index 67c8264..22ab102 100644
> --- a/ui/filter.glade
> +++ b/ui/filter.glade
> @@ -187,7 +187,7 @@
> <property name="visible">True</property>
> <property name="items" translatable="yes">Interconnect
> Vendor
> -WWID</property>
> +Identifier</property>
> <property name="add_tearoffs">False</property>
> <property name="has_frame">True</property>
> <property name="focus_on_click">True</property>
> @@ -365,7 +365,7 @@ WWID</property>
> <child>
> <widget class="GtkLabel" id="label32">
> <property name="visible">True</property>
> - <property name="label" translatable="yes">Show WWIDs that Include:</property>
> + <property name="label" translatable="yes">Show Identifiers that Include:</property>
> <property name="use_underline">False</property>
> <property name="use_markup">False</property>
> <property name="justify">GTK_JUSTIFY_LEFT</property>
> @@ -388,7 +388,7 @@ WWID</property>
> </child>
>
> <child>
> - <widget class="GtkEntry" id="mpathWWIDEntry">
> + <widget class="GtkEntry" id="mpathIDEntry">
> <property name="visible">True</property>
> <property name="can_focus">True</property>
> <property name="editable">True</property>
> @@ -540,7 +540,7 @@ WWID</property>
> <property name="visible">True</property>
> <property name="items" translatable="yes">Interconnect
> Vendor
> -WWID</property>
> +Identifier</property>
> <property name="add_tearoffs">False</property>
> <property name="has_frame">True</property>
> <property name="focus_on_click">True</property>
> @@ -718,7 +718,7 @@ WWID</property>
> <child>
> <widget class="GtkLabel" id="label40">
> <property name="visible">True</property>
> - <property name="label" translatable="yes">Show WWIDs that Include:</property>
> + <property name="label" translatable="yes">Show Identifiers that Include:</property>
> <property name="use_underline">False</property>
> <property name="use_markup">False</property>
> <property name="justify">GTK_JUSTIFY_LEFT</property>
> @@ -741,7 +741,7 @@ WWID</property>
> </child>
>
> <child>
> - <widget class="GtkEntry" id="otherWWIDEntry">
> + <widget class="GtkEntry" id="otherIDEntry">
> <property name="visible">True</property>
> <property name="can_focus">True</property>
> <property name="editable">True</property>
> @@ -892,7 +892,7 @@ WWID</property>
> <widget class="GtkComboBoxEntry" id="searchFilterBy">
> <property name="visible">True</property>
> <property name="items" translatable="yes">Port / Target / LUN
> -Target WWID</property>
> +Target Identifier</property>
> <property name="add_tearoffs">False</property>
> <property name="has_frame">True</property>
> <property name="focus_on_click">True</property>
> @@ -1092,7 +1092,7 @@ Target WWID</property>
> <child>
> <widget class="GtkLabel" id="label21">
> <property name="visible">True</property>
> - <property name="label" translatable="yes">WWID:</property>
> + <property name="label" translatable="yes">Identifier:</property>
> <property name="use_underline">False</property>
> <property name="use_markup">False</property>
> <property name="justify">GTK_JUSTIFY_LEFT</property>
> @@ -1115,7 +1115,7 @@ Target WWID</property>
> </child>
>
> <child>
> - <widget class="GtkEntry" id="searchWWIDEntry">
> + <widget class="GtkEntry" id="searchIDEntry">
> <property name="visible">True</property>
> <property name="can_focus">True</property>
> <property name="editable">True</property>


_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
 

Thread Tools




All times are GMT. The time now is 01:48 PM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org