Sat, 12 Feb 2011 12:53:41 +0100
Added a dialog to move a global method.
7
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
1 | # -*- coding: utf-8 -*- |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
2 | |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
3 | # Copyright (c) 2011 Detlev Offenbach <detlev@die-offenbachs.de> |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
4 | # |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
5 | |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
6 | """ |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
7 | Module implementing the Move Method dialog. |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
8 | """ |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
9 | |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
10 | from PyQt4.QtCore import pyqtSlot |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
11 | from PyQt4.QtGui import QDialogButtonBox, QAbstractButton |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
12 | |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
13 | from Ui_MoveMethodDialog import Ui_MoveMethodDialog |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
14 | from RefactoringDialogBase import RefactoringDialogBase |
5ecf9ef23d39
Added the 'move' 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:
7
diff
changeset
|
16 | |
7
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
17 | class MoveMethodDialog(RefactoringDialogBase, Ui_MoveMethodDialog): |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
18 | """ |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
19 | Class implementing the Move Method dialog. |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
20 | """ |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
21 | def __init__(self, refactoring, title, mover, parent=None): |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
22 | """ |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
23 | Constructor |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
24 | |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
25 | @param refactoring reference to the main refactoring object |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
26 | (Refactoring) |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
27 | @param title title of the dialog (string) |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
28 | @param mover reference to the mover object |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
29 | (rope.refactor.move.MoveMethod) |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
30 | @param parent reference to the parent widget (QWidget) |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
31 | """ |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
32 | RefactoringDialogBase.__init__(self, refactoring, title, parent) |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
33 | self.setupUi(self) |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
34 | |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
35 | self.__mover = mover |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
36 | |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
37 | self.__okButton = self.buttonBox.button(QDialogButtonBox.Ok) |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
38 | self.__okButton.setEnabled(False) |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
39 | self.__previewButton = self.buttonBox.addButton( |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
40 | self.trUtf8("Preview"), QDialogButtonBox.ActionRole) |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
41 | self.__previewButton.setDefault(True) |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
42 | |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
43 | self.methodEdit.setText(self.__mover.get_method_name()) |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
44 | self.methodEdit.selectAll() |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
45 | |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
46 | @pyqtSlot(str) |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
47 | def on_attributeEdit_textChanged(self, text): |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
48 | """ |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
49 | Private slot to react to changes of the attribute. |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
50 | |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
51 | @param text text entered into the edit (string) |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
52 | """ |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
53 | self.__updateUI() |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
54 | |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
55 | @pyqtSlot(str) |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
56 | def on_methodEdit_textChanged(self, text): |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
57 | """ |
26
b02d1f37b55e
Added a dialog to move a global method.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
20
diff
changeset
|
58 | Private slot to react to changes of the method. |
7
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
59 | |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
60 | @param text text entered into the edit (string) |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
61 | """ |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
62 | self.__updateUI() |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
63 | |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
64 | @pyqtSlot(QAbstractButton) |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
65 | def on_buttonBox_clicked(self, button): |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
66 | """ |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
67 | Private slot to act on the button pressed. |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
68 | |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
69 | @param button reference to the button pressed (QAbstractButton) |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
70 | """ |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
71 | if button == self.__previewButton: |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
72 | self.previewChanges() |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
73 | elif button == self.__okButton: |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
74 | self.applyChanges() |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
75 | |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
76 | def __updateUI(self): |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
77 | """ |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
78 | Private method to perform various UI updates. |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
79 | """ |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
80 | self.__okButton.setEnabled( |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
81 | self.attributeEdit.text() != "" and \ |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
82 | self.methodEdit.text() != "") |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
83 | |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
84 | def _calculateChanges(self, handle): |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
85 | """ |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
86 | Protected method to calculate the changes. |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
87 | |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
88 | @param handle reference to the task handle |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
89 | (rope.base.taskhandle.TaskHandle) |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
90 | @return reference to the Changes object (rope.base.change.ChangeSet) |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
91 | """ |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
92 | try: |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
93 | newName = self.methodEdit.text() |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
94 | if not newName: |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
95 | newName = None |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
96 | changes = self.__mover.get_changes(self.attributeEdit.text(), |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
97 | newName, task_handle=handle) |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
98 | return changes |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
99 | except Exception as err: |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
100 | self._refactoring.handleRopeError(err, self._title, handle) |
5ecf9ef23d39
Added the 'move' refactoring functions.
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
101 | return None |