Sat, 23 Feb 2013 15:28:39 +0100
Modified the code to do lazy import.
11
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
1 | # -*- coding: utf-8 -*- |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
2 | |
43
39924831c795
Updated copyright for 2013.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
35
diff
changeset
|
3 | # Copyright (c) 2010 - 2013 Detlev Offenbach <detlev@die-offenbachs.de> |
11
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
4 | # |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
5 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
6 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
7 | Module implementing the Change Signature dialog. |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
8 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
9 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
10 | import copy |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
11 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
12 | from PyQt4.QtCore import pyqtSlot, Qt |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
13 | from PyQt4.QtGui import QDialog, QDialogButtonBox, QListWidgetItem, \ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
14 | QAbstractButton |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
15 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
16 | import rope.refactor.change_signature |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
17 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
18 | from Ui_ChangeSignatureDialog import Ui_ChangeSignatureDialog |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
19 | from RefactoringDialogBase import RefactoringDialogBase |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
20 | |
20
83b71483e198
Made the code PEP-8 compliant.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
11
diff
changeset
|
21 | |
11
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
22 | class ChangeSignatureDialog(RefactoringDialogBase, Ui_ChangeSignatureDialog): |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
23 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
24 | Class implementing the Change Signature dialog. |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
25 | """ |
20
83b71483e198
Made the code PEP-8 compliant.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
11
diff
changeset
|
26 | NameRole = Qt.UserRole |
11
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
27 | IsAddedRole = Qt.UserRole + 1 |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
28 | DefaultRole = Qt.UserRole + 2 |
20
83b71483e198
Made the code PEP-8 compliant.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
11
diff
changeset
|
29 | ValueRole = Qt.UserRole + 3 |
11
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
30 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
31 | def __init__(self, refactoring, title, changer, parent=None): |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
32 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
33 | Constructor |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
34 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
35 | @param refactoring reference to the main refactoring object |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
36 | (Refactoring) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
37 | @param title title of the dialog (string) |
20
83b71483e198
Made the code PEP-8 compliant.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
11
diff
changeset
|
38 | @param changer reference to the signature changer object |
11
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
39 | (rope.refactor.change_signature.ChangeSignature) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
40 | @param parent reference to the parent widget (QWidget) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
41 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
42 | RefactoringDialogBase.__init__(self, refactoring, title, parent) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
43 | self.setupUi(self) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
44 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
45 | self.__signature = changer |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
46 | self.__definition_info = self.__signature.get_args() |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
47 | self.__to_be_removed = [] |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
48 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
49 | self.__okButton = self.buttonBox.button(QDialogButtonBox.Ok) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
50 | self.__previewButton = self.buttonBox.addButton( |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
51 | self.trUtf8("Preview"), QDialogButtonBox.ActionRole) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
52 | self.__previewButton.setDefault(True) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
53 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
54 | # populate the parameters list |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
55 | for arg, default in self.__definition_info: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
56 | if default is None: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
57 | itm = QListWidgetItem(arg, self.parameterList) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
58 | else: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
59 | itm = QListWidgetItem("{0} = {1}".format(arg, default), |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
60 | self.parameterList) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
61 | itm.setData(ChangeSignatureDialog.NameRole, arg) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
62 | itm.setData(ChangeSignatureDialog.IsAddedRole, False) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
63 | itm.setData(ChangeSignatureDialog.DefaultRole, None) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
64 | itm.setData(ChangeSignatureDialog.ValueRole, None) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
65 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
66 | if self.parameterList.count(): |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
67 | self.parameterList.setCurrentRow(0) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
68 | else: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
69 | self.on_parameterList_currentRowChanged(-1) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
70 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
71 | @pyqtSlot(int) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
72 | def on_parameterList_currentRowChanged(self, currentRow): |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
73 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
74 | Private slot called, when the current row is changed. |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
75 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
76 | @param currentRow index of the current row (integer) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
77 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
78 | if currentRow == -1: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
79 | self.upButton.setEnabled(False) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
80 | self.downButton.setEnabled(False) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
81 | self.removeButton.setEnabled(False) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
82 | else: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
83 | maxIndex = self.parameterList.count() - 1 |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
84 | self.upButton.setEnabled(currentRow != 0) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
85 | self.downButton.setEnabled(currentRow != maxIndex) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
86 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
87 | @pyqtSlot() |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
88 | def on_upButton_clicked(self): |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
89 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
90 | Private slot called to move the selected item up in the list. |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
91 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
92 | row = self.parameterList.currentRow() |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
93 | if row == 0: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
94 | # we're already at the top |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
95 | return |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
96 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
97 | itm = self.parameterList.takeItem(row) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
98 | self.parameterList.insertItem(row - 1, itm) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
99 | self.parameterList.setCurrentItem(itm) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
100 | if row == 1: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
101 | self.upButton.setEnabled(False) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
102 | else: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
103 | self.upButton.setEnabled(True) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
104 | self.downButton.setEnabled(True) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
105 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
106 | @pyqtSlot() |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
107 | def on_downButton_clicked(self): |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
108 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
109 | Private slot called to move the selected item down in the list. |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
110 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
111 | rows = self.parameterList.count() |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
112 | row = self.parameterList.currentRow() |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
113 | if row == rows - 1: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
114 | # we're already at the end |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
115 | return |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
116 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
117 | itm = self.parameterList.takeItem(row) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
118 | self.parameterList.insertItem(row + 1, itm) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
119 | self.parameterList.setCurrentItem(itm) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
120 | self.upButton.setEnabled(True) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
121 | if row == rows - 2: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
122 | self.downButton.setEnabled(False) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
123 | else: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
124 | self.downButton.setEnabled(True) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
125 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
126 | @pyqtSlot() |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
127 | def on_removeButton_clicked(self): |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
128 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
129 | Private slot to remove a parameter. |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
130 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
131 | itm = self.parameterList.takeItem(self.parameterList.currentRow()) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
132 | self.__to_be_removed.append(itm) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
133 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
134 | @pyqtSlot() |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
135 | def on_addButton_clicked(self): |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
136 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
137 | Private slot to add a new parameter. |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
138 | """ |
46
a726d19facfc
Modified the code to do lazy import.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
43
diff
changeset
|
139 | from AddParameterDialog import AddParameterDialog |
11
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
140 | dlg = AddParameterDialog(self) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
141 | if dlg.exec_() == QDialog.Accepted: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
142 | name, default, value = dlg.getData() |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
143 | if default: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
144 | s = "%s = %s" % (name, default) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
145 | else: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
146 | s = name |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
147 | itm = QListWidgetItem(s) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
148 | itm.setData(ChangeSignatureDialog.NameRole, name) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
149 | itm.setData(ChangeSignatureDialog.IsAddedRole, True) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
150 | if default: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
151 | itm.setData(ChangeSignatureDialog.DefaultRole, default) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
152 | else: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
153 | itm.setData(ChangeSignatureDialog.DefaultRole, None) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
154 | if value: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
155 | itm.setData(ChangeSignatureDialog.ValueRole, value) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
156 | else: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
157 | itm.setData(ChangeSignatureDialog.ValueRole, None) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
158 | if self.parameterList.count(): |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
159 | self.parameterList.insertItem( |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
160 | self.parameterList.currentRow() + 1, itm) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
161 | else: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
162 | self.parameterList.addItem(itm) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
163 | self.parameterList.setCurrentItem(itm) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
164 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
165 | @pyqtSlot(QAbstractButton) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
166 | def on_buttonBox_clicked(self, button): |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
167 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
168 | Private slot to act on the button pressed. |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
169 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
170 | @param button reference to the button pressed (QAbstractButton) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
171 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
172 | if button == self.__previewButton: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
173 | self.previewChanges() |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
174 | elif button == self.__okButton: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
175 | self.applyChanges() |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
176 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
177 | def __getParameterIndex(self, definition_info, name): |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
178 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
179 | Private method to calculate the index of the given paramter. |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
180 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
181 | @param definition_info object containing the method definition |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
182 | @param name parameter name (string) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
183 | @return index of the parameter (integer) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
184 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
185 | for index, pair in enumerate(definition_info): |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
186 | if pair[0] == name: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
187 | return index |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
188 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
189 | def _calculateChanges(self, handle): |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
190 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
191 | Protected method to calculate the changes. |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
192 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
193 | @param handle reference to the task handle |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
194 | (rope.base.taskhandle.TaskHandle) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
195 | @return reference to the Changes object (rope.base.change.ChangeSet) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
196 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
197 | changers = [] |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
198 | definition_info = copy.deepcopy(self.__definition_info) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
199 | for itm in self.__to_be_removed: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
200 | if itm.data(ChangeSignatureDialog.IsAddedRole): |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
201 | continue |
20
83b71483e198
Made the code PEP-8 compliant.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
11
diff
changeset
|
202 | index = self.__getParameterIndex(definition_info, |
11
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
203 | itm.data(ChangeSignatureDialog.NameRole)) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
204 | remover = rope.refactor.change_signature.ArgumentRemover(index) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
205 | changers.append(remover) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
206 | del definition_info[index] |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
207 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
208 | for index in range(self.parameterList.count()): |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
209 | itm = self.parameterList.item(index) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
210 | if itm.data(ChangeSignatureDialog.IsAddedRole): |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
211 | name = itm.data(ChangeSignatureDialog.NameRole) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
212 | default = itm.data(ChangeSignatureDialog.DefaultRole) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
213 | value = itm.data(ChangeSignatureDialog.ValueRole) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
214 | adder = rope.refactor.change_signature.ArgumentAdder( |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
215 | index, name, default, value) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
216 | changers.append(adder) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
217 | try: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
218 | definition_info.insert(index, (name, default)) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
219 | except Exception as err: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
220 | self._refactoring.handleRopeError(err, self._title) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
221 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
222 | new_ordering = [] |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
223 | for row in range(self.parameterList.count()): |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
224 | itm = self.parameterList.item(row) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
225 | name = itm.data(ChangeSignatureDialog.NameRole) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
226 | new_ordering.append( |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
227 | self.__getParameterIndex(definition_info, name)) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
228 | autodef = self.autodefEdit.text() |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
229 | if not autodef: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
230 | autodef = None |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
231 | changers.append(rope.refactor.change_signature.ArgumentReorderer( |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
232 | new_ordering, autodef=autodef)) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
233 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
234 | try: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
235 | changes = self.__signature.get_changes( |
20
83b71483e198
Made the code PEP-8 compliant.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
11
diff
changeset
|
236 | changers, in_hierarchy=self.hierarchyCheckBox.isChecked(), |
11
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
237 | task_handle=handle) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
238 | return changes |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
239 | except Exception as err: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
240 | self._refactoring.handleRopeError(err, self._title, handle) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
241 | return None |