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