author | Claus Gittinger <cg@exept.de> |
Fri, 15 Jun 2018 04:00:37 +0200 | |
changeset 18220 | d1ebaddf1100 |
parent 18147 | 03d5bfe1c546 |
child 18279 | afdae9cde5cd |
permissions | -rw-r--r-- |
18094 | 1 |
"{ Encoding: utf8 }" |
2 |
||
2626 | 3 |
" |
4 |
COPYRIGHT (c) 2000 by eXept Software AG |
|
11453 | 5 |
All Rights Reserved |
2626 | 6 |
|
7 |
This software is furnished under a license and may be used |
|
8 |
only in accordance with the terms of that license and with the |
|
9 |
inclusion of the above copyright notice. This software may not |
|
10 |
be provided or otherwise made available to, or used by, any |
|
11 |
other person. No title to or ownership of the software is |
|
12 |
hereby transferred. |
|
13 |
" |
|
2650
447c99ac6f76
oops - inspect & browse menu was broken.
Claus Gittinger <cg@exept.de>
parents:
2626
diff
changeset
|
14 |
"{ Package: 'stx:libtool' }" |
447c99ac6f76
oops - inspect & browse menu was broken.
Claus Gittinger <cg@exept.de>
parents:
2626
diff
changeset
|
15 |
|
15167 | 16 |
"{ NameSpace: Smalltalk }" |
17 |
||
2221 | 18 |
ApplicationModel subclass:#VersionDiffBrowser |
2856 | 19 |
instanceVariableNames:'classChangeSet diffTextView boxAVisible boxBVisible boxMVisible |
20 |
classIfSingleClassDiff versionAIfSingleClassDiff |
|
3147 | 21 |
versionBIfSingleClassDiff changedLabelHolder diffTextLabelA |
11596
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
22 |
diffTextLabelB methodText methodsChanged methodsChangedFiltered |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
23 |
methodsChangedSelection methodsOnlyInA methodsOnlyInAFiltered |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
24 |
methodsOnlyInASelection methodsOnlyInB methodsOnlyInBFiltered |
5182 | 25 |
methodsOnlyInBSelection onlyInALabelHolder onlyInBLabelHolder |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
26 |
classHolder versionAHolder versionBHolder infoHolder |
11596
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
27 |
includeExtensionsHolder canIncludeExtensionsHolder |
14076 | 28 |
includeCategoryChangesHolder includeVersionMethodsHolder |
15621 | 29 |
isMultipleVersionBrowser versionInfoList versionList |
17827 | 30 |
versionEntriesList selectedVersionHolder symbolicVersionList |
17341 | 31 |
selectedSymbolicVersionHolder symbolicToVersionMapping |
17347 | 32 |
filteredClasses filteredMethods filteredSelectors |
17827 | 33 |
classPatternFilters selectorPatternFilters |
34 |
selectedVersionIndexHolder' |
|
17931 | 35 |
classVariableNames:'RememberedFilters LastSearchString' |
2221 | 36 |
poolDictionaries:'' |
37 |
category:'Interface-Browsers' |
|
38 |
! |
|
39 |
||
2497 | 40 |
HierarchicalItem subclass:#ClassChangeSet |
13137 | 41 |
instanceVariableNames:'class1BeingCompared class2BeingCompared labelA labelB diffSet |
42 |
versionA versionB' |
|
2497 | 43 |
classVariableNames:'' |
44 |
poolDictionaries:'' |
|
45 |
privateIn:VersionDiffBrowser |
|
46 |
! |
|
47 |
||
17341 | 48 |
Object subclass:#FilterParameters |
49 |
instanceVariableNames:'filteredMethodNames filteredClassNames filteredSelectors |
|
50 |
filteredClassNameMatchPattern filteredSelectorMatchPattern' |
|
51 |
classVariableNames:'' |
|
52 |
poolDictionaries:'' |
|
53 |
privateIn:VersionDiffBrowser |
|
54 |
! |
|
55 |
||
2626 | 56 |
!VersionDiffBrowser class methodsFor:'documentation'! |
57 |
||
58 |
copyright |
|
59 |
" |
|
60 |
COPYRIGHT (c) 2000 by eXept Software AG |
|
11453 | 61 |
All Rights Reserved |
2626 | 62 |
|
63 |
This software is furnished under a license and may be used |
|
64 |
only in accordance with the terms of that license and with the |
|
65 |
inclusion of the above copyright notice. This software may not |
|
66 |
be provided or otherwise made available to, or used by, any |
|
67 |
other person. No title to or ownership of the software is |
|
68 |
hereby transferred. |
|
69 |
" |
|
70 |
||
71 |
! |
|
72 |
||
73 |
documentation |
|
74 |
" |
|
75 |
This is not yet finished (work in progress). |
|
76 |
||
15621 | 77 |
A quickly hacked up browser to show differences between class versions, |
2626 | 78 |
allowing easy comparison; will also eventually add capabilities |
79 |
to checkIn / load classes into / from the repository. |
|
80 |
||
81 |
[see also:] |
|
82 |
||
83 |
[instance variables:] |
|
84 |
||
85 |
[class variables:] |
|
86 |
" |
|
87 |
! ! |
|
2221 | 88 |
|
89 |
!VersionDiffBrowser class methodsFor:'interface specs'! |
|
90 |
||
91 |
windowSpec |
|
92 |
"This resource specification was automatically generated |
|
93 |
by the UIPainter of ST/X." |
|
94 |
||
95 |
"Do not manually edit this!! If it is corrupted, |
|
96 |
the UIPainter may not be able to read the specification." |
|
97 |
||
98 |
" |
|
99 |
UIPainter new openOnClass:VersionDiffBrowser andSelector:#windowSpec |
|
100 |
VersionDiffBrowser new openInterface:#windowSpec |
|
101 |
VersionDiffBrowser open |
|
102 |
" |
|
103 |
||
104 |
<resource: #canvas> |
|
105 |
||
17397 | 106 |
^ |
107 |
#(FullSpec |
|
108 |
name: windowSpec |
|
109 |
window: |
|
110 |
(WindowSpec |
|
111 |
label: 'Version DiffBrowser' |
|
112 |
name: 'Version DiffBrowser' |
|
113 |
min: (Point 10 10) |
|
114 |
bounds: (Rectangle 0 0 865 504) |
|
115 |
menu: mainMenu |
|
116 |
) |
|
117 |
component: |
|
118 |
(SpecCollection |
|
119 |
collection: ( |
|
120 |
(VariableVerticalPanelSpec |
|
121 |
name: 'VariableVerticalPanel1' |
|
122 |
layout: (LayoutFrame 0 0.0 0 0.0 0 1.0 -25 1.0) |
|
123 |
component: |
|
124 |
(SpecCollection |
|
125 |
collection: ( |
|
126 |
(HorizontalPanelViewSpec |
|
127 |
name: 'TopHorizontalPanel' |
|
128 |
horizontalLayout: fit |
|
129 |
verticalLayout: fit |
|
130 |
horizontalSpace: 3 |
|
131 |
verticalSpace: 3 |
|
132 |
component: |
|
133 |
(SpecCollection |
|
134 |
collection: ( |
|
135 |
(ViewSpec |
|
136 |
name: 'BoxA' |
|
137 |
level: 0 |
|
138 |
visibilityChannel: boxAVisible |
|
139 |
component: |
|
140 |
(SpecCollection |
|
141 |
collection: ( |
|
142 |
(LabelSpec |
|
143 |
label: 'Only in A' |
|
144 |
name: 'OnlyInALabel' |
|
145 |
layout: (LayoutFrame 0 0.0 0 0.0 0 1.0 20 0) |
|
146 |
translateLabel: true |
|
147 |
labelChannel: onlyInALabelHolder |
|
17341 | 148 |
) |
17397 | 149 |
(SequenceViewSpec |
150 |
name: 'ListA' |
|
151 |
layout: (LayoutFrame 0 0.0 20 0.0 0 1.0 -3 1.0) |
|
152 |
model: methodsOnlyInASelection |
|
153 |
menu: menuAHolder |
|
154 |
hasHorizontalScrollBar: true |
|
155 |
hasVerticalScrollBar: true |
|
156 |
miniScrollerHorizontal: true |
|
157 |
doubleClickSelector: methodInADoubleClicked: |
|
158 |
valueChangeSelector: methodsOnlyInASelectionChanged |
|
159 |
useIndex: true |
|
160 |
sequenceList: methodsOnlyInA |
|
161 |
) |
|
162 |
) |
|
163 |
||
164 |
) |
|
165 |
extent: (Point 286 199) |
|
166 |
) |
|
167 |
(ViewSpec |
|
168 |
name: 'BoxM' |
|
169 |
visibilityChannel: boxMVisible |
|
170 |
component: |
|
171 |
(SpecCollection |
|
172 |
collection: ( |
|
173 |
(LabelSpec |
|
174 |
label: 'Changed' |
|
175 |
name: 'ChangedLabel' |
|
176 |
layout: (LayoutFrame 0 0.0 0 0.0 0 1.0 20 0) |
|
177 |
translateLabel: true |
|
178 |
labelChannel: changedLabelHolder |
|
179 |
) |
|
180 |
(SequenceViewSpec |
|
181 |
name: 'ListM' |
|
182 |
layout: (LayoutFrame 0 0.0 20 0.0 0 1.0 -3 1.0) |
|
183 |
model: methodsChangedSelection |
|
184 |
menu: menuMHolder |
|
185 |
hasHorizontalScrollBar: true |
|
186 |
hasVerticalScrollBar: true |
|
187 |
miniScrollerHorizontal: true |
|
188 |
doubleClickSelector: methodInChangedDoubleClicked: |
|
189 |
valueChangeSelector: methodsChangedSelectionChanged |
|
190 |
useIndex: true |
|
191 |
sequenceList: methodsChanged |
|
17341 | 192 |
) |
17397 | 193 |
) |
194 |
||
195 |
) |
|
196 |
extent: (Point 286 199) |
|
197 |
) |
|
198 |
(ViewSpec |
|
199 |
name: 'BoxB' |
|
200 |
visibilityChannel: boxBVisible |
|
201 |
component: |
|
202 |
(SpecCollection |
|
203 |
collection: ( |
|
204 |
(LabelSpec |
|
205 |
label: 'Only in B' |
|
206 |
name: 'OnlyInBLabel' |
|
207 |
layout: (LayoutFrame 0 0.0 0 0.0 0 1.0 20 0) |
|
208 |
translateLabel: true |
|
209 |
labelChannel: onlyInBLabelHolder |
|
17341 | 210 |
) |
17397 | 211 |
(SequenceViewSpec |
212 |
name: 'ListB' |
|
213 |
layout: (LayoutFrame 0 0.0 20 0.0 0 1.0 -3 1.0) |
|
214 |
model: methodsOnlyInBSelection |
|
215 |
menu: menuBHolder |
|
216 |
hasHorizontalScrollBar: true |
|
217 |
hasVerticalScrollBar: true |
|
218 |
miniScrollerHorizontal: true |
|
219 |
doubleClickSelector: methodInBDoubleClicked: |
|
220 |
valueChangeSelector: methodsOnlyInBSelectionChanged |
|
221 |
useIndex: true |
|
222 |
sequenceList: methodsOnlyInB |
|
223 |
) |
|
224 |
) |
|
225 |
||
226 |
) |
|
227 |
extent: (Point 287 199) |
|
17341 | 228 |
) |
17397 | 229 |
) |
230 |
||
231 |
) |
|
232 |
) |
|
233 |
(ViewSpec |
|
234 |
name: 'DiffOrSingleMethodCodeBox' |
|
235 |
component: |
|
236 |
(SpecCollection |
|
237 |
collection: ( |
|
238 |
(ViewSpec |
|
239 |
name: 'diffTextViewBox' |
|
240 |
layout: (LayoutFrame 0 0.0 0 0.0 0 1.0 0 1.0) |
|
241 |
initiallyInvisible: true |
|
242 |
component: |
|
243 |
(SpecCollection |
|
244 |
collection: ( |
|
245 |
(LabelSpec |
|
246 |
label: 'A' |
|
247 |
name: 'DiffTextLabelA' |
|
248 |
layout: (LayoutFrame 0 0 0 0 0 0.5 20 0) |
|
249 |
translateLabel: true |
|
250 |
labelChannel: diffTextLabelA |
|
251 |
) |
|
252 |
(LabelSpec |
|
253 |
label: 'B' |
|
254 |
name: 'DiffTextLabelB' |
|
255 |
layout: (LayoutFrame 0 0.5 0 0 0 1 20 0) |
|
256 |
translateLabel: true |
|
257 |
labelChannel: diffTextLabelB |
|
258 |
) |
|
259 |
(ArbitraryComponentSpec |
|
260 |
name: 'diffTextView' |
|
261 |
layout: (LayoutFrame 0 0.0 20 0.0 0 1.0 0 1.0) |
|
262 |
hasBorder: false |
|
263 |
component: diffTextView |
|
17341 | 264 |
) |
17397 | 265 |
) |
266 |
||
267 |
) |
|
17341 | 268 |
) |
17397 | 269 |
(CodeViewSpec |
270 |
name: 'singleTextView' |
|
271 |
layout: (LayoutFrame 0 0.0 0 0.0 0 1.0 0 1.0) |
|
272 |
model: methodText |
|
273 |
hasHorizontalScrollBar: true |
|
274 |
hasVerticalScrollBar: true |
|
275 |
miniScrollerHorizontal: true |
|
276 |
isReadOnly: true |
|
277 |
hasKeyboardFocusInitially: false |
|
278 |
) |
|
279 |
) |
|
280 |
||
281 |
) |
|
17341 | 282 |
) |
17397 | 283 |
) |
284 |
||
285 |
) |
|
286 |
handles: (Any 0.41999999999999998 1.0) |
|
17341 | 287 |
) |
17397 | 288 |
(LabelSpec |
289 |
name: 'InfoLabel' |
|
290 |
layout: (LayoutFrame 0 0 -25 1 0 1 0 1) |
|
291 |
level: -1 |
|
292 |
translateLabel: true |
|
293 |
labelChannel: infoHolder |
|
294 |
adjust: left |
|
295 |
) |
|
296 |
) |
|
297 |
||
298 |
) |
|
299 |
) |
|
14076 | 300 |
! |
301 |
||
302 |
windowSpecForMultipleVersions |
|
303 |
"This resource specification was automatically generated |
|
304 |
by the UIPainter of ST/X." |
|
305 |
||
306 |
"Do not manually edit this!! If it is corrupted, |
|
307 |
the UIPainter may not be able to read the specification." |
|
308 |
||
309 |
" |
|
310 |
UIPainter new openOnClass:VersionDiffBrowser andSelector:#windowSpecForMultipleVersions |
|
311 |
VersionDiffBrowser new openInterface:#windowSpecForMultipleVersions |
|
312 |
" |
|
313 |
||
314 |
<resource: #canvas> |
|
315 |
||
316 |
^ |
|
317 |
#(FullSpec |
|
318 |
name: windowSpecForMultipleVersions |
|
17827 | 319 |
uuid: '55cb9d62-d4fd-11e7-af82-c42c033b4871' |
14076 | 320 |
window: |
321 |
(WindowSpec |
|
322 |
label: 'Version DiffBrowser' |
|
323 |
name: 'Version DiffBrowser' |
|
17827 | 324 |
uuid: '2b3f2546-d4fd-11e7-af82-c42c033b4871' |
14076 | 325 |
min: (Point 10 10) |
326 |
bounds: (Rectangle 0 0 865 504) |
|
327 |
menu: mainMenu |
|
328 |
) |
|
329 |
component: |
|
330 |
(SpecCollection |
|
331 |
collection: ( |
|
332 |
(VariableVerticalPanelSpec |
|
333 |
name: 'VariableVerticalPanel1' |
|
334 |
layout: (LayoutFrame 0 0.0 0 0.0 0 1.0 -25 1.0) |
|
17827 | 335 |
uuid: '2b3f297e-d4fd-11e7-af82-c42c033b4871' |
14076 | 336 |
component: |
337 |
(SpecCollection |
|
338 |
collection: ( |
|
339 |
(ViewSpec |
|
340 |
name: 'Box1' |
|
17827 | 341 |
uuid: '2b3f2c08-d4fd-11e7-af82-c42c033b4871' |
14076 | 342 |
component: |
343 |
(SpecCollection |
|
344 |
collection: ( |
|
345 |
(SequenceViewSpec |
|
346 |
name: 'VersionList' |
|
347 |
layout: (LayoutFrame 0 0 0 0 0 0.5 0 1) |
|
17827 | 348 |
uuid: '2b3f2d20-d4fd-11e7-af82-c42c033b4871' |
349 |
model: selectedVersionIndexHolder |
|
14076 | 350 |
hasHorizontalScrollBar: true |
351 |
hasVerticalScrollBar: true |
|
17827 | 352 |
useIndex: true |
353 |
sequenceList: versionEntriesList |
|
17954 | 354 |
menu: versionsMenu |
14076 | 355 |
) |
356 |
(SequenceViewSpec |
|
357 |
name: 'List1' |
|
358 |
layout: (LayoutFrame 0 0.5 0 0 0 1 0 1) |
|
17827 | 359 |
uuid: '2b3f3086-d4fd-11e7-af82-c42c033b4871' |
14076 | 360 |
model: selectedSymbolicVersionHolder |
361 |
hasHorizontalScrollBar: true |
|
362 |
hasVerticalScrollBar: true |
|
363 |
useIndex: false |
|
364 |
sequenceList: symbolicVersionList |
|
17954 | 365 |
menu: symbolicVersionsMenu |
14076 | 366 |
) |
367 |
) |
|
368 |
||
369 |
) |
|
370 |
) |
|
371 |
(HorizontalPanelViewSpec |
|
372 |
name: 'TopHorizontalPanel' |
|
17827 | 373 |
uuid: '2b3f3234-d4fd-11e7-af82-c42c033b4871' |
14076 | 374 |
horizontalLayout: fit |
375 |
verticalLayout: fit |
|
376 |
horizontalSpace: 3 |
|
377 |
verticalSpace: 3 |
|
378 |
component: |
|
379 |
(SpecCollection |
|
380 |
collection: ( |
|
381 |
(ViewSpec |
|
382 |
name: 'BoxA' |
|
17827 | 383 |
uuid: '2b3f3464-d4fd-11e7-af82-c42c033b4871' |
14076 | 384 |
level: 0 |
385 |
visibilityChannel: boxAVisible |
|
386 |
component: |
|
387 |
(SpecCollection |
|
388 |
collection: ( |
|
389 |
(LabelSpec |
|
390 |
label: 'Only in A' |
|
391 |
name: 'OnlyInALabel' |
|
392 |
layout: (LayoutFrame 0 0.0 0 0.0 0 1.0 20 0) |
|
17827 | 393 |
uuid: '2b3f3568-d4fd-11e7-af82-c42c033b4871' |
14076 | 394 |
translateLabel: true |
395 |
labelChannel: onlyInALabelHolder |
|
396 |
) |
|
397 |
(SequenceViewSpec |
|
398 |
name: 'ListA' |
|
399 |
layout: (LayoutFrame 0 0.0 20 0.0 0 1.0 -3 1.0) |
|
17827 | 400 |
uuid: '2b3f37c0-d4fd-11e7-af82-c42c033b4871' |
14076 | 401 |
model: methodsOnlyInASelection |
17341 | 402 |
menu: menuAHolder |
14076 | 403 |
hasHorizontalScrollBar: true |
404 |
hasVerticalScrollBar: true |
|
405 |
miniScrollerHorizontal: true |
|
406 |
doubleClickSelector: methodInADoubleClicked: |
|
407 |
valueChangeSelector: methodsOnlyInASelectionChanged |
|
408 |
useIndex: true |
|
409 |
sequenceList: methodsOnlyInA |
|
410 |
) |
|
411 |
) |
|
412 |
||
413 |
) |
|
414 |
extent: (Point 286 207) |
|
415 |
) |
|
416 |
(ViewSpec |
|
417 |
name: 'BoxM' |
|
17827 | 418 |
uuid: '2b3f39a0-d4fd-11e7-af82-c42c033b4871' |
14076 | 419 |
visibilityChannel: boxMVisible |
420 |
component: |
|
421 |
(SpecCollection |
|
422 |
collection: ( |
|
423 |
(LabelSpec |
|
424 |
label: 'Changed' |
|
425 |
name: 'ChangedLabel' |
|
426 |
layout: (LayoutFrame 0 0.0 0 0.0 0 1.0 20 0) |
|
17827 | 427 |
uuid: '2b3f3a86-d4fd-11e7-af82-c42c033b4871' |
14076 | 428 |
translateLabel: true |
429 |
labelChannel: changedLabelHolder |
|
430 |
) |
|
431 |
(SequenceViewSpec |
|
432 |
name: 'ListM' |
|
433 |
layout: (LayoutFrame 0 0.0 20 0.0 0 1.0 -3 1.0) |
|
17827 | 434 |
uuid: '2b3f3b94-d4fd-11e7-af82-c42c033b4871' |
14076 | 435 |
model: methodsChangedSelection |
17341 | 436 |
menu: menuMHolder |
14076 | 437 |
hasHorizontalScrollBar: true |
438 |
hasVerticalScrollBar: true |
|
439 |
miniScrollerHorizontal: true |
|
440 |
doubleClickSelector: methodInChangedDoubleClicked: |
|
441 |
valueChangeSelector: methodsChangedSelectionChanged |
|
442 |
useIndex: true |
|
443 |
sequenceList: methodsChanged |
|
444 |
) |
|
445 |
) |
|
446 |
||
447 |
) |
|
448 |
extent: (Point 286 207) |
|
449 |
) |
|
450 |
(ViewSpec |
|
451 |
name: 'BoxB' |
|
17827 | 452 |
uuid: '2b3f3d1a-d4fd-11e7-af82-c42c033b4871' |
14076 | 453 |
visibilityChannel: boxBVisible |
454 |
component: |
|
455 |
(SpecCollection |
|
456 |
collection: ( |
|
457 |
(LabelSpec |
|
458 |
label: 'Only in B' |
|
459 |
name: 'OnlyInBLabel' |
|
460 |
layout: (LayoutFrame 0 0.0 0 0.0 0 1.0 20 0) |
|
17827 | 461 |
uuid: '2b3f3dec-d4fd-11e7-af82-c42c033b4871' |
14076 | 462 |
translateLabel: true |
463 |
labelChannel: onlyInBLabelHolder |
|
464 |
) |
|
465 |
(SequenceViewSpec |
|
466 |
name: 'ListB' |
|
467 |
layout: (LayoutFrame 0 0.0 20 0.0 0 1.0 -3 1.0) |
|
17827 | 468 |
uuid: '2b3f3f04-d4fd-11e7-af82-c42c033b4871' |
14076 | 469 |
model: methodsOnlyInBSelection |
17341 | 470 |
menu: menuBHolder |
14076 | 471 |
hasHorizontalScrollBar: true |
472 |
hasVerticalScrollBar: true |
|
473 |
miniScrollerHorizontal: true |
|
474 |
doubleClickSelector: methodInBDoubleClicked: |
|
475 |
valueChangeSelector: methodsOnlyInBSelectionChanged |
|
476 |
useIndex: true |
|
477 |
sequenceList: methodsOnlyInB |
|
478 |
) |
|
479 |
) |
|
480 |
||
481 |
) |
|
482 |
extent: (Point 287 207) |
|
483 |
) |
|
484 |
) |
|
485 |
||
486 |
) |
|
487 |
) |
|
488 |
(ViewSpec |
|
489 |
name: 'DiffBox' |
|
17827 | 490 |
uuid: '2b3f408a-d4fd-11e7-af82-c42c033b4871' |
14076 | 491 |
component: |
492 |
(SpecCollection |
|
493 |
collection: ( |
|
494 |
(ViewSpec |
|
495 |
name: 'diffTextViewBox' |
|
496 |
layout: (LayoutFrame 0 0.0 0 0.0 0 1.0 0 1.0) |
|
17827 | 497 |
uuid: '2b3f415c-d4fd-11e7-af82-c42c033b4871' |
14076 | 498 |
initiallyInvisible: true |
499 |
component: |
|
500 |
(SpecCollection |
|
501 |
collection: ( |
|
502 |
(LabelSpec |
|
503 |
label: 'A' |
|
504 |
name: 'DiffTextLabelA' |
|
505 |
layout: (LayoutFrame 0 0 0 0 0 0.5 20 0) |
|
17827 | 506 |
uuid: '2b3f426a-d4fd-11e7-af82-c42c033b4871' |
14076 | 507 |
translateLabel: true |
508 |
labelChannel: diffTextLabelA |
|
509 |
) |
|
510 |
(LabelSpec |
|
511 |
label: 'B' |
|
512 |
name: 'DiffTextLabelB' |
|
513 |
layout: (LayoutFrame 0 0.5 0 0 0 1 20 0) |
|
17827 | 514 |
uuid: '2b3f43d2-d4fd-11e7-af82-c42c033b4871' |
14076 | 515 |
translateLabel: true |
516 |
labelChannel: diffTextLabelB |
|
517 |
) |
|
518 |
(ArbitraryComponentSpec |
|
519 |
name: 'diffTextView' |
|
520 |
layout: (LayoutFrame 0 0.0 20 0.0 0 1.0 0 1.0) |
|
17827 | 521 |
uuid: '2b3f4508-d4fd-11e7-af82-c42c033b4871' |
14076 | 522 |
hasBorder: false |
523 |
component: diffTextView |
|
524 |
) |
|
525 |
) |
|
526 |
||
527 |
) |
|
528 |
) |
|
529 |
(CodeViewSpec |
|
530 |
name: 'singleTextView' |
|
531 |
layout: (LayoutFrame 0 0.0 0 0.0 0 1.0 0 1.0) |
|
17827 | 532 |
uuid: '2b3f4738-d4fd-11e7-af82-c42c033b4871' |
14076 | 533 |
model: methodText |
534 |
hasHorizontalScrollBar: true |
|
535 |
hasVerticalScrollBar: true |
|
536 |
miniScrollerHorizontal: true |
|
537 |
isReadOnly: true |
|
538 |
hasKeyboardFocusInitially: false |
|
539 |
) |
|
540 |
) |
|
541 |
||
542 |
) |
|
543 |
) |
|
544 |
) |
|
545 |
||
546 |
) |
|
17827 | 547 |
handles: (Any 0.22 0.67000000000000004 1.0) |
14076 | 548 |
) |
549 |
(LabelSpec |
|
550 |
name: 'InfoLabel' |
|
551 |
layout: (LayoutFrame 0 0 -25 1 0 1 0 1) |
|
17827 | 552 |
uuid: '2b3f4a44-d4fd-11e7-af82-c42c033b4871' |
14076 | 553 |
level: -1 |
554 |
translateLabel: true |
|
555 |
labelChannel: infoHolder |
|
556 |
adjust: left |
|
557 |
) |
|
558 |
) |
|
559 |
||
560 |
) |
|
561 |
) |
|
2221 | 562 |
! ! |
563 |
||
564 |
!VersionDiffBrowser class methodsFor:'menu specs'! |
|
565 |
||
17341 | 566 |
filterMenu |
567 |
"This resource specification was automatically generated |
|
568 |
by the MenuEditor of ST/X." |
|
569 |
||
570 |
"Do not manually edit this!! If it is corrupted, |
|
571 |
the MenuEditor may not be able to read the specification." |
|
572 |
||
573 |
||
574 |
" |
|
575 |
MenuEditor new openOnClass:VersionDiffBrowser andSelector:#mainMenu |
|
576 |
(Menu new fromLiteralArrayEncoding:(VersionDiffBrowser mainMenu)) startUp |
|
577 |
" |
|
578 |
||
579 |
<resource: #menu> |
|
580 |
||
581 |
^ |
|
582 |
#(Menu |
|
583 |
( |
|
584 |
(MenuItem |
|
585 |
label: 'Filter Classes Matching...' |
|
586 |
itemValue: filterClassesMatching |
|
587 |
) |
|
588 |
(MenuItem |
|
17348 | 589 |
label: 'Filter Selectors Matching...' |
590 |
itemValue: filterSelectorsMatching |
|
591 |
) |
|
592 |
(MenuItem |
|
17952 | 593 |
label: 'Remove Class Filter' |
594 |
itemValue: removeClassFilter |
|
595 |
) |
|
596 |
(MenuItem |
|
597 |
label: 'Remove Selector Filter' |
|
598 |
itemValue: removeSelectorFilter |
|
599 |
) |
|
600 |
(MenuItem |
|
17341 | 601 |
label: '-' |
602 |
) |
|
603 |
(MenuItem |
|
604 |
label: 'Remember Filter As...' |
|
605 |
itemValue: rememberFilterAs |
|
606 |
) |
|
607 |
(MenuItem |
|
17344 | 608 |
label: 'Load Filter Named...' |
609 |
itemValue: loadFilterNamed |
|
610 |
) |
|
611 |
(MenuItem |
|
612 |
label: '-' |
|
613 |
) |
|
614 |
(MenuItem |
|
17349 | 615 |
label: 'Export Filter Settings' |
616 |
itemValue: exportFilterParameters |
|
617 |
) |
|
618 |
(MenuItem |
|
619 |
label: 'Import Filter Settings From...' |
|
620 |
itemValue: importFilterParameters |
|
621 |
) |
|
622 |
(MenuItem |
|
623 |
label: '-' |
|
624 |
) |
|
625 |
(MenuItem |
|
17341 | 626 |
label: 'Forget Filter Named...' |
627 |
itemValue: forgetFilterNamed |
|
628 |
) |
|
17344 | 629 |
(MenuItem |
630 |
label: '-' |
|
631 |
) |
|
632 |
(MenuItem |
|
633 |
label: 'Inspect Current Filter Settings' |
|
634 |
itemValue: inspectFilterParameters |
|
635 |
) |
|
636 |
||
17341 | 637 |
) |
638 |
nil |
|
639 |
nil |
|
640 |
) |
|
641 |
||
642 |
"Created: / 05-02-2017 / 11:11:30 / cg" |
|
17349 | 643 |
"Modified: / 06-02-2017 / 09:27:22 / cg" |
17341 | 644 |
! |
645 |
||
2221 | 646 |
mainMenu |
647 |
"This resource specification was automatically generated |
|
648 |
by the MenuEditor of ST/X." |
|
649 |
||
650 |
"Do not manually edit this!! If it is corrupted, |
|
651 |
the MenuEditor may not be able to read the specification." |
|
652 |
||
10703 | 653 |
|
2221 | 654 |
" |
655 |
MenuEditor new openOnClass:VersionDiffBrowser andSelector:#mainMenu |
|
656 |
(Menu new fromLiteralArrayEncoding:(VersionDiffBrowser mainMenu)) startUp |
|
657 |
" |
|
658 |
||
659 |
<resource: #menu> |
|
660 |
||
11596
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
661 |
^ |
7419
e4e108674fe6
conditionalRight in startGroup
Claus Gittinger <cg@exept.de>
parents:
7396
diff
changeset
|
662 |
#(Menu |
11596
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
663 |
( |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
664 |
(MenuItem |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
665 |
label: 'File' |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
666 |
submenu: |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
667 |
(Menu |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
668 |
( |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
669 |
(MenuItem |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
670 |
label: 'Create PatchFile...' |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
671 |
itemValue: createPatchFile |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
672 |
) |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
673 |
(MenuItem |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
674 |
label: '-' |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
675 |
) |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
676 |
(MenuItem |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
677 |
label: 'Exit' |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
678 |
itemValue: closeRequest |
11853
fe611dbb05ca
added: #removeClassFromChangeSet
Claus Gittinger <cg@exept.de>
parents:
11596
diff
changeset
|
679 |
) |
fe611dbb05ca
added: #removeClassFromChangeSet
Claus Gittinger <cg@exept.de>
parents:
11596
diff
changeset
|
680 |
) |
fe611dbb05ca
added: #removeClassFromChangeSet
Claus Gittinger <cg@exept.de>
parents:
11596
diff
changeset
|
681 |
nil |
fe611dbb05ca
added: #removeClassFromChangeSet
Claus Gittinger <cg@exept.de>
parents:
11596
diff
changeset
|
682 |
nil |
fe611dbb05ca
added: #removeClassFromChangeSet
Claus Gittinger <cg@exept.de>
parents:
11596
diff
changeset
|
683 |
) |
fe611dbb05ca
added: #removeClassFromChangeSet
Claus Gittinger <cg@exept.de>
parents:
11596
diff
changeset
|
684 |
) |
fe611dbb05ca
added: #removeClassFromChangeSet
Claus Gittinger <cg@exept.de>
parents:
11596
diff
changeset
|
685 |
(MenuItem |
fe611dbb05ca
added: #removeClassFromChangeSet
Claus Gittinger <cg@exept.de>
parents:
11596
diff
changeset
|
686 |
label: 'Class' |
fe611dbb05ca
added: #removeClassFromChangeSet
Claus Gittinger <cg@exept.de>
parents:
11596
diff
changeset
|
687 |
submenu: |
fe611dbb05ca
added: #removeClassFromChangeSet
Claus Gittinger <cg@exept.de>
parents:
11596
diff
changeset
|
688 |
(Menu |
fe611dbb05ca
added: #removeClassFromChangeSet
Claus Gittinger <cg@exept.de>
parents:
11596
diff
changeset
|
689 |
( |
fe611dbb05ca
added: #removeClassFromChangeSet
Claus Gittinger <cg@exept.de>
parents:
11596
diff
changeset
|
690 |
(MenuItem |
fe611dbb05ca
added: #removeClassFromChangeSet
Claus Gittinger <cg@exept.de>
parents:
11596
diff
changeset
|
691 |
label: 'Remove from Changeset' |
fe611dbb05ca
added: #removeClassFromChangeSet
Claus Gittinger <cg@exept.de>
parents:
11596
diff
changeset
|
692 |
itemValue: removeClassFromChangeSet |
17341 | 693 |
enabled: showingClassChangeSet |
11596
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
694 |
) |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
695 |
) |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
696 |
nil |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
697 |
nil |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
698 |
) |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
699 |
) |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
700 |
(MenuItem |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
701 |
label: 'View' |
17341 | 702 |
submenuChannel: viewMenu |
703 |
) |
|
704 |
(MenuItem |
|
705 |
label: 'Filter' |
|
706 |
submenuChannel: filterMenu |
|
707 |
) |
|
708 |
(MenuItem |
|
709 |
label: 'MENU_Help' |
|
710 |
startGroup: conditionalRight |
|
11596
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
711 |
submenu: |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
712 |
(Menu |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
713 |
( |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
714 |
(MenuItem |
17341 | 715 |
label: 'Documentation' |
716 |
itemValue: openDocumentation |
|
717 |
) |
|
718 |
(MenuItem |
|
719 |
label: '-' |
|
720 |
) |
|
721 |
(MenuItem |
|
722 |
label: 'About this Application...' |
|
723 |
itemValue: openAboutThisApplication |
|
724 |
) |
|
725 |
) |
|
726 |
nil |
|
727 |
nil |
|
728 |
) |
|
729 |
) |
|
730 |
) |
|
731 |
nil |
|
732 |
nil |
|
733 |
) |
|
734 |
||
735 |
"Modified: / 05-02-2017 / 11:09:50 / cg" |
|
736 |
! |
|
737 |
||
738 |
menuA |
|
739 |
"This resource specification was automatically generated |
|
740 |
by the MenuEditor of ST/X." |
|
741 |
||
742 |
"Do not manually edit this!! If it is corrupted, |
|
743 |
the MenuEditor may not be able to read the specification." |
|
744 |
||
745 |
" |
|
746 |
MenuEditor new openOnClass:VersionDiffBrowser andSelector:#menu1 |
|
747 |
(Menu new fromLiteralArrayEncoding:(VersionDiffBrowser menu1)) startUp |
|
748 |
" |
|
749 |
||
750 |
<resource: #menu> |
|
751 |
||
752 |
^ |
|
753 |
#(#Menu |
|
754 |
#( |
|
755 |
#(#MenuItem |
|
756 |
#label: 'Apply' |
|
757 |
#translateLabel: true |
|
758 |
#value: #applySelectedChangeInA |
|
759 |
#enabled: #hasChangeSelectedInA |
|
760 |
) |
|
761 |
#(#MenuItem |
|
762 |
#label: 'Browse' |
|
763 |
#translateLabel: true |
|
764 |
#value: #browseClassInA |
|
765 |
#enabled: #hasChangeSelectedInA |
|
766 |
) |
|
767 |
#(#MenuItem |
|
768 |
#label: '-' |
|
769 |
) |
|
770 |
#(#MenuItem |
|
771 |
#label: 'Filter this Change' |
|
772 |
#translateLabel: true |
|
773 |
#value: #filterMethodInA |
|
774 |
#enabled: #hasChangeSelectedInA |
|
775 |
) |
|
776 |
#(#MenuItem |
|
777 |
#label: 'Filter Class' |
|
778 |
#translateLabel: true |
|
779 |
#value: #filterClassInA |
|
780 |
#enabled: #hasChangeSelectedInA |
|
781 |
) |
|
782 |
#(#MenuItem |
|
17931 | 783 |
#label: 'Filter Selector' |
17341 | 784 |
#translateLabel: true |
785 |
#value: #filterSelectorFromA |
|
786 |
#enabled: #hasChangeSelectedInA |
|
787 |
) |
|
788 |
#(#MenuItem |
|
789 |
#label: '-' |
|
790 |
) |
|
791 |
#(#MenuItem |
|
17931 | 792 |
#label: 'Find Next Change with String...' |
793 |
#translateLabel: true |
|
794 |
#value: #findNextChangeWithStringInA |
|
795 |
#enabled: #hasChangeSelectedInA |
|
796 |
) |
|
797 |
#(#MenuItem |
|
798 |
#label: '-' |
|
799 |
) |
|
800 |
#(#MenuItem |
|
17341 | 801 |
#label: 'Inspect' |
802 |
#translateLabel: true |
|
803 |
#value: #inspectSelectedChangeInA |
|
804 |
#choiceValue: 'nil "UndefinedObject" ' |
|
805 |
#enabled: #hasChangeSelectedInA |
|
806 |
) |
|
807 |
) |
|
808 |
nil |
|
809 |
nil |
|
810 |
) |
|
811 |
||
812 |
"Modified: / 05-02-2017 / 11:21:54 / cg" |
|
813 |
! |
|
814 |
||
815 |
menuB |
|
816 |
"This resource specification was automatically generated |
|
817 |
by the MenuEditor of ST/X." |
|
818 |
||
819 |
"Do not manually edit this!! If it is corrupted, |
|
820 |
the MenuEditor may not be able to read the specification." |
|
821 |
||
822 |
" |
|
823 |
MenuEditor new openOnClass:VersionDiffBrowser andSelector:#menuB |
|
824 |
(Menu new fromLiteralArrayEncoding:(VersionDiffBrowser menuB)) startUp |
|
825 |
" |
|
826 |
||
827 |
<resource: #menu> |
|
828 |
||
829 |
^ |
|
830 |
#(#Menu |
|
831 |
#( |
|
832 |
#(#MenuItem |
|
833 |
#label: 'Apply' |
|
834 |
#translateLabel: true |
|
835 |
#value: #applySelectedChangeInB |
|
836 |
#enabled: #hasChangeSelectedInB |
|
837 |
) |
|
838 |
#(#MenuItem |
|
839 |
#label: 'Browse' |
|
840 |
#translateLabel: true |
|
841 |
#value: #browseClassInB |
|
842 |
#enabled: #hasChangeSelectedInB |
|
843 |
) |
|
844 |
#(#MenuItem |
|
845 |
#label: '-' |
|
846 |
) |
|
847 |
#(#MenuItem |
|
848 |
#label: 'Remove in Image' |
|
849 |
#translateLabel: true |
|
850 |
#value: #removeSelectedChangeInBFromImage |
|
851 |
#enabled: #hasChangeSelectedInB |
|
852 |
) |
|
853 |
#(#MenuItem |
|
854 |
#label: '-' |
|
855 |
) |
|
856 |
#(#MenuItem |
|
857 |
#label: 'Filter this Change' |
|
858 |
#translateLabel: true |
|
859 |
#value: #filterMethodInB |
|
860 |
#enabled: #hasChangeSelectedInB |
|
861 |
) |
|
862 |
#(#MenuItem |
|
863 |
#label: 'Filter Class' |
|
864 |
#translateLabel: true |
|
865 |
#value: #filterClassInB |
|
866 |
#enabled: #hasChangeSelectedInB |
|
867 |
) |
|
868 |
#(#MenuItem |
|
869 |
#label: 'Filter Selector ' |
|
870 |
#translateLabel: true |
|
871 |
#value: #filterSelectorFromB |
|
872 |
#enabled: #hasChangeSelectedInB |
|
873 |
) |
|
874 |
#(#MenuItem |
|
875 |
#label: '-' |
|
876 |
) |
|
877 |
#(#MenuItem |
|
17931 | 878 |
#label: '-' |
879 |
) |
|
880 |
#(#MenuItem |
|
881 |
#label: 'Find Next Change with String...' |
|
882 |
#translateLabel: true |
|
883 |
#value: #findNextChangeWithStringInB |
|
884 |
#enabled: #hasChangeSelectedInB |
|
885 |
) |
|
886 |
#(#MenuItem |
|
17341 | 887 |
#label: 'Inspect' |
888 |
#translateLabel: true |
|
889 |
#value: #inspectSelectedChangeInB |
|
890 |
#choiceValue: 'nil "UndefinedObject" ' |
|
891 |
#enabled: #hasChangeSelectedInB |
|
892 |
) |
|
893 |
) |
|
894 |
nil |
|
895 |
nil |
|
896 |
) |
|
897 |
||
898 |
"Modified: / 05-02-2017 / 11:22:10 / cg" |
|
899 |
! |
|
900 |
||
901 |
menuM |
|
902 |
"This resource specification was automatically generated |
|
903 |
by the MenuEditor of ST/X." |
|
904 |
||
905 |
"Do not manually edit this!! If it is corrupted, |
|
906 |
the MenuEditor may not be able to read the specification." |
|
907 |
||
908 |
" |
|
909 |
MenuEditor new openOnClass:VersionDiffBrowser andSelector:#menu1 |
|
910 |
(Menu new fromLiteralArrayEncoding:(VersionDiffBrowser menu1)) startUp |
|
911 |
" |
|
912 |
||
913 |
<resource: #menu> |
|
914 |
||
915 |
^ |
|
916 |
#(#Menu |
|
917 |
#( |
|
918 |
#(#MenuItem |
|
919 |
#label: 'Browse' |
|
920 |
#translateLabel: true |
|
921 |
#value: #browseClassInM |
|
922 |
#enabled: #hasChangeSelectedInM |
|
923 |
) |
|
924 |
#(#MenuItem |
|
925 |
#label: '-' |
|
926 |
) |
|
927 |
#(#MenuItem |
|
928 |
#label: 'Apply Left Version' |
|
929 |
#translateLabel: true |
|
930 |
#value: #acceptInLeftView |
|
931 |
#enabled: #hasChangeSelectedInM |
|
932 |
) |
|
933 |
#(#MenuItem |
|
934 |
#label: 'Apply Right Version' |
|
935 |
#translateLabel: true |
|
936 |
#value: #acceptInRightView |
|
937 |
#enabled: #hasChangeSelectedInM |
|
938 |
) |
|
939 |
#(#MenuItem |
|
940 |
#label: '-' |
|
941 |
) |
|
942 |
#(#MenuItem |
|
943 |
#label: 'Filter this Change' |
|
944 |
#translateLabel: true |
|
945 |
#value: #filterMethodInM |
|
946 |
#enabled: #hasChangeSelectedInM |
|
947 |
) |
|
948 |
#(#MenuItem |
|
949 |
#label: 'Filter Class' |
|
950 |
#translateLabel: true |
|
951 |
#value: #filterClassInM |
|
952 |
#enabled: #hasChangeSelectedInM |
|
953 |
) |
|
954 |
#(#MenuItem |
|
955 |
#label: 'Filter Selector ' |
|
956 |
#translateLabel: true |
|
957 |
#value: #filterSelectorFromM |
|
958 |
#enabled: #hasChangeSelectedInM |
|
959 |
) |
|
960 |
#(#MenuItem |
|
961 |
#label: '-' |
|
962 |
) |
|
963 |
#(#MenuItem |
|
17931 | 964 |
#label: 'Find Next Change with String...' |
965 |
#translateLabel: true |
|
966 |
#value: #findNextChangeWithStringInM |
|
967 |
#enabled: #hasChangeSelectedInM |
|
968 |
) |
|
969 |
#(#MenuItem |
|
970 |
#label: '-' |
|
971 |
) |
|
972 |
#(#MenuItem |
|
17341 | 973 |
#label: 'Inspect both' |
974 |
#translateLabel: true |
|
975 |
#value: #inspectSelectedChangeInM |
|
976 |
#choiceValue: 'nil "UndefinedObject" ' |
|
977 |
#enabled: #hasChangeSelectedInM |
|
978 |
) |
|
979 |
) |
|
980 |
nil |
|
981 |
nil |
|
982 |
) |
|
983 |
||
984 |
"Modified: / 05-02-2017 / 11:21:44 / cg" |
|
985 |
! |
|
986 |
||
17954 | 987 |
versionsMenu |
988 |
"This resource specification was automatically generated |
|
989 |
by the MenuEditor of ST/X." |
|
990 |
||
991 |
"Do not manually edit this!! If it is corrupted, |
|
992 |
the MenuEditor may not be able to read the specification." |
|
993 |
||
994 |
||
995 |
" |
|
996 |
MenuEditor new openOnClass:VersionDiffBrowser andSelector:#mainMenu |
|
997 |
(Menu new fromLiteralArrayEncoding:(VersionDiffBrowser mainMenu)) startUp |
|
998 |
" |
|
999 |
||
1000 |
<resource: #menu> |
|
1001 |
||
1002 |
^ |
|
1003 |
#(Menu |
|
1004 |
( |
|
1005 |
(MenuItem |
|
1006 |
label: 'Compare against Previous (Older) Version' |
|
1007 |
nameKey: CompareAgainstPreviousVersion |
|
1008 |
itemValue: versionMenuCompareAgainstPreviousVersion |
|
1009 |
enabled: hasPreviousVersionInVersionList |
|
1010 |
) |
|
1011 |
(MenuItem |
|
1012 |
label: 'Compare against Next (Younger) Version' |
|
1013 |
nameKey: CompareAgainstNextVersion |
|
1014 |
itemValue: versionMenuCompareAgainstNextVersion |
|
1015 |
enabled: hasNextVersionInVersionList |
|
1016 |
) |
|
1017 |
||
1018 |
) |
|
1019 |
nil |
|
1020 |
nil |
|
1021 |
) |
|
1022 |
||
1023 |
"Created: / 05-02-2017 / 11:11:30 / cg" |
|
1024 |
"Modified: / 06-02-2017 / 09:27:22 / cg" |
|
1025 |
! |
|
1026 |
||
17341 | 1027 |
viewMenu |
1028 |
"This resource specification was automatically generated |
|
1029 |
by the MenuEditor of ST/X." |
|
1030 |
||
1031 |
"Do not manually edit this!! If it is corrupted, |
|
1032 |
the MenuEditor may not be able to read the specification." |
|
1033 |
||
1034 |
||
1035 |
" |
|
1036 |
MenuEditor new openOnClass:VersionDiffBrowser andSelector:#mainMenu |
|
1037 |
(Menu new fromLiteralArrayEncoding:(VersionDiffBrowser mainMenu)) startUp |
|
1038 |
" |
|
1039 |
||
1040 |
<resource: #menu> |
|
1041 |
||
1042 |
^ |
|
1043 |
#(Menu |
|
1044 |
( |
|
1045 |
(MenuItem |
|
11596
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1046 |
enabled: canIncludeExtensionsHolder |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1047 |
label: 'Include Extensions' |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1048 |
itemValue: includeExtensions: |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1049 |
indication: includeExtensionsHolder |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1050 |
) |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1051 |
(MenuItem |
11958 | 1052 |
enabled: canIncludeExtensionsHolder |
1053 |
label: 'Include Version Methods' |
|
1054 |
itemValue: includeVersionMethods: |
|
1055 |
indication: includeVersionMethodsHolder |
|
1056 |
) |
|
1057 |
(MenuItem |
|
1058 |
label: '-' |
|
1059 |
) |
|
1060 |
(MenuItem |
|
11596
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1061 |
label: 'Include Category Changes' |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1062 |
itemValue: includeCategoryChanges: |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1063 |
indication: includeCategoryChangesHolder |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1064 |
) |
15632 | 1065 |
(MenuItem |
17334 | 1066 |
label: 'Hide Comment- and Formatting-Only Changes' |
1067 |
itemValue: hideDiffsWithCommentOrFormattingChangeOnlyHolder: |
|
1068 |
indication: hideDiffsWithCommentOrFormattingChangeOnlyHolder |
|
1069 |
) |
|
1070 |
(MenuItem |
|
15632 | 1071 |
label: '-' |
1072 |
) |
|
1073 |
(MenuItem |
|
1074 |
label: 'Show Log Messages' |
|
1075 |
itemValue: showLogMessages |
|
1076 |
) |
|
11596
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1077 |
) |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1078 |
nil |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1079 |
nil |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1080 |
) |
17341 | 1081 |
|
1082 |
"Created: / 05-02-2017 / 11:06:24 / cg" |
|
2221 | 1083 |
! ! |
1084 |
||
2695 | 1085 |
!VersionDiffBrowser class methodsFor:'plugIn spec'! |
1086 |
||
1087 |
aspectSelectors |
|
1088 |
"This resource specification was automatically generated |
|
1089 |
by the UIPainter of ST/X." |
|
1090 |
||
1091 |
"Do not manually edit this. If it is corrupted, |
|
1092 |
the UIPainter may not be able to read the specification." |
|
1093 |
||
1094 |
"Return a description of exported aspects; |
|
1095 |
these can be connected to aspects of an embedding application |
|
1096 |
(if this app is embedded in a subCanvas)." |
|
1097 |
||
1098 |
^ #( |
|
14076 | 1099 |
#changedLabelHolder |
1100 |
#onlyInALabelHolder |
|
1101 |
#onlyInBLabelHolder |
|
2695 | 1102 |
). |
1103 |
||
1104 |
! ! |
|
1105 |
||
2221 | 1106 |
!VersionDiffBrowser class methodsFor:'startup'! |
1107 |
||
14076 | 1108 |
openOnAllVersionsOfClass:aClass |
1109 |
"create a VersionDiffBrowser instance and set the class change set of the |
|
1110 |
browser. The class diff set is generated from aClasses current version against |
|
1111 |
the selected version in the top selection list" |
|
1112 |
||
1113 |
|theBrowser| |
|
1114 |
||
1115 |
theBrowser := self new. |
|
1116 |
theBrowser allButOpenInterface:#windowSpecForMultipleVersions. |
|
1117 |
theBrowser beMultipleVersionBrowser. |
|
1118 |
theBrowser classHolder:(ValueHolder with:aClass). |
|
1119 |
theBrowser setupForClass:aClass againstVersion:nil. |
|
17334 | 1120 |
theBrowser window label:(self resources string:'Versions of %1' with:aClass name). |
14076 | 1121 |
theBrowser openWindow. |
1122 |
^ theBrowser. |
|
1123 |
||
1124 |
" |
|
1125 |
self openOnAllVersionsOfClass:Object |
|
1126 |
" |
|
17334 | 1127 |
|
1128 |
"Modified: / 04-02-2017 / 19:52:33 / cg" |
|
14076 | 1129 |
! |
1130 |
||
11453 | 1131 |
openOnClass:aClass againstVersion:aVersionA |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1132 |
"create a VersionDiffBrowser instance and set the class change set of the |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1133 |
browser. The class diff set is generated from aClasses current against some version |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1134 |
via the source code manager." |
2507 | 1135 |
|
2497 | 1136 |
|theBrowser| |
2224 | 1137 |
|
2497 | 1138 |
theBrowser := self new. |
1139 |
theBrowser allButOpen. |
|
2695 | 1140 |
theBrowser setupForClass:aClass againstVersion:aVersionA. |
17334 | 1141 |
theBrowser window label:(self resources string:'Diffs of %1 %2 against current' with:aClass name with:aVersionA). |
2497 | 1142 |
theBrowser openWindow. |
1143 |
^ theBrowser. |
|
2224 | 1144 |
|
2695 | 1145 |
" |
1146 |
self openOnClass:Array againstVersion:'1.116' |
|
1147 |
self openOnClass:Array againstVersion:nil - against the version on which Array is based upon |
|
1148 |
self openOnClass:Array againstVersion:#newest - against the newest repository version |
|
1149 |
||
1150 |
self openOnClass:VersionDiffBrowser againstVersion:nil |
|
1151 |
self openOnClass:VersionDiffBrowser againstVersion:#newest |
|
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1152 |
|
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1153 |
(self openOnClass:ByteArray againstVersion:'1.116') includeExtensionsHolder value:true |
2695 | 1154 |
" |
17334 | 1155 |
|
1156 |
"Modified: / 04-02-2017 / 19:58:16 / cg" |
|
2695 | 1157 |
! |
1158 |
||
17334 | 1159 |
openOnClass:classA labelA:aLabelA andClass:classB labelB:aLabelB title:title |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1160 |
"create a VersionDiffBrowser instance and set the class change set of the |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1161 |
browser. The class diff set is generated from two classes." |
2949 | 1162 |
|
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1163 |
^ self |
17334 | 1164 |
openOnClass:classA labelA:aLabelA |
1165 |
andClass:classB labelB:aLabelB |
|
1166 |
title:title ifSame:nil |
|
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1167 |
|
3147 | 1168 |
" |
11453 | 1169 |
self |
17334 | 1170 |
openOnClass:Array labelA:'Array' |
1171 |
andClass:Collection labelB:'Collection' |
|
1172 |
title:'Array vs. Collection' |
|
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1173 |
" |
17334 | 1174 |
|
1175 |
"Modified (comment): / 04-02-2017 / 20:02:38 / cg" |
|
2949 | 1176 |
! |
1177 |
||
17334 | 1178 |
openOnClass:classA labelA:aLabelA andClass:classB labelB:aLabelB title:title ifSame:sameAction |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1179 |
"create a VersionDiffBrowser instance and set the class change set of the |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1180 |
browser. The class diff set is generated from two classes." |
2949 | 1181 |
|
8891 | 1182 |
|classChangeSet| |
2949 | 1183 |
|
8891 | 1184 |
classChangeSet := ClassChangeSet newForClass:classA labelA:aLabelA andClass:classB labelB:aLabelB. |
1185 |
classChangeSet removeAllVersionMethods. |
|
2949 | 1186 |
sameAction notNil ifTrue:[ |
17334 | 1187 |
"/ check if same ... |
1188 |
(classChangeSet methodsOnlyInA isEmpty |
|
1189 |
and:[classChangeSet methodsOnlyInB isEmpty |
|
1190 |
and:[classChangeSet diffSet isEmpty]]) ifTrue:[ |
|
1191 |
sameAction value. |
|
1192 |
^ self "/ do not open |
|
1193 |
] |
|
2949 | 1194 |
]. |
8891 | 1195 |
|
17334 | 1196 |
^ self openOnClassChangeSet:classChangeSet title:title |
1197 |
||
1198 |
"Modified (format): / 04-02-2017 / 19:59:16 / cg" |
|
2949 | 1199 |
! |
1200 |
||
11453 | 1201 |
openOnClass:aClass labelA:aLabelA sourceA:aSourceA labelB:aLabelB sourceB:aSourceB |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1202 |
"create a VersionDiffBrowser instance and set the class change set of the |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1203 |
browser. The class diff set is generated from two source files." |
2695 | 1204 |
|
2716
8451e7057f00
some common startup protocol between DiffViewer and VersionDiffBrowser
Claus Gittinger <cg@exept.de>
parents:
2714
diff
changeset
|
1205 |
^ self |
11453 | 1206 |
openOnClass:aClass labelA:aLabelA sourceA:aSourceA labelB:aLabelB sourceB:aSourceB |
1207 |
title:nil |
|
2716
8451e7057f00
some common startup protocol between DiffViewer and VersionDiffBrowser
Claus Gittinger <cg@exept.de>
parents:
2714
diff
changeset
|
1208 |
! |
8451e7057f00
some common startup protocol between DiffViewer and VersionDiffBrowser
Claus Gittinger <cg@exept.de>
parents:
2714
diff
changeset
|
1209 |
|
8451e7057f00
some common startup protocol between DiffViewer and VersionDiffBrowser
Claus Gittinger <cg@exept.de>
parents:
2714
diff
changeset
|
1210 |
openOnClass:aClass labelA:aLabelA sourceA:aSourceA labelB:aLabelB sourceB:aSourceB title:ignoredTitle |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1211 |
"create a VersionDiffBrowser instance and set the class change set of the |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1212 |
browser. The class diff set is generated from two source files." |
2716
8451e7057f00
some common startup protocol between DiffViewer and VersionDiffBrowser
Claus Gittinger <cg@exept.de>
parents:
2714
diff
changeset
|
1213 |
|
11453 | 1214 |
^ self |
1215 |
openOnClass:aClass labelA:aLabelA sourceA:aSourceA |
|
1216 |
labelB:aLabelB sourceB:aSourceB |
|
1217 |
title:ignoredTitle ifSame:nil |
|
2849 | 1218 |
! |
1219 |
||
1220 |
openOnClass:aClass labelA:aLabelA sourceA:aSourceA labelB:aLabelB sourceB:aSourceB title:ignoredTitle ifSame:sameAction |
|
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1221 |
"create a VersionDiffBrowser instance and set the class change set of the |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1222 |
browser. The class diff set is generated from two source files." |
8891 | 1223 |
|
1224 |
|diffs hasChangedMethods classChangeSet versionMethodChanges| |
|
2695 | 1225 |
|
8891 | 1226 |
classChangeSet := ClassChangeSet newForClass:aClass labelA:aLabelA sourceA:aSourceA labelB:aLabelB sourceB:aSourceB. |
1227 |
classChangeSet removeAllVersionMethods. |
|
2849 | 1228 |
sameAction notNil ifTrue:[ |
17347 | 1229 |
"/ check if same ... |
1230 |
(classChangeSet methodsOnlyInA isEmpty |
|
1231 |
and:[classChangeSet methodsOnlyInB isEmpty]) ifTrue:[ |
|
1232 |
diffs := classChangeSet diffSet. |
|
1233 |
||
1234 |
hasChangedMethods := diffs changed notEmpty. |
|
1235 |
hasChangedMethods ifTrue:[ |
|
1236 |
versionMethodChanges := diffs changed select:[:eachDifference | |
|
1237 |
|eachChange| |
|
1238 |
eachChange := eachDifference first. |
|
1239 |
(eachChange isMethodChange |
|
1240 |
and:[ (AbstractSourceCodeManager isVersionMethodSelector:eachChange changeSelector) |
|
1241 |
and:[ eachChange changeClass isMeta ]]). |
|
1242 |
]. |
|
1243 |
diffs changed size == versionMethodChanges size ifTrue: [ |
|
1244 |
hasChangedMethods := false. |
|
1245 |
]. |
|
8811
65e08571a2f3
changed: #openOnClass:labelA:sourceA:labelB:sourceB:title:ifSame:
fm
parents:
8657
diff
changeset
|
1246 |
"/ diffs changed size == 1 ifTrue:[ |
65e08571a2f3
changed: #openOnClass:labelA:sourceA:labelB:sourceB:title:ifSame:
fm
parents:
8657
diff
changeset
|
1247 |
"/ theOnlyDifference := diffs changed first. |
65e08571a2f3
changed: #openOnClass:labelA:sourceA:labelB:sourceB:title:ifSame:
fm
parents:
8657
diff
changeset
|
1248 |
"/ theOnlyChange := theOnlyDifference first. |
11453 | 1249 |
"/ (theOnlyChange isMethodChange |
8811
65e08571a2f3
changed: #openOnClass:labelA:sourceA:labelB:sourceB:title:ifSame:
fm
parents:
8657
diff
changeset
|
1250 |
"/ and:[ ((theOnlyChange selector == #version) or:[theOnlyChange selector startsWith:'version_']) |
65e08571a2f3
changed: #openOnClass:labelA:sourceA:labelB:sourceB:title:ifSame:
fm
parents:
8657
diff
changeset
|
1251 |
"/ and:[ theOnlyChange changeClass isMeta ]]) ifTrue:[ |
65e08571a2f3
changed: #openOnClass:labelA:sourceA:labelB:sourceB:title:ifSame:
fm
parents:
8657
diff
changeset
|
1252 |
"/ hasChangedMethods := false |
65e08571a2f3
changed: #openOnClass:labelA:sourceA:labelB:sourceB:title:ifSame:
fm
parents:
8657
diff
changeset
|
1253 |
"/ ] |
65e08571a2f3
changed: #openOnClass:labelA:sourceA:labelB:sourceB:title:ifSame:
fm
parents:
8657
diff
changeset
|
1254 |
"/ ] |
17347 | 1255 |
]. |
1256 |
||
1257 |
(hasChangedMethods not |
|
1258 |
and:[diffs onlyInArg isEmpty |
|
1259 |
and:[diffs onlyInReceiver isEmpty]]) ifTrue:[ |
|
1260 |
sameAction value. |
|
1261 |
^ self "/ do not open |
|
1262 |
]. |
|
1263 |
] |
|
2849 | 1264 |
]. |
8891 | 1265 |
|
1266 |
^ self openOnClassChangeSet:classChangeSet title:ignoredTitle |
|
7396 | 1267 |
|
17347 | 1268 |
"Modified: / 06-02-2017 / 02:31:51 / cg" |
2224 | 1269 |
! |
1270 |
||
17954 | 1271 |
openOnClass:aClass versionA:versionA versionB:versionB |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1272 |
"create a VersionDiffBrowser instance and set the class change set of the |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1273 |
browser. The class diff set is generated from two versions of aClass via |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1274 |
the source code manager." |
2507 | 1275 |
|
2497 | 1276 |
|theBrowser| |
2221 | 1277 |
|
2497 | 1278 |
theBrowser := self new. |
1279 |
theBrowser allButOpen. |
|
17954 | 1280 |
theBrowser setupForClass:aClass versionA:versionA versionB:versionB. |
1281 |
theBrowser window label:(self resources |
|
1282 |
string:'%1 Versions %2 vs. %3' |
|
1283 |
with:aClass name with:versionA with:versionB). |
|
2497 | 1284 |
theBrowser openWindow. |
1285 |
^ theBrowser. |
|
2221 | 1286 |
|
1287 |
" |
|
1288 |
self openOnClass:Array versionA:'1.116' versionB:'1.113' |
|
17954 | 1289 |
self openOnClass:Array versionA:'1.113' versionB:'1.116' |
2221 | 1290 |
" |
3678 | 1291 |
! |
1292 |
||
17334 | 1293 |
openOnClassChangeSet:classChangeSet title:title |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1294 |
"create a VersionDiffBrowser instance and set the class change set of the |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1295 |
browser. The class diff set is generated from two classes." |
8891 | 1296 |
|
1297 |
|theBrowser| |
|
1298 |
||
1299 |
theBrowser := self new. |
|
1300 |
theBrowser allButOpen. |
|
1301 |
theBrowser classChangeSet:classChangeSet. |
|
1302 |
theBrowser openWindow. |
|
17334 | 1303 |
theBrowser window label:title. |
8891 | 1304 |
^ theBrowser. |
17334 | 1305 |
|
1306 |
"Modified: / 04-02-2017 / 19:59:06 / cg" |
|
8891 | 1307 |
! |
1308 |
||
3678 | 1309 |
openOnDiffSet:diffSet labelA:aLabelA labelB:aLabelB title:ignoredTitle |
10691 | 1310 |
^ self |
11961
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1311 |
openOnDiffSet:diffSet |
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1312 |
labelA:aLabelA labelB:aLabelB |
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1313 |
title:ignoredTitle |
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1314 |
ignoreExtensions:false |
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1315 |
ignoreVersionMethods:false |
10691 | 1316 |
|
1317 |
"Modified: / 12-09-2011 / 11:52:44 / cg" |
|
1318 |
! |
|
1319 |
||
11961
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1320 |
openOnDiffSet:diffSet labelA:aLabelA labelB:aLabelB title:ignoredTitle ignoreExtensions:ignoreExtensionsBoolean |
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1321 |
^ self |
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1322 |
openOnDiffSet:diffSet |
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1323 |
labelA:aLabelA labelB:aLabelB |
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1324 |
title:ignoredTitle |
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1325 |
ignoreExtensions:ignoreExtensionsBoolean |
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1326 |
ignoreVersionMethods:false |
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1327 |
! |
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1328 |
|
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1329 |
openOnDiffSet:diffSet labelA:aLabelA labelB:aLabelB title:ignoredTitle ignoreExtensions:ignoreExtensionsBoolean ignoreVersionMethods:ignoreVersionMethodsBoolean |
3678 | 1330 |
|theBrowser| |
1331 |
||
1332 |
theBrowser := self new. |
|
11961
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1333 |
theBrowser ignoreExtensions:ignoreExtensionsBoolean. |
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1334 |
theBrowser ignoreVersionMethods:ignoreVersionMethodsBoolean. |
3678 | 1335 |
theBrowser allButOpen. |
1336 |
theBrowser setupForDiffSet:diffSet labelA:aLabelA labelB:aLabelB. |
|
1337 |
theBrowser window label:ignoredTitle. |
|
1338 |
theBrowser openWindow. |
|
1339 |
^ theBrowser. |
|
10691 | 1340 |
|
1341 |
"Created: / 12-09-2011 / 11:50:18 / cg" |
|
5520 | 1342 |
! |
1343 |
||
1344 |
openOnDiffsBetweenFile:file1 and:file2 |
|
1345 |
|set1 set2 diffset theBrowser| |
|
1346 |
||
1347 |
set1 := ChangeSet fromFile:file1. |
|
1348 |
set2 := ChangeSet fromFile:file2. |
|
1349 |
diffset := set1 diffSetsAgainst:set2. |
|
1350 |
||
1351 |
theBrowser := self new. |
|
1352 |
theBrowser allButOpen. |
|
1353 |
theBrowser setupForDiffSet:diffset labelA:file1 asFilename baseName labelB:file2 asFilename baseName. |
|
1354 |
theBrowser window label:('%1 vs. %2' bindWith:file1 asFilename baseName with:file2 asFilename baseName). |
|
1355 |
theBrowser openWindow. |
|
1356 |
^ theBrowser. |
|
1357 |
||
1358 |
" |
|
11453 | 1359 |
VersionDiffBrowser |
1360 |
openOnDiffsBetweenFile:'/phys/exept/tmp/ws/Workflow::BlockDescription.st' |
|
1361 |
and:'/phys/exept/tmp/workflowClasses/Workflow::BlockDescription.st' |
|
5520 | 1362 |
" |
2221 | 1363 |
! ! |
1364 |
||
1365 |
!VersionDiffBrowser methodsFor:'accessing'! |
|
1366 |
||
14076 | 1367 |
beMultipleVersionBrowser |
1368 |
isMultipleVersionBrowser := true |
|
1369 |
! |
|
1370 |
||
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1371 |
canIncludeExtensions:something |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1372 |
self canIncludeExtensionsHolder value:something. |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1373 |
! |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1374 |
|
2497 | 1375 |
changeSetA |
3147 | 1376 |
" |
1377 |
gets the change set which contains only the new methods |
|
1378 |
in versionA of the class |
|
2221 | 1379 |
|
3147 | 1380 |
<return: ChangeSet> |
1381 |
" |
|
2221 | 1382 |
|
2497 | 1383 |
^ self classChangeSet methodsOnlyInA |
2221 | 1384 |
! |
1385 |
||
2497 | 1386 |
changeSetB |
3147 | 1387 |
" |
1388 |
gets the change set which contains only the new methods |
|
1389 |
in versionB of the class |
|
2497 | 1390 |
|
3147 | 1391 |
<return: ChangeSet> |
1392 |
" |
|
2497 | 1393 |
|
1394 |
^ self classChangeSet methodsOnlyInB |
|
1395 |
! |
|
1396 |
||
2856 | 1397 |
class:aClass versionA:revA versionB:revB |
1398 |
classIfSingleClassDiff := aClass. |
|
1399 |
versionAIfSingleClassDiff := revA. |
|
1400 |
versionBIfSingleClassDiff := revB. |
|
1401 |
! |
|
1402 |
||
2497 | 1403 |
classBeingCompared |
13137 | 1404 |
"returns the class from the change set which is compared. |
2224 | 1405 |
|
3147 | 1406 |
<return: Class> |
11453 | 1407 |
" |
2497 | 1408 |
^ self classChangeSet classBeingCompared |
1409 |
! |
|
1410 |
||
1411 |
classChangeSet |
|
11453 | 1412 |
"returns the class change set which is the model of the version diff browser." |
2224 | 1413 |
|
2507 | 1414 |
^ classChangeSet |
1415 |
! |
|
2497 | 1416 |
|
2507 | 1417 |
classChangeSet:aClassChangeSet |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1418 |
"sets the class change set which is the model of the version diff browser. |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1419 |
The labels, the lists, the visibility and the selection of the lists must be reseted |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1420 |
when a new change set is given." |
2497 | 1421 |
|
2507 | 1422 |
classChangeSet := aClassChangeSet. |
2501 | 1423 |
self updateLabels. |
1424 |
self updateLists. |
|
1425 |
self resetSelectionHolders. |
|
1426 |
! |
|
2224 | 1427 |
|
10691 | 1428 |
ignoreExtensions |
1429 |
"if true, extension methods are suppressed. |
|
1430 |
Ignoring is useful when comparing for baseClass checkin, |
|
1431 |
as opposed to extension-checking, where extensions should not be ignored. |
|
1432 |
the default is false" |
|
1433 |
||
11371
920084723b9c
exclude extension stuff fixed
Claus Gittinger <cg@exept.de>
parents:
10892
diff
changeset
|
1434 |
^ self includeExtensions not |
10691 | 1435 |
|
1436 |
"Created: / 12-09-2011 / 11:54:11 / cg" |
|
1437 |
! |
|
1438 |
||
1439 |
ignoreExtensions:aBoolean |
|
1440 |
"if true, extension methods are suppressed. |
|
1441 |
Ignoring is useful when comparing for baseClass checkin, |
|
1442 |
as opposed to extension-checking, where extensions should not be ignored. |
|
1443 |
the default is false" |
|
1444 |
||
11371
920084723b9c
exclude extension stuff fixed
Claus Gittinger <cg@exept.de>
parents:
10892
diff
changeset
|
1445 |
self includeExtensions:aBoolean not |
10691 | 1446 |
|
11371
920084723b9c
exclude extension stuff fixed
Claus Gittinger <cg@exept.de>
parents:
10892
diff
changeset
|
1447 |
"Modified: / 06-03-2012 / 15:37:32 / cg" |
10691 | 1448 |
! |
1449 |
||
11961
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1450 |
ignoreVersionMethods:aBoolean |
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1451 |
"if true, version methods (version and version_XXX) are suppressed. |
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1452 |
Ignoring is useful when comparing for real code changes. |
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1453 |
the default is false" |
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1454 |
|
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1455 |
self includeVersionMethods:aBoolean not |
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1456 |
|
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1457 |
"Modified: / 06-03-2012 / 15:37:32 / cg" |
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1458 |
! |
094f448dc1f0
added ignoreVersionMethods option
Claus Gittinger <cg@exept.de>
parents:
11958
diff
changeset
|
1459 |
|
14076 | 1460 |
isMultipleVersionBrowser |
14078 | 1461 |
^ isMultipleVersionBrowser ? false |
14076 | 1462 |
! |
1463 |
||
2507 | 1464 |
selectedChangeInA |
3147 | 1465 |
" |
1466 |
gets the selected method change for the 'method only in version A'. |
|
2507 | 1467 |
|
3147 | 1468 |
<return: MethodChange | nil> |
1469 |
" |
|
11962 | 1470 |
|selIndex change| |
1471 |
||
1472 |
selIndex := self methodsOnlyInASelection value. |
|
1473 |
selIndex notNil ifTrue:[ |
|
1474 |
change := methodsOnlyInAFiltered "self changeSetA" at:selIndex. |
|
2507 | 1475 |
]. |
3753 | 1476 |
^ change |
2507 | 1477 |
! |
1478 |
||
1479 |
selectedChangeInB |
|
3147 | 1480 |
" |
1481 |
gets the selected method change for the 'method only in version B'. |
|
2507 | 1482 |
|
3147 | 1483 |
<return: MethodChange | nil> |
1484 |
" |
|
11962 | 1485 |
|selIndex change| |
1486 |
||
1487 |
selIndex := self methodsOnlyInBSelection value. |
|
1488 |
selIndex notNil ifTrue:[ |
|
1489 |
change := methodsOnlyInBFiltered "self changeSetB" at:selIndex. |
|
2507 | 1490 |
]. |
3753 | 1491 |
^ change |
2507 | 1492 |
! |
1493 |
||
1494 |
selectedChangesInM |
|
3147 | 1495 |
" |
1496 |
get the two method changes for the selected 'changed method'. |
|
2507 | 1497 |
|
3147 | 1498 |
<return: <Array with:MethodChange with:MethodChange | nil>> |
1499 |
" |
|
11962 | 1500 |
|selIndex theTwoChanges| |
1501 |
||
1502 |
selIndex := self methodsChangedSelection value. |
|
1503 |
selIndex notNil ifTrue:[ |
|
1504 |
theTwoChanges := methodsChangedFiltered "self classChangeSet methodsChanged" at:selIndex. |
|
1505 |
^ theTwoChanges. |
|
2507 | 1506 |
]. |
3753 | 1507 |
^ #(nil nil) |
2507 | 1508 |
! |
1509 |
||
2221 | 1510 |
versionA |
3147 | 1511 |
" |
1512 |
gets the first class version to be compared |
|
2221 | 1513 |
|
3147 | 1514 |
<return: String> |
1515 |
" |
|
2731 | 1516 |
classChangeSet isNil ifTrue:[^ 'A']. |
1517 |
^ classChangeSet labelA |
|
2221 | 1518 |
! |
1519 |
||
1520 |
versionB |
|
3147 | 1521 |
" |
1522 |
gets the second class version to be compared |
|
2221 | 1523 |
|
3147 | 1524 |
<return: String> |
1525 |
" |
|
2731 | 1526 |
classChangeSet isNil ifTrue:[^ 'B']. |
1527 |
^ classChangeSet labelB |
|
2221 | 1528 |
! ! |
1529 |
||
1530 |
!VersionDiffBrowser methodsFor:'actions'! |
|
1531 |
||
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1532 |
acceptInLeftView |
11453 | 1533 |
|change| |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1534 |
|
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1535 |
change := self selectedChangesInM first. |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1536 |
self applyChange:change |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1537 |
! |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1538 |
|
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1539 |
acceptInRightView |
11453 | 1540 |
|change| |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1541 |
|
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1542 |
change := self selectedChangesInM second. |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1543 |
self applyChange:change |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1544 |
! |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1545 |
|
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1546 |
acceptInSingleView |
11453 | 1547 |
|change| |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1548 |
|
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1549 |
change := self selectedChangeInA. |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1550 |
change isNil ifTrue:[ |
11453 | 1551 |
change := self selectedChangeInB |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1552 |
]. |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1553 |
self applyChange:change |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1554 |
! |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1555 |
|
17342 | 1556 |
applyChange:change |
1557 |
change notNil ifTrue:[ |
|
1558 |
change apply |
|
1559 |
] |
|
1560 |
! |
|
1561 |
||
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1562 |
applySelectedChangeInA |
11453 | 1563 |
self applyChange:(self selectedChangeInA) |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1564 |
! |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1565 |
|
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1566 |
applySelectedChangeInB |
11453 | 1567 |
self applyChange:(self selectedChangeInB) |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1568 |
! |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1569 |
|
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1570 |
methodInADoubleClicked:index |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1571 |
"/ |change| |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1572 |
"/ |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1573 |
"/ change := self changeSetA at:index. |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1574 |
"/ self browseChange:change |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1575 |
self browseClassInA |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1576 |
! |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1577 |
|
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1578 |
methodInBDoubleClicked:index |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1579 |
"/ |change| |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1580 |
"/ |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1581 |
"/ change := self changeSetB at:index. |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1582 |
"/ self browseChange:change |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1583 |
self browseClassInB |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1584 |
! |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1585 |
|
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1586 |
methodInChangedDoubleClicked:index |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1587 |
self browseClassInM. |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1588 |
! |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1589 |
|
9550
531c77d5d3e3
added: #removeSelectedChangeInBFromImage
Claus Gittinger <cg@exept.de>
parents:
9383
diff
changeset
|
1590 |
removeSelectedChangeInBFromImage |
531c77d5d3e3
added: #removeSelectedChangeInBFromImage
Claus Gittinger <cg@exept.de>
parents:
9383
diff
changeset
|
1591 |
|change cls sel op| |
531c77d5d3e3
added: #removeSelectedChangeInBFromImage
Claus Gittinger <cg@exept.de>
parents:
9383
diff
changeset
|
1592 |
|
531c77d5d3e3
added: #removeSelectedChangeInBFromImage
Claus Gittinger <cg@exept.de>
parents:
9383
diff
changeset
|
1593 |
change := self selectedChangeInB. |
531c77d5d3e3
added: #removeSelectedChangeInBFromImage
Claus Gittinger <cg@exept.de>
parents:
9383
diff
changeset
|
1594 |
change isMethodChange ifTrue:[ |
17347 | 1595 |
(cls := change changeClass) notNil ifTrue:[ |
1596 |
(sel := change changeSelector) notNil ifTrue:[ |
|
1597 |
RefactoryChangeManager notNil ifTrue:[ |
|
1598 |
op := RemoveMethodChange remove:sel from:cls. |
|
1599 |
RefactoryChangeManager performChange: op. |
|
1600 |
] ifFalse:[ |
|
1601 |
cls removeSelector:sel |
|
1602 |
] |
|
1603 |
] |
|
1604 |
] |
|
9550
531c77d5d3e3
added: #removeSelectedChangeInBFromImage
Claus Gittinger <cg@exept.de>
parents:
9383
diff
changeset
|
1605 |
] |
531c77d5d3e3
added: #removeSelectedChangeInBFromImage
Claus Gittinger <cg@exept.de>
parents:
9383
diff
changeset
|
1606 |
|
17347 | 1607 |
"Modified: / 06-02-2017 / 02:31:40 / cg" |
15632 | 1608 |
! |
1609 |
||
1610 |
showLogMessages |
|
15633 | 1611 |
"the UI shows code-diffs if any change is selected, and version log entries otherwise. |
1612 |
But there is no way to deselect any of the method-change-lists, to go back to log-messages, |
|
1613 |
once a method has been looked at. |
|
1614 |
This view-menu entry helps." |
|
1615 |
||
15632 | 1616 |
methodsOnlyInASelection value:nil. |
1617 |
methodsOnlyInBSelection value:nil. |
|
1618 |
methodsChangedSelection value:nil. |
|
2221 | 1619 |
! ! |
1620 |
||
1621 |
!VersionDiffBrowser methodsFor:'aspects'! |
|
1622 |
||
2585 | 1623 |
boxAVisible |
2745 | 1624 |
boxAVisible isNil ifTrue:[ |
11453 | 1625 |
boxAVisible := true asValue. |
2585 | 1626 |
]. |
2745 | 1627 |
^ boxAVisible. |
2585 | 1628 |
! |
1629 |
||
1630 |
boxBVisible |
|
2745 | 1631 |
boxBVisible isNil ifTrue:[ |
11453 | 1632 |
boxBVisible := true asValue. |
2585 | 1633 |
]. |
2745 | 1634 |
^ boxBVisible. |
2585 | 1635 |
! |
1636 |
||
1637 |
boxMVisible |
|
2745 | 1638 |
boxMVisible isNil ifTrue:[ |
11453 | 1639 |
boxMVisible := true asValue. |
2585 | 1640 |
]. |
2745 | 1641 |
^ boxMVisible. |
2585 | 1642 |
! |
1643 |
||
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1644 |
canIncludeExtensionsHolder |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1645 |
"can include extensions ? (menu item enabler)" |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1646 |
|
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1647 |
canIncludeExtensionsHolder isNil ifTrue:[ |
11453 | 1648 |
canIncludeExtensionsHolder := true asValue. |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1649 |
]. |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1650 |
^ canIncludeExtensionsHolder. |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1651 |
! |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1652 |
|
2221 | 1653 |
changedLabelHolder |
3147 | 1654 |
" |
1655 |
aspect for the label of the changed method box. |
|
2221 | 1656 |
|
3147 | 1657 |
<return: ValueHolder on: nil> |
1658 |
" |
|
2221 | 1659 |
|
3147 | 1660 |
changedLabelHolder isNil ifTrue:[ |
11453 | 1661 |
changedLabelHolder := ValueHolder new. |
2221 | 1662 |
]. |
3147 | 1663 |
^ changedLabelHolder. |
2221 | 1664 |
! |
1665 |
||
1666 |
diffTextLabelA |
|
3147 | 1667 |
" |
1668 |
aspect for the label for version A. |
|
2221 | 1669 |
|
3147 | 1670 |
<return: ValueHolder on: nil> |
1671 |
" |
|
2221 | 1672 |
|
3147 | 1673 |
diffTextLabelA isNil ifTrue:[ |
11453 | 1674 |
diffTextLabelA := ValueHolder new. |
2221 | 1675 |
]. |
3147 | 1676 |
^ diffTextLabelA. |
2221 | 1677 |
! |
1678 |
||
1679 |
diffTextLabelB |
|
3147 | 1680 |
" |
1681 |
aspect for the label for version B. |
|
2221 | 1682 |
|
3147 | 1683 |
<return: ValueHolder on: nil> |
1684 |
" |
|
2221 | 1685 |
|
3147 | 1686 |
diffTextLabelB isNil ifTrue:[ |
11453 | 1687 |
diffTextLabelB := ValueHolder new. |
2221 | 1688 |
]. |
3147 | 1689 |
^ diffTextLabelB. |
2221 | 1690 |
! |
1691 |
||
2497 | 1692 |
diffTextView |
3147 | 1693 |
" |
1694 |
return the component for the diff text view. |
|
2497 | 1695 |
|
3147 | 1696 |
<return: HVScrollableView> |
1697 |
" |
|
7364
e7613aec8035
use codeViews (instead of TextViews)
Claus Gittinger <cg@exept.de>
parents:
7108
diff
changeset
|
1698 |
|v| |
e7613aec8035
use codeViews (instead of TextViews)
Claus Gittinger <cg@exept.de>
parents:
7108
diff
changeset
|
1699 |
|
2497 | 1700 |
diffTextView isNil ifTrue:[ |
13951 | 1701 |
v := DiffCodeView new. |
1702 |
v textViewClass:CodeView. |
|
1703 |
v initialize. |
|
1704 |
diffTextView := HVScrollableView |
|
1705 |
forView:v |
|
1706 |
miniScrollerH:true miniScrollerV:false. |
|
1707 |
diffTextView addNextPreviousButtons. |
|
2497 | 1708 |
]. |
1709 |
^ diffTextView |
|
7364
e7613aec8035
use codeViews (instead of TextViews)
Claus Gittinger <cg@exept.de>
parents:
7108
diff
changeset
|
1710 |
|
e7613aec8035
use codeViews (instead of TextViews)
Claus Gittinger <cg@exept.de>
parents:
7108
diff
changeset
|
1711 |
"Modified: / 06-10-2006 / 14:30:42 / cg" |
2497 | 1712 |
! |
1713 |
||
17334 | 1714 |
hideDiffsWithCommentOrFormattingChangeOnly |
1715 |
^ self hideDiffsWithCommentOrFormattingChangeOnlyHolder value |
|
1716 |
||
1717 |
"Created: / 04-02-2017 / 22:54:07 / cg" |
|
1718 |
! |
|
1719 |
||
1720 |
hideDiffsWithCommentOrFormattingChangeOnlyHolder |
|
1721 |
"hide changes which have no semantic effect?" |
|
1722 |
||
1723 |
^ self builder booleanValueAspectFor:#hideDiffsWithCommentOrFormattingChangeOnlyHolder |
|
1724 |
||
1725 |
"Created: / 04-02-2017 / 16:55:12 / cg" |
|
1726 |
! |
|
1727 |
||
11596
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1728 |
includeCategoryChanges |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1729 |
^ self includeCategoryChangesHolder value |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1730 |
|
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1731 |
"Created: / 17-07-2012 / 18:22:34 / cg" |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1732 |
! |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1733 |
|
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1734 |
includeCategoryChanges:aBoolean |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1735 |
self includeCategoryChangesHolder value:aBoolean |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1736 |
|
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1737 |
"Created: / 17-07-2012 / 17:50:41 / cg" |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1738 |
! |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1739 |
|
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1740 |
includeCategoryChangesHolder |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1741 |
includeCategoryChangesHolder isNil ifTrue:[ |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1742 |
includeCategoryChangesHolder := true asValue. |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1743 |
includeCategoryChangesHolder addDependent:self. |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1744 |
]. |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1745 |
^ includeCategoryChangesHolder. |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1746 |
|
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1747 |
"Modified: / 06-03-2012 / 15:36:52 / cg" |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1748 |
"Created: / 17-07-2012 / 16:47:58 / cg" |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1749 |
! |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
1750 |
|
10703 | 1751 |
includeExtensions |
11371
920084723b9c
exclude extension stuff fixed
Claus Gittinger <cg@exept.de>
parents:
10892
diff
changeset
|
1752 |
^ self includeExtensionsHolder value |
10703 | 1753 |
|
1754 |
"Created: / 13-09-2011 / 11:48:51 / cg" |
|
1755 |
! |
|
1756 |
||
1757 |
includeExtensions:aBoolean |
|
11371
920084723b9c
exclude extension stuff fixed
Claus Gittinger <cg@exept.de>
parents:
10892
diff
changeset
|
1758 |
self includeExtensionsHolder value:aBoolean |
10703 | 1759 |
|
1760 |
"Created: / 13-09-2011 / 11:48:41 / cg" |
|
1761 |
! |
|
1762 |
||
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1763 |
includeExtensionsHolder |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1764 |
"only compare base-methods or include extensions ?" |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1765 |
|
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1766 |
includeExtensionsHolder isNil ifTrue:[ |
11453 | 1767 |
includeExtensionsHolder := false asValue. |
1768 |
includeExtensionsHolder addDependent:self. |
|
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1769 |
]. |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1770 |
^ includeExtensionsHolder. |
11371
920084723b9c
exclude extension stuff fixed
Claus Gittinger <cg@exept.de>
parents:
10892
diff
changeset
|
1771 |
|
920084723b9c
exclude extension stuff fixed
Claus Gittinger <cg@exept.de>
parents:
10892
diff
changeset
|
1772 |
"Modified: / 06-03-2012 / 15:36:52 / cg" |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1773 |
! |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
1774 |
|
11958 | 1775 |
includeVersionMethods |
1776 |
^ self includeVersionMethodsHolder value |
|
1777 |
||
1778 |
"Created: / 13-09-2011 / 11:48:51 / cg" |
|
1779 |
! |
|
1780 |
||
1781 |
includeVersionMethods:aBoolean |
|
1782 |
self includeVersionMethodsHolder value:aBoolean |
|
1783 |
||
1784 |
"Created: / 13-09-2011 / 11:48:41 / cg" |
|
1785 |
! |
|
1786 |
||
1787 |
includeVersionMethodsHolder |
|
1788 |
includeVersionMethodsHolder isNil ifTrue:[ |
|
1789 |
includeVersionMethodsHolder := true asValue. |
|
1790 |
includeVersionMethodsHolder addDependent:self. |
|
1791 |
]. |
|
1792 |
^ includeVersionMethodsHolder. |
|
1793 |
||
1794 |
"Modified: / 06-03-2012 / 15:36:52 / cg" |
|
1795 |
"Created: / 17-07-2012 / 16:47:58 / cg" |
|
1796 |
! |
|
1797 |
||
8602 | 1798 |
infoHolder |
1799 |
infoHolder isNil ifTrue:[ |
|
11453 | 1800 |
infoHolder := ValueHolder new. |
8602 | 1801 |
]. |
1802 |
^ infoHolder. |
|
1803 |
! |
|
1804 |
||
2221 | 1805 |
methodText |
3147 | 1806 |
" |
1807 |
aspect for the text in the method text view. |
|
2221 | 1808 |
|
3147 | 1809 |
<return: ValueHolder on: nil> |
1810 |
" |
|
2221 | 1811 |
|
3147 | 1812 |
methodText isNil ifTrue:[ |
11453 | 1813 |
methodText := ValueHolder new. |
2221 | 1814 |
]. |
3147 | 1815 |
^ methodText. |
2221 | 1816 |
! |
1817 |
||
1818 |
methodsChanged |
|
3147 | 1819 |
" |
1820 |
aspect for the 'changed method' list. |
|
2221 | 1821 |
|
3147 | 1822 |
<return: List> |
1823 |
" |
|
2221 | 1824 |
|
3147 | 1825 |
methodsChanged isNil ifTrue:[ |
11453 | 1826 |
methodsChanged := List new. |
2221 | 1827 |
]. |
3147 | 1828 |
^ methodsChanged. |
2221 | 1829 |
! |
1830 |
||
1831 |
methodsChangedSelection |
|
3147 | 1832 |
" |
1833 |
aspect for the selection holder of 'changed method'-list. |
|
2221 | 1834 |
|
3147 | 1835 |
<return: ValueHolder on: nil> |
1836 |
" |
|
2221 | 1837 |
|
3147 | 1838 |
methodsChangedSelection isNil ifTrue:[ |
11453 | 1839 |
methodsChangedSelection := ValueHolder new. |
2221 | 1840 |
]. |
3147 | 1841 |
^ methodsChangedSelection. |
2221 | 1842 |
! |
1843 |
||
1844 |
methodsOnlyInA |
|
6828 | 1845 |
"aspect for the 'method only in version A' list." |
2221 | 1846 |
|
3147 | 1847 |
methodsOnlyInA isNil ifTrue:[ |
11453 | 1848 |
methodsOnlyInA := List new. |
2221 | 1849 |
]. |
3147 | 1850 |
^ methodsOnlyInA. |
6828 | 1851 |
|
1852 |
"Modified: / 25-07-2006 / 11:14:43 / cg" |
|
2221 | 1853 |
! |
1854 |
||
1855 |
methodsOnlyInASelection |
|
6828 | 1856 |
"aspect for the selection holder of 'method only in version A'-list." |
2221 | 1857 |
|
3147 | 1858 |
methodsOnlyInASelection isNil ifTrue:[ |
11453 | 1859 |
methodsOnlyInASelection := ValueHolder new. |
2221 | 1860 |
]. |
3147 | 1861 |
^ methodsOnlyInASelection. |
6828 | 1862 |
|
1863 |
"Modified: / 25-07-2006 / 11:14:53 / cg" |
|
2221 | 1864 |
! |
1865 |
||
1866 |
methodsOnlyInB |
|
6828 | 1867 |
"aspect for the 'method only in version B' list" |
2221 | 1868 |
|
3147 | 1869 |
methodsOnlyInB isNil ifTrue:[ |
11453 | 1870 |
methodsOnlyInB := List new. |
2221 | 1871 |
]. |
3147 | 1872 |
^ methodsOnlyInB. |
6828 | 1873 |
|
1874 |
"Modified: / 25-07-2006 / 11:14:48 / cg" |
|
2221 | 1875 |
! |
1876 |
||
1877 |
methodsOnlyInBSelection |
|
6828 | 1878 |
"aspect for the selection holder of 'method only in version B'-list." |
2221 | 1879 |
|
3147 | 1880 |
methodsOnlyInBSelection isNil ifTrue:[ |
11453 | 1881 |
methodsOnlyInBSelection := ValueHolder new. |
2221 | 1882 |
]. |
3147 | 1883 |
^ methodsOnlyInBSelection. |
6828 | 1884 |
|
1885 |
"Modified: / 25-07-2006 / 11:15:00 / cg" |
|
2221 | 1886 |
! |
1887 |
||
1888 |
onlyInALabelHolder |
|
6828 | 1889 |
"aspect for the label for 'method only in version A'." |
2221 | 1890 |
|
3147 | 1891 |
onlyInALabelHolder isNil ifTrue:[ |
11453 | 1892 |
onlyInALabelHolder := ValueHolder new. |
2221 | 1893 |
]. |
3147 | 1894 |
^ onlyInALabelHolder. |
6828 | 1895 |
|
1896 |
"Modified: / 25-07-2006 / 11:15:12 / cg" |
|
2221 | 1897 |
! |
1898 |
||
1899 |
onlyInBLabelHolder |
|
6828 | 1900 |
"aspect for the label for 'method only in version B'." |
2221 | 1901 |
|
3147 | 1902 |
onlyInBLabelHolder isNil ifTrue:[ |
11453 | 1903 |
onlyInBLabelHolder := ValueHolder new. |
2221 | 1904 |
]. |
3147 | 1905 |
^ onlyInBLabelHolder. |
6828 | 1906 |
|
1907 |
"Modified: / 25-07-2006 / 11:15:07 / cg" |
|
14076 | 1908 |
! |
1909 |
||
1910 |
selectedSymbolicVersionHolder |
|
1911 |
selectedSymbolicVersionHolder isNil ifTrue:[ |
|
1912 |
selectedSymbolicVersionHolder := nil asValue. |
|
1913 |
selectedSymbolicVersionHolder onChangeSend:#selectedSymbolicVersionHolderChanged to:self. |
|
1914 |
]. |
|
1915 |
^ selectedSymbolicVersionHolder |
|
1916 |
! |
|
1917 |
||
1918 |
selectedVersionHolder |
|
1919 |
selectedVersionHolder isNil ifTrue:[ |
|
15621 | 1920 |
selectedVersionHolder := nil asValue. |
14076 | 1921 |
selectedVersionHolder onChangeSend:#selectedVersionHolderChanged to:self. |
1922 |
]. |
|
1923 |
^ selectedVersionHolder |
|
1924 |
! |
|
1925 |
||
18147 | 1926 |
selectedVersionIndex |
1927 |
^ self selectedVersionIndexHolder value ? 0 |
|
1928 |
||
1929 |
"Created: / 23-05-2018 / 11:35:50 / Claus Gittinger" |
|
1930 |
! |
|
1931 |
||
17827 | 1932 |
selectedVersionIndexHolder |
1933 |
selectedVersionIndexHolder isNil ifTrue:[ |
|
1934 |
selectedVersionIndexHolder := nil asValue. |
|
1935 |
selectedVersionIndexHolder onChangeSend:#selectedVersionIndexHolderChanged to:self. |
|
1936 |
]. |
|
1937 |
^ selectedVersionIndexHolder |
|
1938 |
||
1939 |
"Created: / 29-11-2017 / 13:02:49 / cg" |
|
1940 |
! |
|
1941 |
||
14076 | 1942 |
symbolicVersionList |
1943 |
symbolicVersionList isNil ifTrue:[ |
|
1944 |
symbolicVersionList := List new. |
|
1945 |
]. |
|
1946 |
^ symbolicVersionList |
|
1947 |
! |
|
1948 |
||
17827 | 1949 |
versionEntriesList |
1950 |
versionEntriesList isNil ifTrue:[ |
|
1951 |
versionEntriesList := List new. |
|
1952 |
]. |
|
1953 |
^ versionEntriesList |
|
1954 |
||
1955 |
"Created: / 29-11-2017 / 13:06:14 / cg" |
|
1956 |
! |
|
1957 |
||
14076 | 1958 |
versionList |
1959 |
versionList isNil ifTrue:[ |
|
1960 |
versionList := List new. |
|
1961 |
]. |
|
1962 |
^ versionList |
|
2221 | 1963 |
! ! |
1964 |
||
2725 | 1965 |
!VersionDiffBrowser methodsFor:'aspects-exported'! |
2695 | 1966 |
|
1967 |
changedLabelHolder:aValueHolder |
|
1968 |
builder aspectAt:#changedLabelHolder put:aValueHolder. |
|
1969 |
||
1970 |
! |
|
1971 |
||
1972 |
classHolder:aValueHolder |
|
5182 | 1973 |
classHolder notNil ifTrue:[ |
11453 | 1974 |
classHolder removeDependent:self |
2695 | 1975 |
]. |
5182 | 1976 |
classHolder := aValueHolder. |
1977 |
classHolder notNil ifTrue:[ |
|
11453 | 1978 |
classHolder addDependent:self |
2695 | 1979 |
] |
1980 |
! |
|
1981 |
||
1982 |
onlyInALabelHolder:aValueHolder |
|
1983 |
builder aspectAt:#onlyInALabelHolder put:aValueHolder. |
|
1984 |
! |
|
1985 |
||
1986 |
onlyInBLabelHolder:aValueHolder |
|
1987 |
builder aspectAt:#onlyInBLabelHolder put:aValueHolder. |
|
1988 |
! |
|
1989 |
||
1990 |
versionAHolder:aValueHolder |
|
5182 | 1991 |
versionAHolder notNil ifTrue:[ |
11453 | 1992 |
versionAHolder removeDependent:self |
2695 | 1993 |
]. |
5182 | 1994 |
versionAHolder := aValueHolder. |
1995 |
versionAHolder notNil ifTrue:[ |
|
11453 | 1996 |
versionAHolder addDependent:self |
2695 | 1997 |
] |
1998 |
! |
|
1999 |
||
2000 |
versionBHolder:aValueHolder |
|
5182 | 2001 |
versionBHolder notNil ifTrue:[ |
11453 | 2002 |
versionBHolder removeDependent:self |
2695 | 2003 |
]. |
5182 | 2004 |
versionBHolder := aValueHolder. |
2005 |
versionBHolder notNil ifTrue:[ |
|
11453 | 2006 |
versionBHolder addDependent:self |
2695 | 2007 |
] |
2008 |
! ! |
|
2009 |
||
4359 | 2010 |
!VersionDiffBrowser methodsFor:'aspects-menu'! |
2011 |
||
2012 |
canAcceptInCodeView |
|
2013 |
^ self hasChangeSelectedInA |
|
2014 |
or:[self hasChangeSelectedInB |
|
2015 |
or:[self hasChangeSelectedInM]] |
|
2016 |
! |
|
2017 |
||
2018 |
hasChangeSelectedInA |
|
2019 |
^ self selectedChangeInA notNil |
|
2020 |
! |
|
2021 |
||
2022 |
hasChangeSelectedInB |
|
2023 |
^ self selectedChangeInB notNil |
|
2024 |
! |
|
2025 |
||
2026 |
hasChangeSelectedInM |
|
2027 |
^ self methodsChangedSelection value notNil |
|
17341 | 2028 |
! |
2029 |
||
17954 | 2030 |
hasNextVersionInVersionList |
18146 | 2031 |
"versions are sorted youngest first. |
2032 |
If the current selected one is the first, there is no next version" |
|
2033 |
||
18147 | 2034 |
^ (self selectedVersionIndex) > 1. |
2035 |
||
2036 |
"Modified: / 23-05-2018 / 11:36:33 / Claus Gittinger" |
|
17954 | 2037 |
! |
2038 |
||
2039 |
hasPreviousVersionInVersionList |
|
18146 | 2040 |
"versions are sorted youngest first. |
2041 |
If the current selected one is the last, there is no previous version" |
|
2042 |
||
18147 | 2043 |
^ (self selectedVersionIndex) < versionList value size. |
2044 |
||
2045 |
"Modified: / 23-05-2018 / 11:36:38 / Claus Gittinger" |
|
17954 | 2046 |
! |
2047 |
||
17341 | 2048 |
showingClassChangeSet |
2049 |
^ classChangeSet notNil |
|
2050 |
||
2051 |
"Created: / 05-02-2017 / 11:09:18 / cg" |
|
17954 | 2052 |
! |
2053 |
||
2054 |
versionsMenu |
|
2055 |
^ [ |
|
2056 |
|m item nextVersion prevVersion| |
|
2057 |
||
2058 |
m := self class versionsMenu. |
|
2059 |
m := m decodeAsLiteralArray. |
|
2060 |
m findGuiResourcesIn:self. |
|
2061 |
||
2062 |
self hasNextVersionInVersionList ifTrue:[ |
|
18147 | 2063 |
nextVersion := self versionList at:(self selectedVersionIndex - 1). |
17954 | 2064 |
item := m atNameKey:#CompareAgainstNextVersion. |
2065 |
item label:(resources string:(item label,' (%1)') with:nextVersion). |
|
2066 |
]. |
|
2067 |
self hasPreviousVersionInVersionList ifTrue:[ |
|
18147 | 2068 |
prevVersion := self versionList at:(self selectedVersionIndex + 1). |
17954 | 2069 |
item := m atNameKey:#CompareAgainstPreviousVersion. |
2070 |
item label:(resources string:(item label,'(%1)') with:prevVersion). |
|
2071 |
]. |
|
2072 |
m |
|
2073 |
] |
|
18147 | 2074 |
|
2075 |
"Modified: / 23-05-2018 / 11:36:18 / Claus Gittinger" |
|
4359 | 2076 |
! ! |
2077 |
||
2695 | 2078 |
!VersionDiffBrowser methodsFor:'change & update'! |
2079 |
||
17342 | 2080 |
filterChanged |
2081 |
self withExecuteCursorDo:[ |
|
2082 |
|listA listB listM selA selB selM posA posB posM| |
|
2083 |
||
17463 | 2084 |
"/ remember selection and scroll positions |
17342 | 2085 |
listA := (self componentAt:'ListA') scrolledView. |
2086 |
listB := (self componentAt:'ListB') scrolledView. |
|
2087 |
listM := (self componentAt:'ListM') scrolledView. |
|
2088 |
selA := listA selection. |
|
2089 |
selB := listB selection. |
|
2090 |
selM := listM selection. |
|
2091 |
posA := listA firstLineShown. |
|
2092 |
posB := listB firstLineShown. |
|
2093 |
posM := listM firstLineShown. |
|
2094 |
||
2095 |
self updateLists. |
|
2096 |
self enqueueDelayedAction: |
|
2097 |
[ |
|
2098 |
selA notNil ifTrue:[ |
|
2099 |
listA selection:selA |
|
2100 |
] ifFalse:[ |
|
2101 |
listA scrollToLine:posA |
|
2102 |
]. |
|
2103 |
selB notNil ifTrue:[ |
|
2104 |
listB selection:selB |
|
2105 |
] ifFalse:[ |
|
2106 |
listB scrollToLine:posB |
|
2107 |
]. |
|
2108 |
selM notNil ifTrue:[ |
|
2109 |
listM selection:selM |
|
2110 |
] ifFalse:[ |
|
2111 |
listM scrollToLine:posM |
|
2112 |
]. |
|
2113 |
]. |
|
2114 |
]. |
|
2115 |
||
2116 |
"Created: / 05-02-2017 / 10:19:41 / cg" |
|
17463 | 2117 |
"Modified (format): / 14-03-2017 / 16:30:58 / cg" |
17342 | 2118 |
! |
2119 |
||
12512 | 2120 |
methodsChangedSelectionChanged |
2121 |
" |
|
2122 |
the selection in the list of the 'changed methods' changed. |
|
2123 |
Reset the selection of the two other lists and calc the method change. |
|
2124 |
||
2125 |
<return: self> |
|
2126 |
" |
|
2127 |
|sel changeA changeB methodsChanged parseError treeA treeB| |
|
2128 |
||
17342 | 2129 |
"/ self information:''. |
12512 | 2130 |
|
2131 |
sel := self methodsChangedSelection value. |
|
2132 |
sel notNil ifTrue:[ |
|
2133 |
self methodsOnlyInASelection value:nil. |
|
2134 |
self methodsOnlyInBSelection value:nil. |
|
2135 |
methodsChanged := self classChangeSet methodsChanged. |
|
2136 |
changeA := (methodsChangedFiltered at:sel) first. |
|
2137 |
changeB := (methodsChangedFiltered at:sel) second. |
|
2138 |
self withReadCursorDo:[ |
|
17827 | 2139 |
|info| |
18000 | 2140 |
|
2141 |
info := self shortChangeInfoFor:changeA and:changeB. |
|
17827 | 2142 |
info notNil ifTrue:[ |
2143 |
self infoHolder value:info |
|
2144 |
]. |
|
12512 | 2145 |
self diffTextView |
2146 |
text1:(changeA prettyPrintedSource) |
|
2147 |
text2:(changeB prettyPrintedSource) |
|
2148 |
]. |
|
2149 |
self showDiffTextView. |
|
2150 |
self diffTextView moveToNextChanged |
|
15621 | 2151 |
] ifFalse:[ |
2152 |
self showVersionInfoIfNothingSelected |
|
12512 | 2153 |
] |
2154 |
||
2155 |
"Modified: / 03-09-1999 / 15:01:30 / ps" |
|
17827 | 2156 |
"Modified: / 29-11-2017 / 12:56:28 / cg" |
12512 | 2157 |
! |
2158 |
||
2159 |
methodsOnlyInASelectionChanged |
|
2160 |
" |
|
2161 |
the selection in the list of the 'method only in version A' changed. |
|
2162 |
Reset the selection of the two other lists and show the method in |
|
2163 |
the text view. |
|
2164 |
||
2165 |
<return: self> |
|
2166 |
" |
|
2167 |
|change sel| |
|
2168 |
||
17342 | 2169 |
"/ self information:''. |
12512 | 2170 |
|
2171 |
sel := self methodsOnlyInASelection value. |
|
2172 |
sel notNil ifTrue:[ |
|
2173 |
self methodsOnlyInBSelection value:nil. |
|
2174 |
self methodsChangedSelection value:nil. |
|
2175 |
change := methodsOnlyInAFiltered "self changeSetA" at:sel. |
|
2176 |
self methodText value:(change prettyPrintedSource). |
|
2177 |
self showSingleTextView. |
|
15621 | 2178 |
] ifFalse:[ |
2179 |
self showVersionInfoIfNothingSelected |
|
2180 |
]. |
|
12512 | 2181 |
|
17342 | 2182 |
"Modified: / 05-02-2017 / 12:08:07 / cg" |
12512 | 2183 |
! |
2184 |
||
2185 |
methodsOnlyInBSelectionChanged |
|
2186 |
" |
|
2187 |
the selection in the list of the 'method only in version B' changed. |
|
2188 |
Reset the selection of the two other lists and show the method in |
|
2189 |
the text view. |
|
2190 |
||
2191 |
<return: self> |
|
2192 |
" |
|
2193 |
|change sel| |
|
2194 |
||
17342 | 2195 |
"/ self information:''. |
12512 | 2196 |
|
2197 |
sel := self methodsOnlyInBSelection value. |
|
2198 |
sel notNil ifTrue:[ |
|
2199 |
self methodsOnlyInASelection value:nil. |
|
2200 |
self methodsChangedSelection value:nil. |
|
2201 |
change := methodsOnlyInBFiltered "self changeSetB" at:sel. |
|
2202 |
self methodText value:(change prettyPrintedSource). |
|
2203 |
self showSingleTextView. |
|
15621 | 2204 |
] ifFalse:[ |
2205 |
self showVersionInfoIfNothingSelected |
|
2206 |
]. |
|
12512 | 2207 |
|
17342 | 2208 |
"Modified: / 05-02-2017 / 11:49:22 / cg" |
12512 | 2209 |
! |
2210 |
||
14076 | 2211 |
selectedSymbolicVersionHolderChanged |
2212 |
self selectedVersionHolder |
|
2213 |
value:(symbolicToVersionMapping at:( self selectedSymbolicVersionHolder value )) |
|
2214 |
! |
|
2215 |
||
2216 |
selectedVersionHolderChanged |
|
2217 |
||
2218 |
self withWaitCursorDo:[ |
|
15621 | 2219 |
|idx prevSelectionIndexA prevSelectionIndexB prevSelectionIndexDifferent |
2220 |
previousChange previousClass previousSelector| |
|
2221 |
||
2222 |
"/ remember the previous selected method |
|
2223 |
(prevSelectionIndexA := methodsOnlyInASelection value) notNil ifTrue:[ |
|
2224 |
previousChange := methodsOnlyInAFiltered at:prevSelectionIndexA. |
|
2225 |
]. |
|
2226 |
(prevSelectionIndexB := methodsOnlyInBSelection value) notNil ifTrue:[ |
|
2227 |
previousChange := methodsOnlyInBFiltered at:prevSelectionIndexB. |
|
2228 |
]. |
|
2229 |
(prevSelectionIndexDifferent := methodsChangedSelection value) notNil ifTrue:[ |
|
2230 |
previousChange := (methodsChangedFiltered at:prevSelectionIndexDifferent) first. |
|
2231 |
]. |
|
2232 |
previousChange notNil ifTrue:[ |
|
2233 |
previousClass := previousChange className. |
|
17347 | 2234 |
previousSelector := previousChange changeSelector. |
15621 | 2235 |
]. |
2236 |
||
2237 |
"/ possibly expand a contracted list |
|
14076 | 2238 |
selectedVersionHolder value = '...' ifTrue:[ |
2239 |
idx := self versionList size. |
|
2240 |
self updateVersionList. |
|
2241 |
selectedVersionHolder setValue:(self versionList at:idx). |
|
2242 |
]. |
|
2243 |
||
15621 | 2244 |
"/ update the view |
14076 | 2245 |
self |
2246 |
setupForClass:classHolder value |
|
2247 |
againstVersion:(selectedVersionHolder value). |
|
15621 | 2248 |
|
2249 |
"/ try to reselect the same method again. |
|
2250 |
previousChange notNil ifTrue:[ |
|
2251 |
"/ search a change for that in the three lists |
|
2252 |
||
2253 |
(idx := methodsChangedFiltered findFirst:[:entry | entry first className = previousClass |
|
17347 | 2254 |
and:[ entry first changeSelector = previousSelector]]) ~= 0 |
15621 | 2255 |
ifTrue:[ |
2256 |
methodsChangedSelection value:idx. self methodsChangedSelectionChanged. |
|
2257 |
] ifFalse:[ |
|
2258 |
(idx := methodsOnlyInAFiltered findFirst:[:chg | chg className = previousClass |
|
17347 | 2259 |
and:[ chg changeSelector = previousSelector]]) ~= 0 |
15621 | 2260 |
ifTrue:[ |
2261 |
methodsOnlyInASelection value:idx. self methodsOnlyInASelectionChanged |
|
2262 |
] ifFalse:[ |
|
2263 |
(idx := methodsOnlyInBFiltered findFirst:[:chg | chg className = previousClass |
|
17347 | 2264 |
and:[ chg changeSelector = previousSelector]]) ~= 0 |
15621 | 2265 |
ifTrue:[ |
2266 |
methodsOnlyInBSelection value:idx. self methodsOnlyInBSelectionChanged. |
|
2267 |
] |
|
2268 |
] |
|
2269 |
]. |
|
2270 |
]. |
|
2271 |
self showVersionInfoIfNothingSelected |
|
14076 | 2272 |
] |
17347 | 2273 |
|
2274 |
"Modified: / 06-02-2017 / 02:32:24 / cg" |
|
14076 | 2275 |
! |
2276 |
||
17827 | 2277 |
selectedVersionIndexHolderChanged |
2278 |
self selectedVersionHolder value:(versionList at:self selectedVersionIndexHolder value) |
|
2279 |
||
2280 |
"Created: / 29-11-2017 / 13:04:12 / cg" |
|
2281 |
! |
|
2282 |
||
18000 | 2283 |
shortChangeInfoFor:changeA and:changeB |
2284 |
|parseError treeA treeB sameSuperClass sameClassName sameInstVars sameClassVars samePools sameCategory append info| |
|
2285 |
||
2286 |
"/ see if the semantics have changed (or only prettyPrinted) |
|
2287 |
RBParser notNil ifTrue:[ |
|
2288 |
(classChangeSet notNil |
|
2289 |
and:[ classChangeSet classBeingCompared notNil ]) ifTrue:[ |
|
2290 |
parseError := false. |
|
2291 |
changeA isMethodChange ifTrue:[ |
|
2292 |
changeB isMethodChange ifFalse:[^ 'Different change types']. |
|
2293 |
||
2294 |
treeA := RBParser parseMethod:(changeA source) onError: [:str :pos | parseError := true]. |
|
2295 |
treeB := RBParser parseMethod:(changeB source) onError: [:str :pos | parseError := true]. |
|
2296 |
parseError ifFalse:[ |
|
2297 |
(treeA equalTo:treeB withMapping:(Dictionary new)) ifTrue:[ |
|
2298 |
^ 'Methods only differ in formatting / comments'. |
|
2299 |
]. |
|
2300 |
(treeA semanticallyEqualTo:treeB withMapping:(Dictionary new)) ifTrue:[ |
|
2301 |
^ 'Methods seem to do the same'. |
|
2302 |
]. |
|
2303 |
^ 'Methods are different'. |
|
2304 |
]. |
|
2305 |
self infoHolder value:'ParseError while comparing - please check the code'. |
|
2306 |
^ nil |
|
2307 |
]. |
|
2308 |
||
2309 |
changeA isClassDefinitionChange ifTrue:[ |
|
2310 |
changeB isClassDefinitionChange ifFalse:[^ 'Different change types']. |
|
2311 |
treeA := RBParser parseExpression:(changeA source) onError: [:str :pos | parseError := true]. |
|
2312 |
treeB := RBParser parseExpression:(changeB source) onError: [:str :pos | parseError := true]. |
|
2313 |
(treeA semanticallyEqualTo:treeB withMapping:(Dictionary new)) ifTrue:[ |
|
2314 |
^ 'Class definitions seem to do the same'. |
|
2315 |
]. |
|
2316 |
sameSuperClass := treeA receiver = treeB receiver. |
|
2317 |
sameClassName := treeA arg1 = treeB arg1. |
|
2318 |
sameInstVars := treeA arg2 = treeB arg2. |
|
2319 |
sameClassVars := treeA arg3 = treeB arg3. |
|
2320 |
samePools := (treeA arg:4) = (treeB arg:4). |
|
2321 |
sameCategory := (treeA arg:5) = (treeB arg:5). |
|
2322 |
||
2323 |
info := ''. |
|
2324 |
append := [:s | info := info isEmpty ifTrue:[s] ifFalse:[info,', ',s]]. |
|
2325 |
||
2326 |
sameSuperClass ifFalse:[ |
|
2327 |
append value:'Superclass'. |
|
2328 |
]. |
|
2329 |
sameClassName ifFalse:[ |
|
2330 |
append value:'Name'. |
|
2331 |
]. |
|
2332 |
sameInstVars ifFalse:[ |
|
2333 |
append value:'Instvars'. |
|
2334 |
]. |
|
2335 |
sameClassVars ifFalse:[ |
|
2336 |
append value:'Classvars'. |
|
2337 |
]. |
|
2338 |
samePools ifFalse:[ |
|
2339 |
append value:'SharedPools'. |
|
2340 |
]. |
|
2341 |
sameCategory ifFalse:[ |
|
2342 |
append value:'Category'. |
|
2343 |
]. |
|
2344 |
info notEmpty ifTrue:[ ^ 'different ',info ]. |
|
2345 |
^ 'Class definitions are different.'. |
|
2346 |
]. |
|
2347 |
||
2348 |
changeA isClassCommentChange ifTrue:[ |
|
2349 |
changeB isClassCommentChange ifFalse:[^ 'Different change types']. |
|
2350 |
changeA comment = changeB comment ifFalse:[^ 'Different class comment']. |
|
2351 |
^ 'Same comment'. |
|
2352 |
]. |
|
2353 |
||
18034 | 2354 |
changeA isPrimitiveChange ifTrue:[ |
2355 |
changeB isPrimitiveChange ifFalse:[^ 'Different change types']. |
|
2356 |
(changeA sameAs:changeB) ifFalse:[^ 'Different class primitive']. |
|
2357 |
^ 'Same primitive'. |
|
2358 |
]. |
|
18000 | 2359 |
]. |
2360 |
]. |
|
2361 |
^ nil |
|
18034 | 2362 |
|
2363 |
"Modified: / 09-04-2018 / 12:21:27 / stefan" |
|
18000 | 2364 |
! |
2365 |
||
2695 | 2366 |
update:something with:parameter from:changedObject |
11596
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
2367 |
((changedObject == includeExtensionsHolder) |
11958 | 2368 |
or:[ changedObject == includeCategoryChangesHolder |
2369 |
or:[ changedObject == includeVersionMethodsHolder ]]) ifTrue:[ |
|
11596
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
2370 |
self updateLists. |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
2371 |
^ self |
11371
920084723b9c
exclude extension stuff fixed
Claus Gittinger <cg@exept.de>
parents:
10892
diff
changeset
|
2372 |
]. |
920084723b9c
exclude extension stuff fixed
Claus Gittinger <cg@exept.de>
parents:
10892
diff
changeset
|
2373 |
|
11453 | 2374 |
(changedObject == classHolder |
2695 | 2375 |
or:[changedObject == versionAHolder |
11453 | 2376 |
or:[changedObject == versionBHolder]]) ifTrue:[ |
11596
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
2377 |
self setupForClass:(classHolder value) versionA:(versionAHolder value) versionB:(versionBHolder value). |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
2378 |
^ self |
2695 | 2379 |
]. |
2380 |
super update:something with:parameter from:changedObject |
|
10703 | 2381 |
|
11596
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
2382 |
"Modified: / 17-07-2012 / 17:48:58 / cg" |
17342 | 2383 |
! |
2384 |
||
2385 |
updateLabels |
|
2386 |
"update the labels of the diff text view. Show the version numbers |
|
2387 |
of the class." |
|
2388 |
||
2389 |
|theVersionA theVersionB builder makeLabel| |
|
2390 |
||
2391 |
makeLabel := |
|
2392 |
[:theVersion | |
|
2393 |
|label date author revInfo| |
|
2394 |
||
2395 |
label := theVersion. |
|
2396 |
versionInfoList notNil ifTrue:[ |
|
2397 |
revInfo := versionInfoList detect:[:info | (info at:#revision) = theVersion] ifNone:nil. |
|
2398 |
revInfo notNil ifTrue:[ |
|
2399 |
date := (Timestamp readGeneralizedFrom:(revInfo at:#date)) asDate. |
|
2400 |
author := (revInfo at:#author) ? '?'. |
|
2401 |
label := theVersion , ' [',date printString,' by ',author,']'. |
|
2402 |
]. |
|
2403 |
]. |
|
2404 |
label |
|
2405 |
]. |
|
2406 |
||
2407 |
builder := self builder. |
|
2408 |
(theVersionA := self versionA) notNil ifTrue:[ |
|
2409 |
self diffTextLabelA value:(makeLabel value:theVersionA). |
|
2410 |
self onlyInALabelHolder value:('Only in ' , theVersionA). |
|
2411 |
(self componentAt:#OnlyInALabel) |
|
2412 |
foregroundColor:Color white; |
|
2413 |
backgroundColor:Color green darkened. |
|
2414 |
]. |
|
2415 |
(theVersionB :=self versionB) notNil ifTrue:[ |
|
2416 |
self diffTextLabelB value:(makeLabel value:theVersionB). |
|
2417 |
self onlyInBLabelHolder value:('Only in ' , theVersionB). |
|
2418 |
(self componentAt:#OnlyInBLabel) |
|
2419 |
foregroundColor:Color white; |
|
2420 |
backgroundColor:Color red darkened. |
|
2421 |
]. |
|
2422 |
||
2423 |
self changedLabelHolder value:('Different'). |
|
2424 |
||
2425 |
"Modified: / 13-10-2006 / 01:07:08 / cg" |
|
2426 |
! |
|
2427 |
||
2428 |
updateLists |
|
17944 | 2429 |
"walk over the changeSet(s) and setup the 3 lists: |
17342 | 2430 |
onlyInA, changed, onlyInB, |
2431 |
Optionally filter extensions, categoryChanges and versionMethods, |
|
17350 | 2432 |
changes with same semantic (variable renames) |
17342 | 2433 |
and changes which have been explicitly filtered by the user." |
2434 |
||
2435 |
|listOnlyInA listOnlyInB listChanged |
|
17944 | 2436 |
printStringGenerator "sortBlockForChangeLists" filteredList |
17342 | 2437 |
numIgnoredExtensions numIgnoredVersionMethods |
2438 |
info needFilter entryIsForCommentOrFormatOnly |
|
17347 | 2439 |
isIgnoredChange isIgnored1 isIgnored2 isIgnored3 isIgnored4 isIgnored5| |
17342 | 2440 |
|
2441 |
printStringGenerator := [:aChange | self printStringForChange:aChange]. |
|
17944 | 2442 |
"/ sortBlockForChangeLists := [:a :b | (printStringGenerator value:a) < (printStringGenerator value:b)]. |
17342 | 2443 |
|
2444 |
numIgnoredExtensions := numIgnoredVersionMethods := 0. |
|
2445 |
needFilter := self includeExtensions not |
|
2446 |
or:[self includeCategoryChanges not |
|
2447 |
or:[self includeVersionMethods not]]. |
|
2448 |
||
2449 |
needFilter ifTrue:[ |
|
2450 |
isIgnoredChange := |
|
2451 |
[:change | |
|
2452 |
|packageOfMethodInChange packageOfMethodInImage changeMethod ignored| |
|
2453 |
||
2454 |
ignored := false. |
|
2455 |
change isMethodCodeChange ifTrue:[ |
|
2456 |
self includeExtensions ifFalse:[ |
|
2457 |
packageOfMethodInChange := change package. |
|
2458 |
(packageOfMethodInChange notNil |
|
2459 |
and:[ packageOfMethodInChange ~= PackageId noProjectID |
|
2460 |
and:[ change changeClass notNil |
|
2461 |
and:[ packageOfMethodInChange ~= change changeClass package ]]]) ifTrue:[ |
|
2462 |
ignored := true |
|
2463 |
]. |
|
2464 |
||
2465 |
changeMethod := change changeMethod. |
|
2466 |
changeMethod notNil ifTrue:[ |
|
2467 |
packageOfMethodInImage := changeMethod package. |
|
2468 |
(true "packageOfMethodInImage notNil" |
|
2469 |
and:[ packageOfMethodInImage ~= PackageId noProjectID |
|
2470 |
and:[ packageOfMethodInImage ~= changeMethod mclass package ]]) ifTrue:[ |
|
2471 |
ignored := true |
|
2472 |
]. |
|
2473 |
]. |
|
2474 |
numIgnoredExtensions := numIgnoredExtensions + (ignored ifTrue:[1] ifFalse:[0]). |
|
2475 |
]. |
|
2476 |
self includeVersionMethods ifFalse:[ |
|
2477 |
(change isMethodChangeForVersionMethod |
|
2478 |
or:[ change isMethodChangeForExtensionsVersionMethod ]) ifTrue:[ |
|
2479 |
ignored := true. |
|
2480 |
numIgnoredVersionMethods := numIgnoredVersionMethods + 1 |
|
2481 |
]. |
|
2482 |
]. |
|
2483 |
] ifFalse:[ |
|
2484 |
change isMethodCategoryChange ifTrue:[ |
|
2485 |
self includeCategoryChanges ifFalse:[ |
|
2486 |
ignored := true |
|
2487 |
] |
|
2488 |
]. |
|
2489 |
]. |
|
2490 |
ignored. |
|
2491 |
]. |
|
2492 |
] ifFalse:[ |
|
2493 |
isIgnoredChange := [:change | false]. |
|
2494 |
]. |
|
2495 |
||
2496 |
filteredClasses notEmptyOrNil ifTrue:[ |
|
2497 |
isIgnored1 := isIgnoredChange. |
|
2498 |
isIgnoredChange := |
|
2499 |
[:change | |
|
2500 |
(isIgnored1 value:change) |
|
2501 |
or:[ |
|
2502 |
filteredClasses includes:change nonMetaClassName |
|
2503 |
] |
|
2504 |
]. |
|
2505 |
]. |
|
2506 |
filteredSelectors notEmptyOrNil ifTrue:[ |
|
2507 |
isIgnored2 := isIgnoredChange. |
|
2508 |
isIgnoredChange := |
|
2509 |
[:change | |
|
2510 |
(isIgnored2 value:change) |
|
2511 |
or:[ filteredSelectors includes:change changeSelector ] |
|
2512 |
]. |
|
2513 |
]. |
|
2514 |
filteredMethods notEmptyOrNil ifTrue:[ |
|
2515 |
isIgnored3 := isIgnoredChange. |
|
2516 |
isIgnoredChange := |
|
2517 |
[:change | |
|
2518 |
(isIgnored3 value:change) |
|
17348 | 2519 |
or:[ |
2520 |
filteredMethods contains:[:filter | |
|
2521 |
|filterSelector filterClassName| |
|
2522 |
||
17349 | 2523 |
"/ migration - will be always an assoc |
17348 | 2524 |
filter isAssociation ifTrue:[ |
2525 |
filterClassName := filter key. |
|
2526 |
filterSelector := filter value. |
|
2527 |
] ifFalse:[ |
|
2528 |
filterClassName := filter className. |
|
2529 |
filterSelector := filter changeSelector. |
|
2530 |
]. |
|
2531 |
((change className = filterClassName) |
|
2532 |
and:[change changeSelector = filterSelector]) |
|
2533 |
] |
|
2534 |
] |
|
2535 |
]. |
|
17342 | 2536 |
]. |
2537 |
||
17347 | 2538 |
classPatternFilters notEmptyOrNil ifTrue:[ |
17344 | 2539 |
isIgnored4 := isIgnoredChange. |
2540 |
isIgnoredChange := |
|
2541 |
[:change | |
|
2542 |
(isIgnored4 value:change) |
|
17348 | 2543 |
or:[ |
2544 |
classPatternFilters contains:[:pattern | pattern match:change nonMetaClassName] |
|
2545 |
] |
|
17347 | 2546 |
]. |
2547 |
]. |
|
2548 |
||
2549 |
selectorPatternFilters notEmptyOrNil ifTrue:[ |
|
2550 |
isIgnored5 := isIgnoredChange. |
|
2551 |
isIgnoredChange := |
|
2552 |
[:change | |
|
2553 |
(isIgnored5 value:change) |
|
17348 | 2554 |
or:[ |
2555 |
selectorPatternFilters contains:[:pattern | pattern match:change changeSelector] |
|
2556 |
] |
|
17344 | 2557 |
]. |
2558 |
]. |
|
2559 |
||
17342 | 2560 |
listOnlyInA := filteredList := self methodsOnlyInA. |
2561 |
listOnlyInA removeAll. |
|
2562 |
classChangeSet notNil ifTrue:[ |
|
2563 |
"/ classChangeSet methodsOnlyInA sort:sortBlockForChangeLists. |
|
2564 |
filteredList := classChangeSet methodsOnlyInA reject:isIgnoredChange. |
|
2565 |
listOnlyInA addAll: (filteredList collect:printStringGenerator). |
|
2566 |
]. |
|
2567 |
methodsOnlyInAFiltered := filteredList. |
|
2568 |
||
2569 |
listOnlyInB := filteredList := self methodsOnlyInB. |
|
2570 |
listOnlyInB removeAll. |
|
2571 |
classChangeSet notNil ifTrue:[ |
|
2572 |
"/ classChangeSet methodsOnlyInB sort:sortBlockForChangeLists. |
|
2573 |
filteredList := classChangeSet methodsOnlyInB reject:isIgnoredChange. |
|
2574 |
listOnlyInB addAll: (filteredList collect:printStringGenerator). |
|
2575 |
]. |
|
2576 |
methodsOnlyInBFiltered := filteredList. |
|
2577 |
||
2578 |
entryIsForCommentOrFormatOnly := |
|
2579 |
[:entry | |
|
2580 |
|commentOnly changeA changeB mthdA mthdB| |
|
2581 |
||
2582 |
commentOnly := false. |
|
2583 |
||
2584 |
changeA := entry first. |
|
2585 |
changeB := entry second. |
|
2586 |
changeA isMethodChange ifTrue:[ |
|
2587 |
changeA isMethodCodeChange ifTrue:[ |
|
17414 | 2588 |
RBCodeDuplicationRule notNil ifTrue:[ |
2589 |
mthdA := RBMethod forMethodChange:changeA. |
|
2590 |
mthdB := RBMethod forMethodChange:changeB. |
|
2591 |
commentOnly := RBCodeDuplicationRule hasMethod:mthdA sameSemanticAs:mthdB. |
|
2592 |
]. |
|
17342 | 2593 |
] ifFalse:[ |
2594 |
changeA isMethodCategoryChange ifTrue:[ |
|
2595 |
commentOnly := true |
|
2596 |
] ifFalse:[ |
|
2597 |
self breakPoint:#cg |
|
2598 |
]. |
|
2599 |
]. |
|
2600 |
] ifFalse:[ |
|
2601 |
changeA isClassDefinitionChange ifTrue:[ |
|
2602 |
changeA source = changeB source ifTrue:[ |
|
2603 |
commentOnly := true |
|
2604 |
] ifFalse:[ |
|
2605 |
self breakPoint:#cg |
|
2606 |
]. |
|
2607 |
] ifFalse:[ |
|
2608 |
self breakPoint:#cg |
|
2609 |
]. |
|
2610 |
]. |
|
2611 |
commentOnly |
|
2612 |
]. |
|
2613 |
||
2614 |
listChanged := filteredList := self methodsChanged. |
|
2615 |
listChanged removeAll. |
|
2616 |
classChangeSet notNil ifTrue:[ |
|
2617 |
"/ classChangeSet methodsChanged sort:[:a :b | sortBlockForChangeLists value:a first value:b first]. |
|
2618 |
filteredList := classChangeSet methodsChanged reject:[:entry | isIgnoredChange value:entry first]. |
|
2619 |
||
2620 |
(self hideDiffsWithCommentOrFormattingChangeOnly) ifTrue:[ |
|
2621 |
filteredList := filteredList reject:entryIsForCommentOrFormatOnly. |
|
2622 |
listChanged addAll: (filteredList collect:[:entry| printStringGenerator value:(entry first)]). |
|
2623 |
] ifFalse:[ |
|
2624 |
filteredList do:[:entry| |
|
2625 |
|string| |
|
2626 |
||
2627 |
string := printStringGenerator value:(entry first). |
|
2628 |
(entryIsForCommentOrFormatOnly value:entry) ifTrue:[ |
|
2629 |
string := string withColor:Color grey. |
|
2630 |
]. |
|
2631 |
listChanged add:string |
|
2632 |
]. |
|
2633 |
]. |
|
2634 |
]. |
|
2635 |
methodsChangedFiltered := filteredList. |
|
2636 |
||
2637 |
self boxAVisible value:(listOnlyInA notEmpty). |
|
2638 |
self boxBVisible value:(listOnlyInB notEmpty). |
|
2639 |
||
2640 |
self boxMVisible value:(listChanged isEmpty |
|
2641 |
and: [(listOnlyInA notEmpty |
|
2642 |
or:[listOnlyInB notEmpty])]) not. |
|
2643 |
||
2644 |
info := '%1 changed, %2 added, %3 removed%4' |
|
2645 |
bindWith:methodsChangedFiltered size |
|
2646 |
with:methodsOnlyInB size |
|
2647 |
with:methodsOnlyInA size |
|
2648 |
with:(isIgnoredChange isNil |
|
2649 |
ifTrue:[''] |
|
2650 |
ifFalse:[' (filtered)']). |
|
2651 |
||
2652 |
numIgnoredExtensions ~~ 0 ifTrue:[ |
|
2653 |
info := info , ('; %1 extension methods ignored. ' bindWith:numIgnoredExtensions). |
|
2654 |
]. |
|
2655 |
numIgnoredVersionMethods ~~ 0 ifTrue:[ |
|
2656 |
info := info , ('; %1 version methods ignored.' bindWith:numIgnoredVersionMethods). |
|
2657 |
]. |
|
2658 |
self information:info |
|
2659 |
||
17350 | 2660 |
"Modified (comment): / 06-02-2017 / 10:09:11 / cg" |
17414 | 2661 |
"Modified (format): / 16-02-2017 / 11:11:49 / stefan" |
17342 | 2662 |
! |
2663 |
||
2664 |
updateVersionList |
|
2665 |
"asks the classes source code manager for a list of revisions; |
|
2666 |
construct versionInfoList (containing the full info), versionList (containing version numbers only) |
|
2667 |
and tagList (containing symbolic names only). |
|
2668 |
This is only used when comparing multiple versions." |
|
2669 |
||
2670 |
|cls sourceCodeManager numShown newNumShown numOverallRevisions |
|
17827 | 2671 |
partialLog newestRev revisions revisionEntries symbolicNames |
17342 | 2672 |
stableRevision releasedRevision tagList logMessages| |
2673 |
||
2674 |
cls := classHolder value. |
|
2675 |
sourceCodeManager := cls sourceCodeManager. |
|
2676 |
||
2677 |
numShown := versionList size - 1. |
|
2678 |
newNumShown := (numShown + 50) max:30. |
|
2679 |
partialLog := sourceCodeManager revisionLogOf:cls numberOfRevisions:newNumShown. |
|
2680 |
||
2681 |
partialLog isNil ifTrue:[ |
|
2682 |
self warn:'Could not find/access the container for class ',cls name,' in the repository. |
|
2683 |
This could be due to: |
|
2684 |
- invalid/wrong CVS-Root setting |
|
2685 |
- missing CVS access rights |
|
2686 |
(no access / not logged in) |
|
2687 |
- changed CVSRoot after compilation |
|
2688 |
(i.e. wrong CVS-path in classes version method) |
|
2689 |
'. |
|
2690 |
^ nil |
|
2691 |
]. |
|
2692 |
||
2693 |
numOverallRevisions := partialLog at:#numberOfRevisions. |
|
2694 |
newestRev := partialLog at:#newestRevision. |
|
2695 |
revisions := partialLog at:#revisions. |
|
2696 |
||
2697 |
"/ fetch the symbolic (tag) version list |
|
2698 |
symbolicNames := partialLog at:#symbolicNames ifAbsent:[]. |
|
2699 |
symbolicNames notNil ifTrue:[ |
|
2700 |
stableRevision := symbolicNames at:'stable' ifAbsent:[]. |
|
2701 |
releasedRevision := symbolicNames at:'released' ifAbsent:[]. |
|
2702 |
tagList := ((symbolicNames associations sort:[:a :b | a key < b key "self versionString:(a value) isLessThan:(b value)"]) |
|
2703 |
collect:[:assoc | assoc key]) reverse. |
|
2704 |
tagList remove:'stable' ifAbsent:[]. |
|
2705 |
tagList notEmpty ifTrue:[tagList addFirst:'-']. |
|
2706 |
tagList addFirst:'stable'. |
|
2707 |
]. |
|
2708 |
logMessages := Dictionary new. |
|
2709 |
||
2710 |
versionInfoList := revisions. |
|
2711 |
||
2712 |
"/ items := versionInfoList collect:[:each | |rev date who flag| |
|
2713 |
"/ rev := each at:#revision. |
|
2714 |
"/ logMessages at:rev put:(each at:#logMessage). |
|
2715 |
"/ date := (each at:#date ifAbsent:nil) ? '?'. |
|
2716 |
"/ who := (each at:#author ifAbsent:nil) ? '?'. |
|
2717 |
"/ rev = stableRevision ifTrue:[ |
|
2718 |
"/ flag := ' Stable' allBold. |
|
2719 |
"/ ] ifFalse:[rev = releasedRevision ifTrue:[ |
|
2720 |
"/ flag := ' Released' allBold. |
|
2721 |
"/ ] ifFalse:[ |
|
2722 |
"/ flag := ' ' |
|
2723 |
"/ ]]. |
|
2724 |
"/ rev allBold , flag, ' [' , date , ' by ' , who , ']' |
|
2725 |
"/ ]. |
|
2726 |
"/ |
|
2727 |
"/ revisions := revisions collect:[:eachItem | eachItem at:#revision]. |
|
17827 | 2728 |
revisions := OrderedCollection new. |
2729 |
revisionEntries := OrderedCollection new. |
|
2730 |
versionInfoList do:[:eachVersionInfo | |
|
2731 |
revisions add:eachVersionInfo revision. |
|
2732 |
revisionEntries add:( |
|
2733 |
'%1 (by %2 at %3)' |
|
2734 |
bindWith:(eachVersionInfo revision) |
|
2735 |
with:(eachVersionInfo user) |
|
2736 |
with:(eachVersionInfo date)). |
|
2737 |
]. |
|
17342 | 2738 |
|
2739 |
revisions size < numOverallRevisions ifTrue:[ |
|
17931 | 2740 |
revisions add:'...'. |
2741 |
revisionEntries add:'...'. |
|
17342 | 2742 |
]. |
2743 |
symbolicToVersionMapping := symbolicNames. |
|
2744 |
self symbolicVersionList contents:tagList. |
|
2745 |
self versionList contents:revisions. |
|
17827 | 2746 |
self versionEntriesList contents:revisionEntries. |
2747 |
||
2748 |
"Modified: / 29-11-2017 / 13:06:21 / cg" |
|
2695 | 2749 |
! ! |
2750 |
||
2221 | 2751 |
!VersionDiffBrowser methodsFor:'initialization & release'! |
2752 |
||
2753 |
postBuildWith:aBuilder |
|
3147 | 2754 |
" |
2755 |
components which are invisible should be ignored by the panel. |
|
2756 |
Cannot be set via the interface builder. |
|
2221 | 2757 |
|
3147 | 2758 |
<return: self> |
2759 |
" |
|
2760 |
||
3843
673fbeb191c9
builder componentAt: -> self componentAt:
Claus Gittinger <cg@exept.de>
parents:
3839
diff
changeset
|
2761 |
(self componentAt:#TopHorizontalPanel) ignoreInvisibleComponents:true. |
6642 | 2762 |
super postBuildWith:aBuilder |
14076 | 2763 |
! |
2764 |
||
2765 |
postOpenWith:aBuilder |
|
2766 |
super postOpenWith:aBuilder. |
|
2767 |
||
2768 |
self isMultipleVersionBrowser ifTrue:[ |
|
2769 |
self withWaitCursorDo:[ |
|
2770 |
self updateVersionList. |
|
2771 |
] |
|
2772 |
] |
|
2221 | 2773 |
! ! |
2774 |
||
2775 |
!VersionDiffBrowser methodsFor:'menu action'! |
|
2776 |
||
2465 | 2777 |
browseChange:aChange |
6828 | 2778 |
"browse the change in aChange" |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
2779 |
|
17663 | 2780 |
|cls sel| |
2650
447c99ac6f76
oops - inspect & browse menu was broken.
Claus Gittinger <cg@exept.de>
parents:
2626
diff
changeset
|
2781 |
|
2578
29264fb3847e
avoid debugger in browse-menu
Claus Gittinger <cg@exept.de>
parents:
2536
diff
changeset
|
2782 |
aChange isNil ifTrue:[^ self]. |
7396 | 2783 |
|
2650
447c99ac6f76
oops - inspect & browse menu was broken.
Claus Gittinger <cg@exept.de>
parents:
2626
diff
changeset
|
2784 |
cls := aChange changeClass. |
11962 | 2785 |
cls isNil ifTrue:[ |
2786 |
Dialog information:'The class is not present in the image'. |
|
2787 |
^ self |
|
2788 |
]. |
|
7396 | 2789 |
aChange isMethodChange ifTrue:[ |
11962 | 2790 |
sel := aChange changeSelector. |
7396 | 2791 |
"/ (cls compiledMethodAt:sel) isNil ifTrue:[ |
2792 |
"/ (self confirm:'Method does not exist.\\Browse Class ?' withCRs) ifTrue:[ |
|
2793 |
"/ browserClass openInClass:cls |
|
2794 |
"/ ]. |
|
2795 |
"/ sel := nil. |
|
2796 |
"/ ] |
|
2797 |
]. |
|
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
2798 |
self withWaitCursorDo:[ |
17663 | 2799 |
SystemBrowser default openInClass:cls selector:sel. |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
2800 |
] |
6828 | 2801 |
|
17663 | 2802 |
"Modified (format): / 01-09-2017 / 14:23:56 / cg" |
2465 | 2803 |
! |
2804 |
||
2805 |
browseClassInA |
|
6828 | 2806 |
"browse the selected method." |
2807 |
self browseChange:(self selectedChangeInA). |
|
2507 | 2808 |
|
6828 | 2809 |
"Modified: / 25-07-2006 / 11:14:02 / cg" |
2465 | 2810 |
! |
2221 | 2811 |
|
2465 | 2812 |
browseClassInB |
6828 | 2813 |
"browse the selected method." |
2814 |
self browseChange:(self selectedChangeInB). |
|
2507 | 2815 |
|
6828 | 2816 |
"Modified: / 25-07-2006 / 11:13:58 / cg" |
2465 | 2817 |
! |
2818 |
||
2819 |
browseClassInM |
|
6828 | 2820 |
"browse the selected method." |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
2821 |
|
6828 | 2822 |
self browseChange:(self selectedChangesInM first). |
2507 | 2823 |
|
6828 | 2824 |
"Modified: / 25-07-2006 / 11:13:54 / cg" |
2465 | 2825 |
! |
2826 |
||
2469 | 2827 |
createPatchFile |
15621 | 2828 |
"create a patchFile, to patch the old version (versionA) into the new version (versionB). |
2856 | 2829 |
I.e. a little changeFile to transport those changes." |
2830 |
||
15621 | 2831 |
|defaultName diffSet f oldVersion newVersion vsnMthdA vsnMthdB vsnA vsnB info classBeingCompared| |
2832 |
||
2833 |
self warn:'Sorry: This function is not yet implemented'. |
|
2856 | 2834 |
defaultName := 'patchFile.chg'. |
2469 | 2835 |
|
6828 | 2836 |
classBeingCompared := classChangeSet classBeingCompared. |
15621 | 2837 |
defaultName := '%1-%2-to-%3.chg' |
2838 |
bindWith:(classBeingCompared isNil ifTrue:['unknown'] ifFalse:[classBeingCompared nameWithoutPrefix]) |
|
2839 |
with:(classChangeSet versionA ? 'old') |
|
2840 |
with:(classChangeSet versionB ? 'new'). |
|
11453 | 2841 |
f := Dialog |
15621 | 2842 |
requestFileNameForSave:'Name of patchFile:' |
2843 |
default:defaultName. |
|
2844 |
self halt. |
|
2845 |
diffSet := classChangeSet diffSet. |
|
2730 | 2846 |
^ self. |
2847 |
||
11453 | 2848 |
"/ vsnMthdA := self changeSetA |
3103 | 2849 |
"/ detect:[:ch | ch isMethodChange |
2850 |
"/ and:[ch changeClass isMeta |
|
2851 |
"/ and:[ch changeSelector = #version]]] |
|
2852 |
"/ ifNone:nil. |
|
11453 | 2853 |
"/ vsnMthdB := self changeSetA |
3103 | 2854 |
"/ detect:[:ch | ch isMethodChange |
2855 |
"/ and:[ch changeClass isMeta |
|
2856 |
"/ and:[ch changeSelector = #version]]] |
|
2857 |
"/ ifNone:nil. |
|
2858 |
"/ |
|
2859 |
"/ vsnMthdA notNil ifTrue:[ |
|
11453 | 2860 |
"/ "/ extract the version |
3103 | 2861 |
"/ vsnA := Class revisionStringFromSource:vsnMthdA source. |
2862 |
"/ vsnA notNil ifTrue:[ |
|
2863 |
"/ info := Class revisionInfoFromString:vsnA. |
|
2864 |
"/ info notNil ifTrue:[ |
|
2865 |
"/ vsnA := info at:#revision ifAbsent:nil. |
|
2866 |
"/ ] ifFalse:[ |
|
2867 |
"/ vsnA := nil. |
|
2868 |
"/ ]. |
|
2869 |
"/ ]. |
|
2870 |
"/ ]. |
|
2871 |
"/ vsnMthdB notNil ifTrue:[ |
|
11453 | 2872 |
"/ "/ extract the version |
3103 | 2873 |
"/ vsnB := Class revisionStringFromSource:vsnMthdB source. |
2874 |
"/ vsnB notNil ifTrue:[ |
|
2875 |
"/ info := Class revisionInfoFromString:vsnB. |
|
2876 |
"/ info notNil ifTrue:[ |
|
2877 |
"/ vsnB := info at:#revision ifAbsent:nil. |
|
2878 |
"/ ] ifFalse:[ |
|
2879 |
"/ vsnB := nil. |
|
2880 |
"/ ]. |
|
2881 |
"/ ]. |
|
2882 |
"/ ]. |
|
2883 |
"/ |
|
2884 |
"/ (vsnA isNil or:[vsnB isNil or:[vsnA = vsnB]]) ifTrue:[ |
|
2885 |
"/ self warn:'The generated patch file will not be able to validate/update the class version'. |
|
2886 |
"/ ]. |
|
2887 |
"/ |
|
2888 |
"/self halt:'not yet finished'. |
|
6828 | 2889 |
|
9602 | 2890 |
"Modified: / 27-10-2010 / 11:34:58 / cg" |
2469 | 2891 |
! |
2892 |
||
17344 | 2893 |
hideDiffsWithCommentOrFormattingChangeOnlyHolder:newValue |
2894 |
"state of hide changes which have no semantic effect has changed" |
|
2895 |
||
2896 |
self updateLists. |
|
2897 |
||
2898 |
"Created: / 04-02-2017 / 19:50:51 / cg" |
|
2899 |
"Modified: / 04-02-2017 / 23:14:45 / cg" |
|
2900 |
! |
|
2901 |
||
2902 |
inspectSelectedChangeInA |
|
2903 |
"inspect the selected method change for the 'method only in version A'." |
|
2904 |
||
2905 |
self selectedChangeInA inspect. |
|
2906 |
||
2907 |
"Modified: / 25-07-2006 / 11:13:30 / cg" |
|
2908 |
! |
|
2909 |
||
2910 |
inspectSelectedChangeInB |
|
2911 |
"inspect the selected method change for the 'method only in version B'." |
|
2912 |
||
2913 |
self selectedChangeInB inspect. |
|
2914 |
||
2915 |
"Modified: / 25-07-2006 / 11:13:22 / cg" |
|
2916 |
! |
|
2917 |
||
2918 |
inspectSelectedChangeInM |
|
2919 |
"inspect the two method changes for the selected 'changed method'." |
|
2920 |
||
2921 |
|changes| |
|
2922 |
||
2923 |
changes := self selectedChangesInM. |
|
2924 |
changes do:[:change | |
|
2925 |
change inspect |
|
2926 |
]. |
|
2927 |
||
2928 |
"Modified: / 25-07-2006 / 11:13:13 / cg" |
|
2929 |
! |
|
2930 |
||
2931 |
mainMenu |
|
2932 |
"if this application runs as an subapplication, |
|
2933 |
the menu bar should not be used." |
|
2934 |
||
2935 |
self masterApplication isNil ifTrue:[ |
|
2936 |
^ self class mainMenu |
|
2937 |
]. |
|
2938 |
^ nil |
|
2939 |
||
2940 |
"Modified: / 25-07-2006 / 11:13:44 / cg" |
|
2941 |
! |
|
2942 |
||
2943 |
menuAHolder |
|
2944 |
^ [ self class menuA ] |
|
2945 |
||
2946 |
"Created: / 05-02-2017 / 10:07:15 / cg" |
|
2947 |
! |
|
2948 |
||
2949 |
menuBHolder |
|
2950 |
^ [ self class menuB ] |
|
2951 |
||
2952 |
"Created: / 05-02-2017 / 10:07:22 / cg" |
|
2953 |
! |
|
2954 |
||
2955 |
menuMHolder |
|
2956 |
^ [ self class menuM ] |
|
2957 |
||
2958 |
"Created: / 05-02-2017 / 10:07:36 / cg" |
|
2959 |
! |
|
2960 |
||
2961 |
openDocumentation |
|
2962 |
HTMLDocumentView openFullOnDocumentationFile:'tools/misc/TOP.html#VERSIONDIFF' |
|
2963 |
! |
|
2964 |
||
2965 |
removeClassFromChangeSet |
|
2966 |
|cls ok changedClasses| |
|
2967 |
||
2968 |
classChangeSet notNil ifTrue:[ |
|
2969 |
"/ only do if we compare a single class |
|
2970 |
(cls := classChangeSet classBeingCompared) notNil ifTrue:[ |
|
2971 |
ok := classChangeSet diffSet isEmptyOrNil. |
|
2972 |
ok ifFalse:[ |
|
2973 |
changedClasses := classChangeSet diffSet changedClasses. |
|
2974 |
(changedClasses size == 1 |
|
2975 |
and:[ changedClasses anElement theNonMetaclass == cls theNonMetaclass]) ifTrue:[ |
|
2976 |
(methodsChangedFiltered isEmptyOrNil |
|
2977 |
and:[ methodsOnlyInAFiltered isEmptyOrNil |
|
2978 |
and:[ methodsOnlyInBFiltered isEmptyOrNil ]]) ifTrue:[ |
|
2979 |
ok := true. |
|
2980 |
] |
|
2981 |
] |
|
2982 |
]. |
|
2983 |
ok ifTrue:[ |
|
2984 |
ChangeSet current condenseChangesForClass:cls |
|
2985 |
]. |
|
2986 |
]. |
|
2987 |
]. |
|
17954 | 2988 |
! |
2989 |
||
2990 |
versionMenuCompareAgainstNextVersion |
|
2991 |
|cls selectedVersion nextVersion| |
|
2992 |
||
2993 |
selectedVersion := self selectedVersionHolder value. |
|
18147 | 2994 |
nextVersion := self versionList at:(self selectedVersionIndex - 1). |
17954 | 2995 |
cls := classHolder value. |
2996 |
||
2997 |
self class openOnClass:cls versionA:selectedVersion versionB:nextVersion |
|
18147 | 2998 |
|
2999 |
"Modified: / 23-05-2018 / 11:37:18 / Claus Gittinger" |
|
17954 | 3000 |
! |
3001 |
||
3002 |
versionMenuCompareAgainstPreviousVersion |
|
3003 |
|cls selectedVersion prevVersion| |
|
3004 |
||
3005 |
selectedVersion := self selectedVersionHolder value. |
|
18147 | 3006 |
selectedVersion isNil ifTrue:[ |
3007 |
self selectedVersionIndexHolder value:1 |
|
3008 |
]. |
|
3009 |
prevVersion := self versionList at:(self selectedVersionIndex + 1). |
|
17954 | 3010 |
cls := classHolder value. |
3011 |
||
3012 |
self class openOnClass:cls versionA:prevVersion versionB:selectedVersion |
|
18147 | 3013 |
|
3014 |
"Modified: / 23-05-2018 / 11:38:13 / Claus Gittinger" |
|
17344 | 3015 |
! ! |
3016 |
||
3017 |
!VersionDiffBrowser methodsFor:'menu action - filters'! |
|
3018 |
||
17349 | 3019 |
currentFilterParameters |
3020 |
"get the current settings" |
|
3021 |
||
3022 |
|filterParameters| |
|
3023 |
||
3024 |
"/ migration - will be removed |
|
3025 |
filteredMethods notEmptyOrNil ifTrue:[ |
|
3026 |
filteredMethods := filteredMethods |
|
3027 |
collect:[:filter | |
|
3028 |
filter isAssociation ifTrue:[ |
|
3029 |
filter. |
|
3030 |
] ifFalse:[ |
|
3031 |
filter className ->filter changeSelector. |
|
3032 |
]. |
|
3033 |
]. |
|
3034 |
]. |
|
3035 |
||
3036 |
filterParameters := FilterParameters new. |
|
3037 |
filterParameters filteredClassNameMatchPattern:classPatternFilters. |
|
3038 |
filterParameters filteredSelectorMatchPattern:selectorPatternFilters. |
|
3039 |
filterParameters filteredClassNames:filteredClasses. |
|
3040 |
filterParameters filteredMethodNames:filteredMethods. |
|
3041 |
filterParameters filteredSelectors:filteredSelectors. |
|
3042 |
||
3043 |
^ filterParameters. |
|
3044 |
||
3045 |
"Created: / 06-02-2017 / 09:20:07 / cg" |
|
3046 |
! |
|
3047 |
||
3048 |
exportFilterParameters |
|
3049 |
"generate XML with the current settings |
|
3050 |
(to be pasted into Jabber for interchange)" |
|
3051 |
||
3052 |
|filterParameters| |
|
3053 |
||
3054 |
filterParameters := self currentFilterParameters. |
|
3055 |
Workspace |
|
3056 |
openWith:(XMLStandardCoder encode:filterParameters) |
|
3057 |
title:'Current Filter' |
|
3058 |
||
3059 |
"Created: / 06-02-2017 / 09:19:52 / cg" |
|
3060 |
! |
|
3061 |
||
17341 | 3062 |
filterClassInA |
17348 | 3063 |
"filter all changes for the selected change's class (don't show it)." |
17341 | 3064 |
|
3065 |
filteredClasses isNil ifTrue:[ |
|
3066 |
filteredClasses := Set new. |
|
3067 |
]. |
|
3068 |
filteredClasses add:(self selectedChangeInA nonMetaClassName). |
|
3069 |
self filterChanged. |
|
3070 |
||
3071 |
"Created: / 05-02-2017 / 10:17:22 / cg" |
|
3072 |
"Modified: / 05-02-2017 / 11:19:05 / cg" |
|
17348 | 3073 |
"Modified (comment): / 06-02-2017 / 09:13:23 / cg" |
17341 | 3074 |
! |
3075 |
||
3076 |
filterClassInB |
|
17348 | 3077 |
"filter all changes for the selected change's class (don't show it)." |
17341 | 3078 |
|
3079 |
filteredClasses isNil ifTrue:[ |
|
3080 |
filteredClasses := Set new. |
|
3081 |
]. |
|
3082 |
filteredClasses add:(self selectedChangeInB nonMetaClassName). |
|
3083 |
self filterChanged. |
|
3084 |
||
3085 |
"Created: / 05-02-2017 / 10:16:45 / cg" |
|
3086 |
"Modified: / 05-02-2017 / 11:19:02 / cg" |
|
17348 | 3087 |
"Modified (comment): / 06-02-2017 / 09:13:29 / cg" |
17341 | 3088 |
! |
3089 |
||
3090 |
filterClassInM |
|
17348 | 3091 |
"filter all changes for the selected change's class (don't show it)." |
17341 | 3092 |
|
3093 |
filteredClasses isNil ifTrue:[ |
|
3094 |
filteredClasses := Set new. |
|
3095 |
]. |
|
3096 |
filteredClasses add:(self selectedChangesInM first nonMetaClassName). |
|
3097 |
self filterChanged. |
|
3098 |
||
3099 |
"Created: / 05-02-2017 / 10:38:06 / cg" |
|
17348 | 3100 |
"Modified (comment): / 06-02-2017 / 09:13:33 / cg" |
17341 | 3101 |
! |
3102 |
||
3103 |
filterClassesMatching |
|
17348 | 3104 |
"ask for a GLOB pattern; |
3105 |
filter all changes with matching classnames ." |
|
3106 |
||
3107 |
|pattern| |
|
3108 |
||
3109 |
pattern := Dialog request:'Matchpattern for Classname Filter (will hide changes with matching classes):'. |
|
17344 | 3110 |
pattern isEmptyOrNil ifTrue:[^ self]. |
17347 | 3111 |
|
3112 |
classPatternFilters isNil ifTrue:[ |
|
3113 |
classPatternFilters := OrderedCollection new. |
|
17344 | 3114 |
]. |
17347 | 3115 |
classPatternFilters add:pattern. |
17344 | 3116 |
self filterChanged |
17341 | 3117 |
|
3118 |
"Created: / 05-02-2017 / 11:11:42 / cg" |
|
17348 | 3119 |
"Modified: / 06-02-2017 / 09:18:15 / cg" |
3120 |
! |
|
3121 |
||
3122 |
filterMethod:aChange |
|
3123 |
"filter this change (don't show it)." |
|
3124 |
||
3125 |
filteredMethods isNil ifTrue:[ |
|
3126 |
filteredMethods := Set new. |
|
3127 |
]. |
|
3128 |
aChange isMethodChange ifTrue:[ |
|
3129 |
filteredMethods add:(aChange className -> aChange changeSelector). |
|
3130 |
] ifFalse:[ |
|
3131 |
]. |
|
3132 |
||
3133 |
self filterChanged. |
|
3134 |
||
3135 |
"Created: / 06-02-2017 / 08:58:17 / cg" |
|
17341 | 3136 |
! |
3137 |
||
3138 |
filterMethodInA |
|
3139 |
"filter this entry (don't show it)." |
|
3140 |
||
17348 | 3141 |
self filterMethod:(self selectedChangeInA) |
17341 | 3142 |
|
3143 |
"Created: / 05-02-2017 / 10:11:48 / cg" |
|
17348 | 3144 |
"Modified: / 06-02-2017 / 08:58:38 / cg" |
17341 | 3145 |
! |
3146 |
||
3147 |
filterMethodInB |
|
3148 |
"filter this entry (don't show it)." |
|
3149 |
||
17348 | 3150 |
self filterMethod:(self selectedChangeInB) |
17341 | 3151 |
|
3152 |
"Created: / 05-02-2017 / 10:15:57 / cg" |
|
17348 | 3153 |
"Modified: / 06-02-2017 / 08:58:46 / cg" |
17341 | 3154 |
! |
3155 |
||
3156 |
filterMethodInM |
|
3157 |
"filter this entry (don't show it)." |
|
3158 |
||
17348 | 3159 |
self filterMethod:(self selectedChangesInM first) |
17341 | 3160 |
|
3161 |
"Created: / 05-02-2017 / 10:39:10 / cg" |
|
17348 | 3162 |
"Modified: / 06-02-2017 / 08:58:59 / cg" |
17341 | 3163 |
! |
3164 |
||
3165 |
filterSelectorFromA |
|
17348 | 3166 |
"filter all changes for the selected change's selector(don't show it)." |
17341 | 3167 |
|
18094 | 3168 |
self filterSelectorFromSet:{self selectedChangeInA} |
17341 | 3169 |
|
3170 |
"Created: / 05-02-2017 / 10:36:08 / cg" |
|
17348 | 3171 |
"Modified (comment): / 06-02-2017 / 09:13:48 / cg" |
17341 | 3172 |
! |
3173 |
||
3174 |
filterSelectorFromB |
|
17348 | 3175 |
"filter all changes for the selected change's selector(don't show it)." |
17341 | 3176 |
|
18094 | 3177 |
self filterSelectorFromSet:{self selectedChangeInB} |
17341 | 3178 |
|
3179 |
"Created: / 05-02-2017 / 10:36:14 / cg" |
|
17348 | 3180 |
"Modified (comment): / 06-02-2017 / 09:13:52 / cg" |
17341 | 3181 |
! |
3182 |
||
17931 | 3183 |
filterSelectorFromChange:aChange |
3184 |
"common to set a filter; |
|
3185 |
gets the change as argument, |
|
3186 |
takes its selector and adds it to the filters" |
|
17341 | 3187 |
|
3188 |
filteredSelectors isNil ifTrue:[ |
|
3189 |
filteredSelectors := Set new. |
|
3190 |
]. |
|
17931 | 3191 |
filteredSelectors add:(aChange changeSelector). |
3192 |
self filterChanged. |
|
3193 |
||
3194 |
"Created: / 05-02-2017 / 10:39:28 / cg" |
|
3195 |
"Modified (comment): / 06-02-2017 / 09:13:55 / cg" |
|
3196 |
! |
|
3197 |
||
3198 |
filterSelectorFromM |
|
3199 |
"filter all changes for the selected change's selector(don't show it)." |
|
3200 |
||
18094 | 3201 |
self filterSelectorFromSet:{self selectedChangesInM} |
17931 | 3202 |
|
3203 |
"Created: / 05-02-2017 / 10:39:28 / cg" |
|
3204 |
"Modified (comment): / 06-02-2017 / 09:13:55 / cg" |
|
3205 |
! |
|
3206 |
||
3207 |
filterSelectorFromSet:setOfChanges |
|
3208 |
"common to set a filter; |
|
3209 |
get the sub-set (A,B,M as argument), |
|
3210 |
takes its first selection and adds it to the filters" |
|
3211 |
||
3212 |
filteredSelectors isNil ifTrue:[ |
|
3213 |
filteredSelectors := Set new. |
|
3214 |
]. |
|
3215 |
filteredSelectors add:(setOfChanges first changeSelector). |
|
17341 | 3216 |
self filterChanged. |
3217 |
||
3218 |
"Created: / 05-02-2017 / 10:39:28 / cg" |
|
17348 | 3219 |
"Modified (comment): / 06-02-2017 / 09:13:55 / cg" |
17341 | 3220 |
! |
3221 |
||
17347 | 3222 |
filterSelectorsMatching |
17348 | 3223 |
"ask for a GLOB pattern; |
3224 |
filter all changes with matching selectors." |
|
3225 |
||
17347 | 3226 |
|pattern| |
3227 |
||
17348 | 3228 |
pattern := Dialog request:'Matchpattern for Selector Filter (will hide changes with matching selectors):'. |
17347 | 3229 |
pattern isEmptyOrNil ifTrue:[^ self]. |
3230 |
||
3231 |
selectorPatternFilters isNil ifTrue:[ |
|
3232 |
selectorPatternFilters := OrderedCollection new. |
|
3233 |
]. |
|
3234 |
selectorPatternFilters add:pattern. |
|
3235 |
self filterChanged |
|
3236 |
||
3237 |
"Created: / 06-02-2017 / 02:29:07 / cg" |
|
17348 | 3238 |
"Modified: / 06-02-2017 / 09:18:20 / cg" |
17347 | 3239 |
! |
3240 |
||
17341 | 3241 |
forgetFilterNamed |
17344 | 3242 |
"forget a remembered named filter" |
3243 |
||
3244 |
|filterName| |
|
3245 |
||
17952 | 3246 |
RememberedFilters isNil ifTrue:[ |
3247 |
Dialog warn:'No filter was defined'. |
|
3248 |
^ self |
|
3249 |
]. |
|
3250 |
||
17345 | 3251 |
filterName := Dialog |
3252 |
request:'Forget Filter Named:' |
|
3253 |
list:(RememberedFilters keys asNewOrderedCollection sort). |
|
17344 | 3254 |
filterName isEmptyOrNil ifTrue:[^ self]. |
3255 |
||
3256 |
(Dialog confirm:'Shure to Forget the Filter Named: ',filterName) ifFalse:[^ self]. |
|
3257 |
||
3258 |
RememberedFilters removeKey:filterName. |
|
17341 | 3259 |
|
3260 |
"Created: / 05-02-2017 / 11:12:08 / cg" |
|
17345 | 3261 |
"Modified: / 06-02-2017 / 02:17:47 / cg" |
17334 | 3262 |
! |
3263 |
||
17349 | 3264 |
importFilterParameters |
3265 |
"ask for XML representation as exported previously. |
|
3266 |
(to be pasted from Jabber for interchange)" |
|
3267 |
||
3268 |
|xml filterParameters| |
|
3269 |
||
3270 |
xml := Dialog requestText:'Paste XML Exported Filter Here and Accept'. |
|
3271 |
xml isEmptyOrNil ifTrue:[^ self]. |
|
3272 |
||
3273 |
filterParameters := XMLStandardCoder decode:xml. |
|
3274 |
(filterParameters isKindOf:FilterParameters) ifTrue:[ |
|
3275 |
self setFilterParameters:filterParameters |
|
3276 |
]. |
|
3277 |
||
3278 |
"Created: / 06-02-2017 / 09:51:11 / cg" |
|
3279 |
! |
|
3280 |
||
17344 | 3281 |
inspectFilterParameters |
3282 |
"inspect the current settings" |
|
3283 |
||
17349 | 3284 |
self currentFilterParameters inspect. |
17344 | 3285 |
|
3286 |
"Created: / 06-02-2017 / 02:00:55 / cg" |
|
17349 | 3287 |
"Modified: / 06-02-2017 / 09:20:15 / cg" |
2465 | 3288 |
! |
3289 |
||
17344 | 3290 |
loadFilterNamed |
3291 |
"load a remembered named filter" |
|
3292 |
||
3293 |
|filterName filterParameters| |
|
3294 |
||
17345 | 3295 |
filterName := Dialog |
3296 |
request:'Load Filter Named:' |
|
3297 |
list:(RememberedFilters keys asNewOrderedCollection sort). |
|
17344 | 3298 |
filterName isEmptyOrNil ifTrue:[^ self]. |
3299 |
||
3300 |
filterParameters := RememberedFilters at:filterName. |
|
17349 | 3301 |
self setFilterParameters:filterParameters. |
17344 | 3302 |
|
3303 |
"Created: / 06-02-2017 / 01:57:48 / cg" |
|
17349 | 3304 |
"Modified: / 06-02-2017 / 09:51:47 / cg" |
11855
52fd36ba842f
changed: #removeClassFromChangeSet
Claus Gittinger <cg@exept.de>
parents:
11853
diff
changeset
|
3305 |
! |
52fd36ba842f
changed: #removeClassFromChangeSet
Claus Gittinger <cg@exept.de>
parents:
11853
diff
changeset
|
3306 |
|
17341 | 3307 |
rememberFilterAs |
17344 | 3308 |
"remember the current settings under a named filter" |
3309 |
||
3310 |
|filterName filterParameters| |
|
3311 |
||
17346 | 3312 |
filterName := Dialog |
3313 |
request:'Name of Filter:' |
|
3314 |
list:(RememberedFilters keys asNewOrderedCollection sort). |
|
17344 | 3315 |
filterName isEmptyOrNil ifTrue:[^ self]. |
3316 |
||
17349 | 3317 |
filterParameters := self currentFilterParameters. |
17344 | 3318 |
|
3319 |
RememberedFilters isNil ifTrue:[ |
|
3320 |
RememberedFilters := Dictionary new. |
|
11855
52fd36ba842f
changed: #removeClassFromChangeSet
Claus Gittinger <cg@exept.de>
parents:
11853
diff
changeset
|
3321 |
]. |
17344 | 3322 |
RememberedFilters at:filterName put:filterParameters. |
3323 |
||
17349 | 3324 |
"Modified: / 06-02-2017 / 09:20:35 / cg" |
3325 |
! |
|
3326 |
||
17952 | 3327 |
removeClassFilter |
3328 |
"clear the current class filter (but not named ones)" |
|
3329 |
||
3330 |
classPatternFilters := nil. |
|
3331 |
self filterChanged |
|
3332 |
! |
|
3333 |
||
3334 |
removeSelectorFilter |
|
3335 |
"clear the current selector filter (but not named ones)" |
|
3336 |
||
3337 |
selectorPatternFilters := nil. |
|
3338 |
self filterChanged |
|
3339 |
||
3340 |
"Created: / 06-02-2017 / 02:29:07 / cg" |
|
3341 |
"Modified: / 06-02-2017 / 09:18:20 / cg" |
|
3342 |
! |
|
3343 |
||
17349 | 3344 |
setFilterParameters:filterParameters |
3345 |
"setup from a remembered filter" |
|
3346 |
||
3347 |
selectorPatternFilters := filterParameters filteredSelectorMatchPattern. |
|
3348 |
classPatternFilters := filterParameters filteredClassNameMatchPattern. |
|
3349 |
filteredClasses := filterParameters filteredClassNames. |
|
3350 |
filteredMethods := filterParameters filteredMethodNames. |
|
3351 |
filteredSelectors := filterParameters filteredSelectors. |
|
3352 |
||
3353 |
self filterChanged. |
|
3354 |
||
3355 |
"Created: / 06-02-2017 / 09:51:37 / cg" |
|
2221 | 3356 |
! ! |
3357 |
||
17931 | 3358 |
!VersionDiffBrowser methodsFor:'menu action - searching'! |
3359 |
||
3360 |
findNextChangeWithStringIn:listOfchanges selectionHolder:whichMethodSelectionHolder |
|
3361 |
"common code to select the next change which contains some string" |
|
3362 |
||
3363 |
|string startIdx| |
|
3364 |
||
3365 |
string := Dialog request:'Search for:' initialAnswer:LastSearchString. |
|
3366 |
string isEmptyOrNil ifTrue:[^ false]. |
|
3367 |
||
3368 |
LastSearchString := string. |
|
3369 |
||
3370 |
startIdx := whichMethodSelectionHolder value. |
|
3371 |
listOfchanges from:startIdx+1 keysAndValuesDo:[:index :eachChange | |
|
3372 |
(eachChange source includesString:string caseSensitive:false) ifTrue:[ |
|
3373 |
whichMethodSelectionHolder value:index. |
|
3374 |
^ true. |
|
3375 |
] |
|
3376 |
]. |
|
3377 |
^ false |
|
3378 |
! |
|
3379 |
||
3380 |
findNextChangeWithStringInA |
|
3381 |
"select the next change which contains some string" |
|
3382 |
||
3383 |
(self findNextChangeWithStringIn:methodsOnlyInAFiltered selectionHolder:methodsOnlyInASelection) ifTrue:[ |
|
3384 |
self methodsOnlyInASelectionChanged |
|
3385 |
] ifFalse:[ |
|
3386 |
self beepInEditor |
|
3387 |
]. |
|
3388 |
! |
|
3389 |
||
3390 |
findNextChangeWithStringInB |
|
3391 |
"select the next change which contains some string" |
|
3392 |
||
3393 |
(self findNextChangeWithStringIn:methodsOnlyInBFiltered selectionHolder:methodsOnlyInBSelection) ifTrue:[ |
|
3394 |
self methodsOnlyInASelectionChanged |
|
3395 |
] |
|
3396 |
! |
|
3397 |
||
3398 |
findNextChangeWithStringInM |
|
3399 |
"select the next change which contains some string" |
|
3400 |
||
3401 |
(self findNextChangeWithStringInPairs:methodsChangedFiltered selectionHolder:methodsChangedSelection) ifTrue:[ |
|
3402 |
self methodsChangedSelectionChanged |
|
3403 |
] |
|
3404 |
! |
|
3405 |
||
3406 |
findNextChangeWithStringInPairs:listOfchangePairs selectionHolder:whichMethodSelectionHolder |
|
3407 |
"select the next change which contains some string" |
|
3408 |
||
3409 |
|string startIdx| |
|
3410 |
||
3411 |
string := Dialog request:'Search for:' initialAnswer:LastSearchString. |
|
3412 |
string isEmptyOrNil ifTrue:[^ false]. |
|
3413 |
||
3414 |
LastSearchString := string. |
|
3415 |
||
3416 |
startIdx := whichMethodSelectionHolder value. |
|
3417 |
listOfchangePairs from:startIdx+1 keysAndValuesDo:[:index :eachChangePair | |
|
3418 |
(eachChangePair first source includesString:string caseSensitive:false) ifTrue:[ |
|
3419 |
whichMethodSelectionHolder value:index. |
|
3420 |
^ true. |
|
3421 |
]. |
|
3422 |
(eachChangePair second source includesString:string caseSensitive:false) ifTrue:[ |
|
3423 |
whichMethodSelectionHolder value:index. |
|
3424 |
^ true. |
|
3425 |
] |
|
3426 |
]. |
|
3427 |
^ false. |
|
3428 |
! ! |
|
3429 |
||
2221 | 3430 |
!VersionDiffBrowser methodsFor:'private'! |
3431 |
||
2745 | 3432 |
addAcceptToTextViewMenus |
8657
4a54180313b5
enable/disable of edit-menu items
Claus Gittinger <cg@exept.de>
parents:
8629
diff
changeset
|
3433 |
"add to the standard diff text view or single text views menu an accept entry. |
4a54180313b5
enable/disable of edit-menu items
Claus Gittinger <cg@exept.de>
parents:
8629
diff
changeset
|
3434 |
The acceptAction will fetch the corresponding change and apply it |
4a54180313b5
enable/disable of edit-menu items
Claus Gittinger <cg@exept.de>
parents:
8629
diff
changeset
|
3435 |
(not the shown text) |
2745 | 3436 |
|
8657
4a54180313b5
enable/disable of edit-menu items
Claus Gittinger <cg@exept.de>
parents:
8629
diff
changeset
|
3437 |
<return: self> |
3147 | 3438 |
" |
2745 | 3439 |
|
8657
4a54180313b5
enable/disable of edit-menu items
Claus Gittinger <cg@exept.de>
parents:
8629
diff
changeset
|
3440 |
|diffTextView leftView rightView singleView| |
2745 | 3441 |
|
3442 |
diffTextView := self diffTextView. |
|
3443 |
leftView := diffTextView leftTextView. |
|
3444 |
rightView := diffTextView rightTextView. |
|
3843
673fbeb191c9
builder componentAt: -> self componentAt:
Claus Gittinger <cg@exept.de>
parents:
3839
diff
changeset
|
3445 |
singleView := self componentAt:#singleTextView. |
2745 | 3446 |
|
11453 | 3447 |
(Array |
13547 | 3448 |
with:leftView |
3449 |
with:rightView |
|
3450 |
with:singleView) |
|
2745 | 3451 |
do:[:v | |
13547 | 3452 |
|mGen| |
3453 |
||
3454 |
mGen := [ |
|
3455 |
|m| |
|
3456 |
||
3457 |
m := v editMenu. |
|
3458 |
||
3459 |
(m selectorAt:#accept) isNil ifTrue:[ |
|
3460 |
m addLabels:(resources array:#('-' 'Accept')) |
|
3461 |
selectors:#(nil #accept) |
|
3462 |
after:#copySelection. |
|
3463 |
]. |
|
3464 |
m |
|
3465 |
actionAt:#accept |
|
3466 |
put:[ |
|
3467 |
v == singleView ifTrue:[ |
|
3468 |
self acceptInSingleView |
|
3469 |
] ifFalse:[ |
|
3470 |
v == leftView ifTrue:[ |
|
3471 |
self acceptInLeftView |
|
3472 |
] ifFalse:[ |
|
3473 |
self acceptInRightView |
|
3474 |
]. |
|
3475 |
]. |
|
3476 |
]. |
|
3477 |
m selectorAt:#accept put:nil. |
|
3478 |
m enable:#copySelection. |
|
3479 |
m setEnable:#accept to:[self canAcceptInCodeView]. |
|
3480 |
m |
|
3481 |
]. |
|
3482 |
v menuHolder:mGen. |
|
3483 |
v menuMessage:#value. |
|
2745 | 3484 |
] |
3485 |
! |
|
3486 |
||
17342 | 3487 |
information:msg |
3488 |
self infoHolder value:msg |
|
3489 |
||
3490 |
"Created: / 05-02-2017 / 11:42:27 / cg" |
|
17341 | 3491 |
! |
3492 |
||
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3493 |
printStringForChange:aChange |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3494 |
"generate a print string for a change." |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3495 |
|
11596
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
3496 |
|m changeClassName classOrMetaString useChangesString selectorString |
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
3497 |
singleComparedClass singleComparedClassesName singleComparedMetaclassesName |
13137 | 3498 |
printString class1 class2| |
3499 |
||
3500 |
class1 := self classChangeSet class1BeingCompared. |
|
3501 |
class2 := self classChangeSet class2BeingCompared. |
|
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3502 |
|
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3503 |
(singleComparedClass := self classBeingCompared) notNil ifTrue:[ |
11533
106faef31ef5
changed: #printStringForChange:
Claus Gittinger <cg@exept.de>
parents:
11453
diff
changeset
|
3504 |
singleComparedClassesName := singleComparedClass name. |
106faef31ef5
changed: #printStringForChange:
Claus Gittinger <cg@exept.de>
parents:
11453
diff
changeset
|
3505 |
singleComparedMetaclassesName := singleComparedClass theMetaclass name. |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3506 |
]. |
2497 | 3507 |
|
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3508 |
aChange isClassDefinitionChange ifTrue:[ |
11533
106faef31ef5
changed: #printStringForChange:
Claus Gittinger <cg@exept.de>
parents:
11453
diff
changeset
|
3509 |
^ aChange printStringWithoutClassName |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3510 |
]. |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3511 |
aChange isDoIt ifTrue:[ |
11533
106faef31ef5
changed: #printStringForChange:
Claus Gittinger <cg@exept.de>
parents:
11453
diff
changeset
|
3512 |
^ aChange source , ' (doIt)' |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3513 |
]. |
10892
98a427ae867b
changed: #printStringForChange:
Claus Gittinger <cg@exept.de>
parents:
10829
diff
changeset
|
3514 |
aChange isNameSpaceCreationChange ifTrue:[ |
11533
106faef31ef5
changed: #printStringForChange:
Claus Gittinger <cg@exept.de>
parents:
11453
diff
changeset
|
3515 |
^ aChange source |
10892
98a427ae867b
changed: #printStringForChange:
Claus Gittinger <cg@exept.de>
parents:
10829
diff
changeset
|
3516 |
]. |
3678 | 3517 |
|
10892
98a427ae867b
changed: #printStringForChange:
Claus Gittinger <cg@exept.de>
parents:
10829
diff
changeset
|
3518 |
changeClassName := aChange className. |
98a427ae867b
changed: #printStringForChange:
Claus Gittinger <cg@exept.de>
parents:
10829
diff
changeset
|
3519 |
changeClassName isNil ifTrue:[ |
11533
106faef31ef5
changed: #printStringForChange:
Claus Gittinger <cg@exept.de>
parents:
11453
diff
changeset
|
3520 |
^ aChange source |
10892
98a427ae867b
changed: #printStringForChange:
Claus Gittinger <cg@exept.de>
parents:
10829
diff
changeset
|
3521 |
]. |
11596
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
3522 |
|
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3523 |
useChangesString := false. |
11596
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
3524 |
classOrMetaString := ''. |
10892
98a427ae867b
changed: #printStringForChange:
Claus Gittinger <cg@exept.de>
parents:
10829
diff
changeset
|
3525 |
|
13137 | 3526 |
(class1 notNil |
3527 |
and:[ class2 notNil |
|
3528 |
and:[ class1 ~= class2 ]]) ifTrue:[ |
|
3529 |
changeClassName := ''. |
|
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3530 |
] ifFalse:[ |
13137 | 3531 |
(changeClassName = singleComparedClassesName) ifTrue:[ |
3532 |
"/ changeClassName := ''. |
|
11533
106faef31ef5
changed: #printStringForChange:
Claus Gittinger <cg@exept.de>
parents:
11453
diff
changeset
|
3533 |
useChangesString := true. |
106faef31ef5
changed: #printStringForChange:
Claus Gittinger <cg@exept.de>
parents:
11453
diff
changeset
|
3534 |
] ifFalse:[ |
13137 | 3535 |
(changeClassName = singleComparedMetaclassesName) ifTrue:[ |
3536 |
"/ classOrMetaString := ' class'. |
|
3537 |
useChangesString := true. |
|
3538 |
] ifFalse:[ |
|
3539 |
((changeClassName includes:$:) and:[ changeClassName startsWith:((singleComparedClassesName ? ''),'::') ]) ifTrue:[ |
|
3540 |
changeClassName := changeClassName copyFrom:(singleComparedClassesName,'::') size+1. |
|
3541 |
]. |
|
11533
106faef31ef5
changed: #printStringForChange:
Claus Gittinger <cg@exept.de>
parents:
11453
diff
changeset
|
3542 |
]. |
106faef31ef5
changed: #printStringForChange:
Claus Gittinger <cg@exept.de>
parents:
11453
diff
changeset
|
3543 |
]. |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3544 |
]. |
6272 | 3545 |
|
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3546 |
selectorString := ''. |
11533
106faef31ef5
changed: #printStringForChange:
Claus Gittinger <cg@exept.de>
parents:
11453
diff
changeset
|
3547 |
(aChange isMethodChange) ifTrue:[ |
17347 | 3548 |
selectorString := aChange changeSelector. |
13137 | 3549 |
|
11533
106faef31ef5
changed: #printStringForChange:
Claus Gittinger <cg@exept.de>
parents:
11453
diff
changeset
|
3550 |
aChange changeClass notNil ifTrue:[ |
106faef31ef5
changed: #printStringForChange:
Claus Gittinger <cg@exept.de>
parents:
11453
diff
changeset
|
3551 |
m := aChange changeMethod. |
106faef31ef5
changed: #printStringForChange:
Claus Gittinger <cg@exept.de>
parents:
11453
diff
changeset
|
3552 |
m notNil ifTrue:[ |
106faef31ef5
changed: #printStringForChange:
Claus Gittinger <cg@exept.de>
parents:
11453
diff
changeset
|
3553 |
m := m originalMethodIfWrapped. |
17347 | 3554 |
selectorString := m printStringForBrowserWithSelector:aChange changeSelector inClass:aChange changeClass. |
11533
106faef31ef5
changed: #printStringForChange:
Claus Gittinger <cg@exept.de>
parents:
11453
diff
changeset
|
3555 |
]. |
106faef31ef5
changed: #printStringForChange:
Claus Gittinger <cg@exept.de>
parents:
11453
diff
changeset
|
3556 |
]. |
11596
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
3557 |
changeClassName notEmptyOrNil ifTrue:[ |
18094 | 3558 |
selectorString := ' » ',selectorString |
11596
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
3559 |
] |
2497 | 3560 |
]. |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3561 |
|
11596
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
3562 |
printString := changeClassName,classOrMetaString,selectorString. |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3563 |
aChange isMethodCategoryChange ifTrue:[ |
11596
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
3564 |
^ printString, ' (category)' |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3565 |
]. |
11596
a82461d12ced
selection fixes (when filtering)
Claus Gittinger <cg@exept.de>
parents:
11533
diff
changeset
|
3566 |
^ printString |
9587
9f33d5c62efa
changed: #printStringForChange:
Claus Gittinger <cg@exept.de>
parents:
9550
diff
changeset
|
3567 |
|
17347 | 3568 |
"Modified: / 06-02-2017 / 02:32:08 / cg" |
2497 | 3569 |
! |
3570 |
||
2501 | 3571 |
resetSelectionHolders |
3147 | 3572 |
" |
3573 |
reset all selection holders when a new change set is given. |
|
11453 | 3574 |
First set the selection to nil. |
2507 | 3575 |
|
3147 | 3576 |
<return: self> |
3577 |
" |
|
2501 | 3578 |
|
3579 |
self methodsChangedSelection value:nil. |
|
3580 |
self methodsOnlyInASelection value:nil. |
|
3581 |
self methodsOnlyInBSelection value:nil. |
|
2507 | 3582 |
self diffTextView text1:'' text2:''. |
3583 |
self methodText value:''. |
|
3584 |
||
3585 |
"/ self methodsChangedSelectionChanged. |
|
3586 |
"/ self methodsOnlyInASelectionChanged. |
|
3587 |
"/ self methodsOnlyInBSelectionChanged. |
|
2501 | 3588 |
! |
3589 |
||
2221 | 3590 |
showDiffTextView |
3147 | 3591 |
" |
3592 |
if a method change is selected, then show the diff text view. |
|
3593 |
Add an accept entry to the popup menu. |
|
2507 | 3594 |
|
3147 | 3595 |
<return: self> |
3596 |
" |
|
2694 | 3597 |
|
3843
673fbeb191c9
builder componentAt: -> self componentAt:
Claus Gittinger <cg@exept.de>
parents:
3839
diff
changeset
|
3598 |
(self componentAt:#diffTextViewBox) raise; beVisible. |
673fbeb191c9
builder componentAt: -> self componentAt:
Claus Gittinger <cg@exept.de>
parents:
3839
diff
changeset
|
3599 |
(self componentAt:#singleTextView) beInvisible. |
673fbeb191c9
builder componentAt: -> self componentAt:
Claus Gittinger <cg@exept.de>
parents:
3839
diff
changeset
|
3600 |
(self componentAt:#diffTextView) realizeAllSubViews. |
2694 | 3601 |
|
2465 | 3602 |
self addAcceptToTextViewMenus. |
2221 | 3603 |
! |
3604 |
||
3605 |
showSingleTextView |
|
3147 | 3606 |
" |
3607 |
if a method is selected which is only in version A or B of the class, |
|
3608 |
then show the text view. |
|
3609 |
Add an accept entry to the popup menu. |
|
2507 | 3610 |
|
3147 | 3611 |
<return: self> |
3612 |
" |
|
2507 | 3613 |
|
3843
673fbeb191c9
builder componentAt: -> self componentAt:
Claus Gittinger <cg@exept.de>
parents:
3839
diff
changeset
|
3614 |
(self componentAt:#singleTextView) raise; beVisible. |
673fbeb191c9
builder componentAt: -> self componentAt:
Claus Gittinger <cg@exept.de>
parents:
3839
diff
changeset
|
3615 |
(self componentAt:#diffTextViewBox) beInvisible. |
2501 | 3616 |
self addAcceptToTextViewMenus. |
2221 | 3617 |
! |
3618 |
||
15621 | 3619 |
showVersionInfoIfNothingSelected |
3620 |
"show the revision info (author, date, time and logMessage), if no method is selected" |
|
3621 |
||
3622 |
|revInfo infoText| |
|
3623 |
||
17342 | 3624 |
"/ self information:''. |
15621 | 3625 |
|
3626 |
(methodsOnlyInASelection value isNil |
|
17334 | 3627 |
and:[ methodsOnlyInBSelection value isNil |
3628 |
and:[ methodsChangedSelection value isNil]] |
|
3629 |
) ifTrue:[ |
|
3630 |
versionInfoList notNil ifTrue:[ |
|
3631 |
revInfo := versionInfoList detect:[:info | (info at:#revision) = selectedVersionHolder value] ifNone:nil. |
|
3632 |
revInfo notNil ifTrue:[ |
|
3633 |
infoText := 'Revision: %1\Author: %2\Date: %3\Log:\\%4' withCRs |
|
3634 |
bindWith:(revInfo at:#revision) |
|
3635 |
with:(revInfo at:#author) |
|
3636 |
with:(revInfo at:#date) |
|
3637 |
with:(revInfo at:#logMessage). |
|
3638 |
]. |
|
3639 |
]. |
|
15621 | 3640 |
self methodText value:infoText. |
3641 |
self showSingleTextView. |
|
3642 |
]. |
|
17334 | 3643 |
|
17342 | 3644 |
"Modified: / 05-02-2017 / 11:49:47 / cg" |
2221 | 3645 |
! ! |
3646 |
||
2501 | 3647 |
!VersionDiffBrowser methodsFor:'setup'! |
3648 |
||
2695 | 3649 |
setupForClass:aClass againstVersion:aVersionA |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3650 |
"compute the class change set for the class aClass of its current version against the repository version A. |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3651 |
When setting the class change set, the labels, list etc. of the receiver |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3652 |
are updated." |
2695 | 3653 |
|
2731 | 3654 |
|changeSet| |
3655 |
||
2745 | 3656 |
aClass isNil ifTrue:[ |
16551 | 3657 |
changeSet := nil |
2745 | 3658 |
] ifFalse:[ |
16551 | 3659 |
(aClass isRealNameSpace) ifFalse:[ |
3660 |
aClass isLoaded ifTrue:[ |
|
3661 |
changeSet := (ClassChangeSet newForClass:aClass againstVersion:aVersionA). |
|
3662 |
] |
|
3663 |
]. |
|
2731 | 3664 |
]. |
2856 | 3665 |
self classChangeSet:changeSet. |
2857
0c467f6b1741
handle case if invoked without versionString
Claus Gittinger <cg@exept.de>
parents:
2856
diff
changeset
|
3666 |
aVersionA isNil ifTrue:[ |
16551 | 3667 |
self class:aClass versionA:'Repository' versionB:'Current'. |
2857
0c467f6b1741
handle case if invoked without versionString
Claus Gittinger <cg@exept.de>
parents:
2856
diff
changeset
|
3668 |
] ifFalse:[ |
16551 | 3669 |
self class:aClass versionA:aVersionA versionB:(aVersionA , 'm'). |
2857
0c467f6b1741
handle case if invoked without versionString
Claus Gittinger <cg@exept.de>
parents:
2856
diff
changeset
|
3670 |
] |
2695 | 3671 |
! |
3672 |
||
2949 | 3673 |
setupForClass:classA labelA:aLabelA andClass:classB labelB:aLabelB |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3674 |
"generate the class change set from the two classes A and B. |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3675 |
When setting the class change set, the labels, list etc. of the receiver |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3676 |
are updated." |
2949 | 3677 |
|
3678 |
|changeSet| |
|
3679 |
||
3680 |
changeSet := (ClassChangeSet newForClass:classA labelA:aLabelA andClass:classB labelB:aLabelB). |
|
3681 |
self classChangeSet:changeSet |
|
3682 |
! |
|
3683 |
||
2510 | 3684 |
setupForClass:aClass labelA:aLabelA sourceA:aSourceA labelB:aLabelB sourceB:aSourceB |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3685 |
"generate the class change set from the two source files A and B. |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3686 |
When setting the class change set, the labels, list etc. of the receiver |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3687 |
are updated." |
2510 | 3688 |
|
2731 | 3689 |
|changeSet| |
3690 |
||
9383 | 3691 |
aClass isRealNameSpace ifFalse:[ |
11453 | 3692 |
aClass isLoaded ifTrue:[ |
3693 |
changeSet := (ClassChangeSet newForClass:aClass labelA:aLabelA sourceA:aSourceA labelB:aLabelB sourceB:aSourceB). |
|
3694 |
] |
|
2731 | 3695 |
]. |
3696 |
self classChangeSet:changeSet |
|
7499 | 3697 |
|
3698 |
"Modified: / 10-11-2006 / 17:15:26 / cg" |
|
2510 | 3699 |
! |
3700 |
||
2501 | 3701 |
setupForClass:aClass versionA:aVersionA versionB:aVersionB |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3702 |
"compute the class change set for the class aClass and the versions A and B. |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3703 |
When setting the class change set, the labels, list etc. of the receiver |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3704 |
are updated." |
2501 | 3705 |
|
2731 | 3706 |
|changeSet| |
3707 |
||
3708 |
aClass isNameSpace ifFalse:[ |
|
11453 | 3709 |
aClass isLoaded ifTrue:[ |
3710 |
changeSet := (ClassChangeSet newForClass:aClass versionA:aVersionA versionB:aVersionB). |
|
3711 |
] |
|
2731 | 3712 |
]. |
2856 | 3713 |
self classChangeSet:changeSet. |
3714 |
self class:aClass versionA:aVersionA versionB:aVersionB. |
|
3678 | 3715 |
! |
3716 |
||
3717 |
setupForDiffSet:diffSet labelA:aLabelA labelB:aLabelB |
|
3718 |
|changeSet| |
|
3719 |
||
3720 |
changeSet := (ClassChangeSet new diffSet:diffSet; labelA:aLabelA; labelB:aLabelB). |
|
3721 |
self classChangeSet:changeSet |
|
2501 | 3722 |
! ! |
3723 |
||
2497 | 3724 |
!VersionDiffBrowser::ClassChangeSet class methodsFor:'instance creation'! |
3725 |
||
11453 | 3726 |
changeSetForClass:aClass |
3756 | 3727 |
"return a ChangeSet for the class aClass and its current source. |
3728 |
The source is generated into a stream and then the change set is generated |
|
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3729 |
from the source stream." |
2695 | 3730 |
|
3756 | 3731 |
^ ChangeSet forExistingClass:aClass. |
2695 | 3732 |
! |
3733 |
||
2497 | 3734 |
changeSetForClass:aClass andRevision:aVersion |
3594
12071dccbbe4
#valueNowOrOnUnwindDo: -> #ensure:
Claus Gittinger <cg@exept.de>
parents:
3344
diff
changeset
|
3735 |
"return a ChangeSet for the class aClass and version aVersion. |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3736 |
The version from the class source stream is checked out from the repositiory |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3737 |
into a source stream. Then the change set is generated from the source stream." |
3594
12071dccbbe4
#valueNowOrOnUnwindDo: -> #ensure:
Claus Gittinger <cg@exept.de>
parents:
3344
diff
changeset
|
3738 |
|
2497 | 3739 |
|theSourceCodeManager theSourceStream theChangeSet| |
3740 |
||
13305 | 3741 |
theSourceCodeManager := SourceCodeManagerUtilities sourceCodeManagerFor:aClass. |
3594
12071dccbbe4
#valueNowOrOnUnwindDo: -> #ensure:
Claus Gittinger <cg@exept.de>
parents:
3344
diff
changeset
|
3742 |
[ |
13305 | 3743 |
theSourceStream := theSourceCodeManager getSourceStreamFor:aClass revision:aVersion. |
15167 | 3744 |
theSourceStream notNil ifTrue:[ |
3745 |
theChangeSet := ChangeSet fromStream:theSourceStream |
|
3746 |
] |
|
3594
12071dccbbe4
#valueNowOrOnUnwindDo: -> #ensure:
Claus Gittinger <cg@exept.de>
parents:
3344
diff
changeset
|
3747 |
] ensure:[ |
13305 | 3748 |
theSourceStream notNil ifTrue:[theSourceStream close] |
3594
12071dccbbe4
#valueNowOrOnUnwindDo: -> #ensure:
Claus Gittinger <cg@exept.de>
parents:
3344
diff
changeset
|
3749 |
]. |
15167 | 3750 |
theChangeSet isNil ifTrue:[ |
3751 |
self error:'Could not read source of class ',aClass name mayProceed:true. |
|
3752 |
^ nil |
|
3753 |
]. |
|
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3754 |
self sortChangeSet:theChangeSet. |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3755 |
^ theChangeSet |
7108 | 3756 |
|
3757 |
"Modified: / 12-09-2006 / 14:24:51 / cg" |
|
2497 | 3758 |
! |
3759 |
||
11453 | 3760 |
changeSetForClass:aClass andSource:aSource |
3594
12071dccbbe4
#valueNowOrOnUnwindDo: -> #ensure:
Claus Gittinger <cg@exept.de>
parents:
3344
diff
changeset
|
3761 |
"return a ChangeSet for the class aClass and source aSource. |
4638
79734f7bdfc4
Handle openErrorSignal in preparition for change openErrorSignal
Stefan Vogel <sv@exept.de>
parents:
4359
diff
changeset
|
3762 |
The source is converted to a stream and then the change set is generated |
79734f7bdfc4
Handle openErrorSignal in preparition for change openErrorSignal
Stefan Vogel <sv@exept.de>
parents:
4359
diff
changeset
|
3763 |
from the source stream. |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3764 |
Raise an error if the source stream cannot be opened." |
4638
79734f7bdfc4
Handle openErrorSignal in preparition for change openErrorSignal
Stefan Vogel <sv@exept.de>
parents:
4359
diff
changeset
|
3765 |
|
2497 | 3766 |
|theChangeSet theSourceStream| |
3767 |
||
3594
12071dccbbe4
#valueNowOrOnUnwindDo: -> #ensure:
Claus Gittinger <cg@exept.de>
parents:
3344
diff
changeset
|
3768 |
[ |
15167 | 3769 |
theSourceStream := aSource readStream. |
17875 | 3770 |
theChangeSet := ChangeSet fromStream:theSourceStream. |
3594
12071dccbbe4
#valueNowOrOnUnwindDo: -> #ensure:
Claus Gittinger <cg@exept.de>
parents:
3344
diff
changeset
|
3771 |
] ensure:[ |
15167 | 3772 |
theSourceStream notNil ifTrue:[ |
3773 |
theSourceStream close |
|
3774 |
] |
|
3775 |
]. |
|
3776 |
theChangeSet isNil ifTrue:[ |
|
3777 |
self error:'Could not read source of class ',aClass name mayProceed:true. |
|
3778 |
^ nil |
|
3594
12071dccbbe4
#valueNowOrOnUnwindDo: -> #ensure:
Claus Gittinger <cg@exept.de>
parents:
3344
diff
changeset
|
3779 |
]. |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3780 |
self sortChangeSet:theChangeSet. |
4653
0a97ce41f5bd
More openErrorSignal change preparations
Stefan Vogel <sv@exept.de>
parents:
4638
diff
changeset
|
3781 |
^ theChangeSet |
17875 | 3782 |
|
3783 |
"Modified: / 02-01-2018 / 20:41:55 / stefan" |
|
2497 | 3784 |
! |
3785 |
||
2695 | 3786 |
newForClass:aClass againstVersion:aVersionA |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3787 |
"return a ClassChangeSet for the class aClass against some verson in the repository. |
11453 | 3788 |
A new instance of ClassChangeSet is generated containing |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3789 |
the correspondenting changes." |
2695 | 3790 |
|
8891 | 3791 |
|theClassChangeSet| |
2695 | 3792 |
|
3793 |
theClassChangeSet := self new. |
|
8891 | 3794 |
theClassChangeSet setupForClass:aClass againstVersion:aVersionA. |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3795 |
^ theClassChangeSet |
2695 | 3796 |
! |
3797 |
||
2949 | 3798 |
newForClass:classA labelA:aLabelA andClass:classB labelB:aLabelB |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3799 |
"return a ClassChangeSet for two classes. |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3800 |
A new instance of ClassChangeSet is generated containing the correspondenting changes." |
2949 | 3801 |
|
3802 |
|aStream sourceA sourceB theChangeSetA theChangeSetB theClassChangeSet| |
|
3803 |
||
3804 |
theClassChangeSet := self new. |
|
13137 | 3805 |
theClassChangeSet class1BeingCompared:classA class2BeingCompared:classB. |
2949 | 3806 |
theClassChangeSet labelA:aLabelA. |
3807 |
theClassChangeSet labelB:aLabelB. |
|
3808 |
||
16615 | 3809 |
theChangeSetA := ChangeSet forExistingClass:classA withExtensions:true withLooseMethods:true. |
3810 |
theChangeSetB := ChangeSet forExistingClass:classB withExtensions:true withLooseMethods:true. |
|
3811 |
||
3812 |
"/ aStream := '' writeStream. |
|
3813 |
"/ Method flushSourceStreamCache. |
|
3814 |
"/ classA fileOutOn:aStream withTimeStamp:false. |
|
3815 |
"/ sourceA := aStream contents asString. |
|
3816 |
||
3817 |
"/ aStream := '' writeStream. |
|
3818 |
"/ Method flushSourceStreamCache. |
|
3819 |
"/ classB fileOutOn:aStream withTimeStamp:false. |
|
3820 |
"/ sourceB := aStream contents asString. |
|
3821 |
||
3822 |
"/ theChangeSetA:=self changeSetForClass:classA andSource:sourceA. |
|
3823 |
"/ theChangeSetB:=self changeSetForClass:classB andSource:sourceB. |
|
3824 |
"/ theChangeSetA isNil ifTrue: [theChangeSetA := ChangeSet new]. |
|
3825 |
"/ theChangeSetB isNil ifTrue: [theChangeSetB := ChangeSet new]. |
|
2949 | 3826 |
|
3827 |
"/ just in case (if comparing a class against another), |
|
3828 |
"/ unify the classes of the changes (to avoid that all changes are detected as different) |
|
3829 |
||
16615 | 3830 |
"/ theChangeSetB do:[:eachChange | |
3831 |
"/ eachChange isMethodChange ifTrue:[ |
|
3832 |
"/ eachChange changeClass isMeta ifTrue:[ |
|
3833 |
"/ eachChange changeClass ~~ classA theMetaclass ifTrue:[ |
|
3834 |
"/ eachChange changeClass:classA theMetaclass. |
|
3835 |
"/ ] |
|
3836 |
"/ ] ifFalse:[ |
|
3837 |
"/ eachChange changeClass ~~ classA theNonMetaclass ifTrue:[ |
|
3838 |
"/ eachChange changeClass:classA theNonMetaclass. |
|
3839 |
"/ ] |
|
3840 |
"/ ]. |
|
3841 |
"/ ]. |
|
3842 |
"/ ]. |
|
2949 | 3843 |
"/ remove all #initialize doIts |
16615 | 3844 |
theChangeSetA := theChangeSetA reject:[:eachChange | eachChange isDoIt and:[eachChange isInitialize]]. |
3845 |
theChangeSetB := theChangeSetB reject:[:eachChange | eachChange isDoIt and:[eachChange isInitialize]]. |
|
3846 |
||
3847 |
theClassChangeSet diffSet:(theChangeSetA diffSetsAgainst:theChangeSetB comparingDifferentClasses:true). |
|
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3848 |
^ theClassChangeSet |
2949 | 3849 |
! |
3850 |
||
2497 | 3851 |
newForClass:aClass labelA:aLabelA sourceA:aSourceA labelB:aLabelB sourceB:aSourceB |
3730
717e425e843e
allow for private classes to be compared
Claus Gittinger <cg@exept.de>
parents:
3687
diff
changeset
|
3852 |
"return a ClassChangeSet for the class aClass and the two sources. |
11453 | 3853 |
The two classes are compared via the source files. A new instance of |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3854 |
ClassChangeSet is generated containing the correspondenting changes." |
3730
717e425e843e
allow for private classes to be compared
Claus Gittinger <cg@exept.de>
parents:
3687
diff
changeset
|
3855 |
|
4128 | 3856 |
|theChangeSetA theChangeSetB theClassChangeSet className metaClassName| |
2497 | 3857 |
|
3858 |
theClassChangeSet := self new. |
|
3859 |
theClassChangeSet classBeingCompared:aClass. |
|
3860 |
theClassChangeSet labelA:aLabelA. |
|
3861 |
theClassChangeSet labelB:aLabelB. |
|
11453 | 3862 |
theChangeSetA:=self changeSetForClass:aClass andSource:aSourceA. |
2497 | 3863 |
theChangeSetB:=self changeSetForClass:aClass andSource:aSourceB. |
3730
717e425e843e
allow for private classes to be compared
Claus Gittinger <cg@exept.de>
parents:
3687
diff
changeset
|
3864 |
|
4128 | 3865 |
className := aClass name. |
3866 |
metaClassName := aClass class name. |
|
3867 |
||
3730
717e425e843e
allow for private classes to be compared
Claus Gittinger <cg@exept.de>
parents:
3687
diff
changeset
|
3868 |
aClass isPrivate ifTrue:[ |
11453 | 3869 |
theChangeSetA := theChangeSetA select:[:change | change isMethodChange not |
3870 |
or:[change className = className or:[change className = metaClassName]]]. |
|
3871 |
theChangeSetA := theChangeSetA select:[:change | change isClassDefinitionChange not |
|
3872 |
or:[change className = className or:[change className = metaClassName]]]. |
|
3873 |
theChangeSetA := theChangeSetA select:[:change | change isDoIt not |
|
3874 |
or:[change receiverClassName = className or:[change receiverClassName = metaClassName]]]. |
|
3875 |
theChangeSetB := theChangeSetB select:[:change | change isMethodChange not |
|
3876 |
or:[change className = className or:[change className = metaClassName]]]. |
|
3877 |
theChangeSetB := theChangeSetB select:[:change | change isClassDefinitionChange not |
|
3878 |
or:[change className = className or:[change className = metaClassName]]]. |
|
3879 |
theChangeSetB := theChangeSetB select:[:change | change isDoIt not |
|
3880 |
or:[change receiverClassName = className or:[change receiverClassName = metaClassName]]]. |
|
3881 |
"/ there might be more needed... |
|
3730
717e425e843e
allow for private classes to be compared
Claus Gittinger <cg@exept.de>
parents:
3687
diff
changeset
|
3882 |
]. |
717e425e843e
allow for private classes to be compared
Claus Gittinger <cg@exept.de>
parents:
3687
diff
changeset
|
3883 |
|
2497 | 3884 |
theChangeSetA isNil |
11453 | 3885 |
ifTrue: [theChangeSetA := ChangeSet new]. |
2497 | 3886 |
theChangeSetB isNil |
11453 | 3887 |
ifTrue: [theChangeSetB := ChangeSet new]. |
2947 | 3888 |
|
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3889 |
theClassChangeSet diffSet:(theChangeSetA diffSetsAgainst:theChangeSetB). |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3890 |
^ theClassChangeSet |
2465 | 3891 |
! |
3892 |
||
2497 | 3893 |
newForClass:aClass versionA:aVersionA versionB:aVersionB |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3894 |
"return a ClassChangeSet for the class aClass and the two versions. |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3895 |
The two class version are checked out from the repository and then being |
11453 | 3896 |
compared. A new instance of ClassChangeSet is generated containing |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3897 |
the correspondenting changes." |
2497 | 3898 |
|
3899 |
|theChangeSetA theChangeSetB theClassChangeSet| |
|
3900 |
||
3901 |
theClassChangeSet := self new. |
|
3902 |
theClassChangeSet classBeingCompared:aClass. |
|
2856 | 3903 |
theClassChangeSet versionA:aVersionA. |
3904 |
theClassChangeSet versionB:aVersionB. |
|
2497 | 3905 |
theClassChangeSet labelA:aVersionA. |
3906 |
theClassChangeSet labelB:aVersionB. |
|
3907 |
theChangeSetA := self changeSetForClass: aClass andRevision: aVersionA. |
|
3908 |
theChangeSetB := self changeSetForClass: aClass andRevision: aVersionB. |
|
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3909 |
theChangeSetA isNil ifTrue: [theChangeSetA := ChangeSet new]. |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3910 |
theChangeSetB isNil ifTrue: [theChangeSetB := ChangeSet new]. |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3911 |
theClassChangeSet diffSet:(theChangeSetA diffSetsAgainst:theChangeSetB). |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3912 |
^ theClassChangeSet |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3913 |
! |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3914 |
|
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3915 |
sortChangeSet:aChangeSet |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3916 |
"/ sort by meta and selector |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
3917 |
|
11453 | 3918 |
aChangeSet |
14363 | 3919 |
sort:[:a :b | |
3920 |
a == b ifTrue:[ |
|
3921 |
false |
|
3922 |
] ifFalse:[ |
|
3923 |
a isClassDefinitionChange == b isClassDefinitionChange ifTrue:[ |
|
3924 |
a isClassChange == b isClassChange ifTrue:[ |
|
3925 |
a isClassChange ifTrue:[ |
|
3926 |
a isForMeta == b isForMeta ifTrue:[ |
|
3927 |
(a isMethodChange and:[b isMethodChange]) ifTrue:[ |
|
3928 |
a className < b className |
|
3929 |
or:[ a className = b className |
|
3930 |
and:[ a isPrimitiveChange not |
|
3931 |
and:[ b isPrimitiveChange not |
|
17347 | 3932 |
and:[ a changeSelector < b changeSelector ]]]] |
14363 | 3933 |
] ifFalse:[ |
3934 |
a isMethodChange not |
|
3935 |
]. |
|
3936 |
] ifFalse:[ |
|
3937 |
a isForMeta |
|
3938 |
]. |
|
3939 |
] ifFalse:[ |
|
3940 |
a printString < b printString |
|
3941 |
]. |
|
3942 |
] ifFalse:[ |
|
3943 |
a isClassChange |
|
3944 |
]. |
|
3945 |
] ifFalse:[ |
|
3946 |
a isClassDefinitionChange |
|
3947 |
]. |
|
3948 |
] |
|
3949 |
]. |
|
17347 | 3950 |
|
3951 |
"Modified: / 06-02-2017 / 02:31:57 / cg" |
|
2497 | 3952 |
! ! |
3953 |
||
3954 |
!VersionDiffBrowser::ClassChangeSet methodsFor:'accessing'! |
|
3955 |
||
13137 | 3956 |
class1BeingCompared |
3957 |
"returns the class which is compared" |
|
3958 |
||
3959 |
^ class1BeingCompared |
|
3960 |
! |
|
3961 |
||
3962 |
class1BeingCompared:class1 class2BeingCompared:class2 |
|
3963 |
"returns the class which is compared" |
|
3964 |
||
3965 |
class1BeingCompared := class1. |
|
3966 |
class2BeingCompared := class2 |
|
3967 |
! |
|
3968 |
||
3969 |
class2BeingCompared |
|
3970 |
"returns the class which is compared" |
|
3971 |
||
3972 |
^ class2BeingCompared |
|
3973 |
! |
|
3974 |
||
2497 | 3975 |
classBeingCompared |
13137 | 3976 |
"returns the class which is compared" |
3977 |
||
3978 |
^ class1BeingCompared |
|
2497 | 3979 |
! |
2221 | 3980 |
|
2497 | 3981 |
classBeingCompared:something |
3756 | 3982 |
"set the value of the class which is compared" |
2497 | 3983 |
|
13137 | 3984 |
class1BeingCompared := class2BeingCompared := something. |
2497 | 3985 |
! |
3986 |
||
2695 | 3987 |
diffSet |
3756 | 3988 |
"returns a diffSet containing the different changes. |
3989 |
it responds to: |
|
2695 | 3990 |
|
11453 | 3991 |
changed OrderedCollection of arrays containing ChangeSets for methods |
3992 |
which are changed and exists in the class versionA and versionB |
|
3756 | 3993 |
onlyInArg ChangeSet for the methods which exists only in the class versionB |
3994 |
onlyInReceiver ChangeSet for the methods which exists only in the class versionA |
|
11453 | 3995 |
" |
3756 | 3996 |
|
2695 | 3997 |
^ diffSet |
3998 |
! |
|
3999 |
||
4000 |
diffSet:something |
|
3756 | 4001 |
"sets the diffSet containing the different changes. |
4002 |
it responds to: |
|
2695 | 4003 |
|
11453 | 4004 |
changed OrderedCollection of arrays containing ChangeSets for methods |
4005 |
which are changed and exists in the class versionA and versionB |
|
3756 | 4006 |
onlyInArg ChangeSet for the methods which exists only in the class versionB |
4007 |
onlyInReceiver ChangeSet for the methods which exists only in the class versionA |
|
11453 | 4008 |
" |
2695 | 4009 |
|
4010 |
diffSet := something. |
|
4011 |
! |
|
4012 |
||
2497 | 4013 |
labelA |
3756 | 4014 |
"returns the label for the class versionA" |
2497 | 4015 |
|
4016 |
^ labelA |
|
4017 |
! |
|
4018 |
||
4019 |
labelA:something |
|
3756 | 4020 |
"sets the label for the class versionA" |
2497 | 4021 |
|
4022 |
labelA := something. |
|
4023 |
! |
|
4024 |
||
4025 |
labelB |
|
3756 | 4026 |
"returns the label for the class versionB" |
2497 | 4027 |
|
4028 |
^ labelB |
|
4029 |
! |
|
2465 | 4030 |
|
2497 | 4031 |
labelB:something |
3756 | 4032 |
"sets the label for the class versionB" |
2497 | 4033 |
|
4034 |
labelB := something. |
|
4035 |
! |
|
4036 |
||
4037 |
methodsChanged |
|
11453 | 4038 |
"returns an OrderedCollection of arrays containing ChangeSets for methods |
3756 | 4039 |
which are changed and exists in the class versionA and versionB |
2497 | 4040 |
|
3756 | 4041 |
<return: OrderedCollection of Array(2)> |
4042 |
" |
|
4043 |
||
4044 |
^ self diffSet changed |
|
2497 | 4045 |
! |
4046 |
||
4047 |
methodsOnlyInA |
|
11453 | 4048 |
"returns a ChangeSet for the methods which exists only in the class versionA" |
2497 | 4049 |
|
3756 | 4050 |
^ self diffSet onlyInReceiver |
2497 | 4051 |
! |
4052 |
||
4053 |
methodsOnlyInB |
|
11453 | 4054 |
"returns a ChangeSet for the methods which exists only in the class versionB" |
2497 | 4055 |
|
3756 | 4056 |
^ self diffSet onlyInArg |
2856 | 4057 |
! |
4058 |
||
4059 |
versionA |
|
4060 |
^ versionA |
|
4061 |
! |
|
4062 |
||
4063 |
versionA:something |
|
4064 |
versionA := something. |
|
4065 |
! |
|
4066 |
||
4067 |
versionB |
|
4068 |
^ versionB |
|
4069 |
! |
|
4070 |
||
4071 |
versionB:something |
|
4072 |
versionB := something. |
|
2221 | 4073 |
! ! |
4074 |
||
8891 | 4075 |
!VersionDiffBrowser::ClassChangeSet methodsFor:'misc'! |
4076 |
||
4077 |
removeAllVersionMethods |
|
4078 |
diffSet removeAllVersionMethods. |
|
4079 |
! ! |
|
4080 |
||
4081 |
!VersionDiffBrowser::ClassChangeSet methodsFor:'setup'! |
|
4082 |
||
4083 |
setupForClass:aClass againstVersion:aVersionA |
|
4084 |
"return a ClassChangeSet for the class aClass against some verison in the repository. |
|
11453 | 4085 |
A new instance of ClassChangeSet is generated containing |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
4086 |
the correspondenting changes." |
8891 | 4087 |
|
4088 |
|theChangeSetA theChangeSetB versionCompared| |
|
4089 |
||
4090 |
self classBeingCompared:aClass. |
|
4091 |
versionCompared := aVersionA. |
|
4092 |
aVersionA isNil ifTrue:[ |
|
11453 | 4093 |
self labelA:(versionCompared := aClass revision). |
4094 |
self versionA:versionCompared. |
|
8891 | 4095 |
] ifFalse:[ |
11453 | 4096 |
aVersionA == #newest ifTrue:[ |
4097 |
self labelA:(VersionDiffBrowser resources string:'Newest'). |
|
4098 |
] ifFalse:[ |
|
4099 |
self labelA:versionCompared. |
|
4100 |
self versionA:versionCompared. |
|
4101 |
] |
|
8891 | 4102 |
]. |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
4103 |
self labelB:(VersionDiffBrowser resources string:'Current'). |
8891 | 4104 |
theChangeSetA := self class changeSetForClass: aClass andRevision: versionCompared. |
4105 |
theChangeSetB := self class changeSetForClass: aClass. |
|
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
4106 |
theChangeSetA isNil ifTrue: [theChangeSetA := ChangeSet new]. |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
4107 |
theChangeSetB isNil ifTrue: [theChangeSetB := ChangeSet new]. |
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
4108 |
|
8891 | 4109 |
"/ if we are comparing a private class, prune out other changes |
4110 |
aClass isPrivate ifTrue:[ |
|
11453 | 4111 |
theChangeSetA removeAllSuchThat:[:aChange | aChange className ~= aClass name ]. |
4112 |
theChangeSetB removeAllSuchThat:[:aChange | aChange className ~= aClass name ]. |
|
8891 | 4113 |
]. |
9239
1cfdf9dae50b
double click; extensions-filter; better printStrings
Claus Gittinger <cg@exept.de>
parents:
8891
diff
changeset
|
4114 |
self activityNotification:'Generating diff-set...'. |
8891 | 4115 |
self diffSet:(theChangeSetA diffSetsAgainst:theChangeSetB). |
4116 |
self activityNotification:nil. |
|
4117 |
^ diffSet |
|
4118 |
! ! |
|
4119 |
||
17341 | 4120 |
!VersionDiffBrowser::FilterParameters methodsFor:'accessing'! |
4121 |
||
4122 |
filteredClassNameMatchPattern |
|
4123 |
^ filteredClassNameMatchPattern |
|
4124 |
! |
|
4125 |
||
4126 |
filteredClassNameMatchPattern:something |
|
4127 |
filteredClassNameMatchPattern := something. |
|
4128 |
! |
|
4129 |
||
4130 |
filteredClassNames |
|
4131 |
^ filteredClassNames |
|
4132 |
! |
|
4133 |
||
4134 |
filteredClassNames:something |
|
4135 |
filteredClassNames := something. |
|
4136 |
! |
|
4137 |
||
4138 |
filteredMethodNames |
|
4139 |
^ filteredMethodNames |
|
4140 |
! |
|
4141 |
||
4142 |
filteredMethodNames:something |
|
4143 |
filteredMethodNames := something. |
|
4144 |
! |
|
4145 |
||
4146 |
filteredSelectorMatchPattern |
|
4147 |
^ filteredSelectorMatchPattern |
|
4148 |
! |
|
4149 |
||
4150 |
filteredSelectorMatchPattern:something |
|
4151 |
filteredSelectorMatchPattern := something. |
|
4152 |
! |
|
4153 |
||
4154 |
filteredSelectors |
|
4155 |
^ filteredSelectors |
|
4156 |
! |
|
4157 |
||
4158 |
filteredSelectors:something |
|
4159 |
filteredSelectors := something. |
|
4160 |
! ! |
|
4161 |
||
2221 | 4162 |
!VersionDiffBrowser class methodsFor:'documentation'! |
4163 |
||
4164 |
version |
|
16530 | 4165 |
^ '$Header$' |
8811
65e08571a2f3
changed: #openOnClass:labelA:sourceA:labelB:sourceB:title:ifSame:
fm
parents:
8657
diff
changeset
|
4166 |
! |
65e08571a2f3
changed: #openOnClass:labelA:sourceA:labelB:sourceB:title:ifSame:
fm
parents:
8657
diff
changeset
|
4167 |
|
65e08571a2f3
changed: #openOnClass:labelA:sourceA:labelB:sourceB:title:ifSame:
fm
parents:
8657
diff
changeset
|
4168 |
version_CVS |
16530 | 4169 |
^ '$Header$' |
2221 | 4170 |
! ! |
12512 | 4171 |