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 Inline Argument Default 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 | from PyQt4.QtCore import pyqtSlot, Qt |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
11 | from PyQt4.QtGui import QDialogButtonBox, QListWidgetItem, QAbstractButton |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
12 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
13 | from Ui_InlineArgumentDefaultDialog import Ui_InlineArgumentDefaultDialog |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
14 | from RefactoringDialogBase import RefactoringDialogBase |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
15 | |
20
83b71483e198
Made the code PEP-8 compliant.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
11
diff
changeset
|
16 | |
83b71483e198
Made the code PEP-8 compliant.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
11
diff
changeset
|
17 | class InlineArgumentDefaultDialog(RefactoringDialogBase, |
11
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
18 | Ui_InlineArgumentDefaultDialog): |
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 | Class implementing the Inline Argument Default dialog. |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
21 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
22 | NameRole = Qt.UserRole |
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 | def __init__(self, refactoring, title, changer, parent=None): |
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 | Constructor |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
27 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
28 | @param refactoring reference to the main refactoring object |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
29 | (Refactoring) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
30 | @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
|
31 | @param changer reference to the signature changer object |
11
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
32 | (rope.refactor.change_signature.ChangeSignature) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
33 | @param parent reference to the parent widget (QWidget) |
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 | RefactoringDialogBase.__init__(self, refactoring, title, parent) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
36 | self.setupUi(self) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
37 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
38 | self.__signature = changer |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
39 | self.__definition_info = self.__signature.get_args() |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
40 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
41 | self.__okButton = self.buttonBox.button(QDialogButtonBox.Ok) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
42 | self.__okButton.setEnabled(False) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
43 | self.__previewButton = self.buttonBox.addButton( |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
44 | self.trUtf8("Preview"), QDialogButtonBox.ActionRole) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
45 | self.__previewButton.setDefault(True) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
46 | self.__previewButton.setEnabled(False) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
47 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
48 | # populate the parameters list |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
49 | for arg, default in self.__definition_info: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
50 | if default is not None: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
51 | itm = QListWidgetItem("{0} = {1}".format(arg, default), |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
52 | self.parameterList) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
53 | itm.setData(InlineArgumentDefaultDialog.NameRole, arg) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
54 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
55 | @pyqtSlot() |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
56 | def on_parameterList_itemSelectionChanged(self): |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
57 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
58 | Private slot called, when the selection changes. |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
59 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
60 | self.__okButton.setEnabled( |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
61 | len(self.parameterList.selectedItems()) > 0) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
62 | self.__previewButton.setEnabled( |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
63 | len(self.parameterList.selectedItems()) > 0) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
64 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
65 | @pyqtSlot(QAbstractButton) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
66 | def on_buttonBox_clicked(self, button): |
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 | Private slot to act on the button pressed. |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
69 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
70 | @param button reference to the button pressed (QAbstractButton) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
71 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
72 | if button == self.__previewButton: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
73 | self.previewChanges() |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
74 | elif button == self.__okButton: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
75 | self.applyChanges() |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
76 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
77 | def __getParameterIndex(self, definition_info, name): |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
78 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
79 | Private method to calculate the index of the given paramter. |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
80 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
81 | @param definition_info object containing the method definition |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
82 | @param name parameter name (string) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
83 | @return index of the parameter (integer) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
84 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
85 | for index, pair in enumerate(definition_info): |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
86 | if pair[0] == name: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
87 | return index |
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 | def _calculateChanges(self, handle): |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
90 | """ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
91 | Protected method to calculate the changes. |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
92 | |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
93 | @param handle reference to the task handle |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
94 | (rope.base.taskhandle.TaskHandle) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
95 | @return reference to the Changes object (rope.base.change.ChangeSet) |
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 | items = self.parameterList.selectedItems() |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
98 | if len(items) > 0: |
46
a726d19facfc
Modified the code to do lazy import.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
43
diff
changeset
|
99 | import rope.refactor.change_signature |
a726d19facfc
Modified the code to do lazy import.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
43
diff
changeset
|
100 | |
11
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
101 | itm = items[0] |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
102 | name = itm.data(InlineArgumentDefaultDialog.NameRole) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
103 | index = self.__getParameterIndex(self.__definition_info, name) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
104 | try: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
105 | inliner = \ |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
106 | rope.refactor.change_signature.ArgumentDefaultInliner( |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
107 | index) |
20
83b71483e198
Made the code PEP-8 compliant.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
11
diff
changeset
|
108 | changes = self.__signature.get_changes([inliner], |
11
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
109 | task_handle=handle) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
110 | return changes |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
111 | except Exception as err: |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
112 | self._refactoring.handleRopeError(err, self._title, handle) |
562f9758d2e1
Added these refactoring functions:
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
113 | return None |