Kiln » TortoiseHg » TortoiseHg
Clone URL:  
Pushed to one repository · View In Graph Contained in 1.9, 1.9.1, and 1.9.2

commit: use thgrepo event signals

Misc cleanups while I was digging around:
* Allow each dialog to connect itself to repo signals
* Give CommitWidget it's own repo reference
* Replace some old-style SIGNAL references

Changeset fab3b7f03ecc

Parent faa1ad160d4e

by Steve Borho

Changes to 3 files · Browse files at fab3b7f03ecc Showing diff from parent faa1ad160d4e Diff from another changeset...

 
46
47
48
 
49
 
50
51
52
 
 
 
53
54
55
 
73
74
75
76
 
77
78
79
 
89
90
91
92
 
93
94
95
 
127
128
129
 
 
 
 
 
 
 
 
130
131
132
133
 
 
 
 
 
 
 
134
135
136
 
137
138
139
 
156
157
158
159
 
160
161
162
 
167
168
169
170
 
171
172
173
174
175
176
177
178
179
180
181
 
 
182
183
184
 
294
295
296
297
298
299
 
 
 
 
300
301
302
303
304
305
306
307
308
 
 
 
 
309
310
311
312
313
314
315
 
 
316
317
318
 
324
325
326
327
 
 
328
329
330
 
331
332
333
334
335
 
336
337
 
338
339
340
 
350
351
352
353
354
355
 
 
 
356
357
 
358
359
360
 
385
386
387
388
389
 
390
391
392
 
395
396
397
398
 
399
400
401
 
404
405
406
407
408
 
409
410
411
412
413
414
 
415
416
417
 
438
439
440
441
 
442
443
444
 
449
450
451
452
 
453
454
 
455
456
457
458
459
460
461
462
463
 
 
464
465
466
 
472
473
474
475
476
477
478
 
579
580
581
 
 
582
583
584
 
613
614
615
616
 
617
618
619
 
742
743
744
745
746
 
 
747
748
749
 
884
885
886
887
888
 
 
889
890
891
 
903
904
905
906
 
907
908
909
 
937
938
939
940
941
942
943
 
46
47
48
49
50
51
52
53
 
54
55
56
57
58
59
 
77
78
79
 
80
81
82
83
 
93
94
95
 
96
97
98
99
 
131
132
133
134
135
136
137
138
139
140
141
142
 
 
 
143
144
145
146
147
148
149
150
151
 
152
153
154
155
 
172
173
174
 
175
176
177
178
 
183
184
185
 
186
187
188
189
 
 
 
190
191
 
 
 
192
193
194
195
196
 
306
307
308
 
 
 
309
310
311
312
313
314
315
316
317
318
 
 
 
319
320
321
322
323
324
325
 
326
 
 
327
328
329
330
331
 
337
338
339
 
340
341
342
343
 
344
345
346
347
 
 
348
349
 
350
351
352
353
 
363
364
365
 
 
 
366
367
368
369
 
370
371
372
373
 
398
399
400
 
 
401
402
403
404
 
407
408
409
 
410
411
412
413
 
416
417
418
 
 
419
420
421
422
423
424
 
425
426
427
428
 
449
450
451
 
452
453
454
455
 
460
461
462
 
463
464
 
465
466
467
468
469
 
470
471
 
 
472
473
474
475
476
 
482
483
484
 
485
486
487
 
588
589
590
591
592
593
594
595
 
624
625
626
 
627
628
629
630
 
753
754
755
 
 
756
757
758
759
760
 
895
896
897
 
 
898
899
900
901
902
 
914
915
916
 
917
918
919
920
 
948
949
950
 
951
952
953
@@ -46,10 +46,14 @@
  self.msghistory = []   self.qref = False   + self.repo = repo = self.stwidget.repo   self.runner = cmdui.Runner(_('Commit'), self, logwidget) + self.runner.commandStarted.connect(repo.incrementBusyCount)   self.runner.commandFinished.connect(self.commandFinished)   - repo = self.stwidget.repo + repo.configChanged.connect(self.configChanged) + repo.repositoryChanged.connect(self.repositoryChanged) +   self.opts['pushafter'] = repo.ui.config('tortoisehg', 'cipushafter', '')   self.opts['autoinc'] = repo.ui.config('tortoisehg', 'autoinc', '')   @@ -73,7 +77,7 @@
  self.buttonHBox = hbox     msgcombo = MessageHistoryCombo() - self.connect(msgcombo, SIGNAL('activated(int)'), self.msgSelected) + msgcombo.activated.connect(self.msgSelected)   hbox.addWidget(msgcombo, 1)   hbox.addSpacing(2)   vbox.addLayout(hbox, 0) @@ -89,7 +93,7 @@
    msgte = QPlainTextEdit()   msgte.setLineWrapMode(QPlainTextEdit.NoWrap) - msgfont = qtlib.getfont(self.stwidget.repo.ui, 'fontcomment') + msgfont = qtlib.getfont(self.repo.ui, 'fontcomment')   msgte.setFont(msgfont.font())   msgfont.changed.connect(lambda fnt: msgte.setFont(fnt))   msgte.textChanged.connect(self.msgChanged) @@ -127,13 +131,25 @@
  if dlg.exec_() == QDialog.Accepted:   self.opts.update(dlg.outopts)   + def repositoryChanged(self): + 'Repository has detected a changelog / dirstate change' + self.refresh() + + def configChanged(self): + 'Repository is reporting its config files have changed' + self.refresh() +   def reload(self): - repo = self.stwidget.repo - repo.thginvalidate() - wctx = repo[None] + 'User has requested a reload' + self.repo.thginvalidate() + self.refresh() + self.stwidget.refreshWctx() # Trigger reload of working context + + def refresh(self): + wctx = self.repo[None]     # Update qrefresh mode - if repo.changectx('.').thgmqappliedpatch(): + if self.repo.changectx('.').thgmqappliedpatch():   self.commitButtonName.emit(_('QRefresh'))   if not self.qref:   self.initQRefreshMode() @@ -156,7 +172,7 @@
  self.branchbutton.setText(title)     # Update parent revision(s) - for i, ctx in enumerate(repo.parents()): + for i, ctx in enumerate(self.repo.parents()):   desc = format_desc(ctx.description(), 80)   fmt = "<span style='font-family:Courier'>%s(%s)</span> %s"   ptext = fmt % (ctx.rev(), short_hex(ctx.node()), desc) @@ -167,18 +183,14 @@
  self.parentlabels.append(lbl)   else:   self.parentlabels[i].setText(ptext) - while len(repo.parents()) > len(self.parentlabels): + while len(self.repo.parents()) > len(self.parentlabels):   w = self.parentlabels.pop()   self.parentvbox.removeWidget(w)   - # Trigger reload of working context - self.stwidget.refreshWctx() -   def initQRefreshMode(self):   'Working parent is a patch. Is it refreshable?' - repo = self.stwidget.repo - qtip = repo['qtip'] - if qtip != repo['.']: + qtip = self.repo['qtip'] + if qtip != self.repo['.']:   self.showMessage.emit(_('Cannot refresh non-tip patch'))   self.commitButtonName.emit(_('N/A'))   return @@ -294,25 +306,26 @@
  def settings():   from tortoisehg.hgqt.settings import SettingsDialog   dlg = SettingsDialog(True, focus='tortoisehg.summarylen') - if dlg.exec_() == QDialog.Accepted: - repo = self.stwidget.repo - repo.ui = hglib.reloadui(repo.root) + self.repo.incrementBusyCount() + ret = dlg.exec_() + self.repo.decrementBusyCount() + if ret == QDialog.Accepted:   self.msgChanged()     menu = self.msgte.createStandardContextMenu()   for name, func in [(_('Paste &Filenames'), paste),   (_('App&ly Format'), apply),   (_('C&onfigure Format'), settings)]: - action = menu.addAction(name) - action.wrapper = lambda f=func: f() - self.connect(action, SIGNAL('triggered()'), action.wrapper) + def add(name, func): + action = menu.addAction(name) + action.triggered.connect(lambda: func()) + add(name, func)   return menu.exec_(point)     def getLengths(self): - repo = self.stwidget.repo   try: - sumlen = int(repo.ui.config('tortoisehg', 'summarylen', 0)) - maxlen = int(repo.ui.config('tortoisehg', 'messagewrap', 0)) + sumlen = int(self.repo.ui.config('tortoisehg', 'summarylen', 0)) + maxlen = int(self.repo.ui.config('tortoisehg', 'messagewrap', 0))   except (TypeError, ValueError):   sumlen, maxlen = 0, 0   return sumlen, maxlen @@ -324,17 +337,17 @@
  return self.stwidget.saveState()     def branchOp(self): - d = branchop.BranchOpDialog(self.stwidget.repo, self.branchop) + d = branchop.BranchOpDialog(self.repo, self.branchop) + self.repo.incrementBusyCount()   if d.exec_() == QDialog.Accepted:   self.branchop = d.branchop - self.reload() + self.repo.decrementBusyCount()     def canUndo(self):   'Returns undo description or None if not valid' - repo = self.stwidget.repo - if os.path.exists(repo.sjoin('undo')): + if os.path.exists(self.repo.sjoin('undo')):   try: - args = repo.opener('undo.desc', 'r').read().splitlines() + args = self.repo.opener('undo.desc', 'r').read().splitlines()   if args[1] != 'commit':   return None   return _('Rollback commit to revision %d') % (int(args[0]) - 1) @@ -350,11 +363,11 @@
  QMessageBox.Ok | QMessageBox.Cancel)   if d != QMessageBox.Ok:   return - repo = self.stwidget.repo - repo.rollback() - repo.thginvalidate() + self.repo.incrementBusyCount() + self.repo.rollback() + self.repo.decrementBusyCount()   self.reload() - QTimer.singleShot(500, lambda: shlib.shell_notify([repo.root])) + QTimer.singleShot(500, lambda: shlib.shell_notify([self.repo.root]))     def getMessage(self):   text = self.msgte.toPlainText() @@ -385,8 +398,7 @@
    def loadConfigs(self, s):   'Load history, etc, from QSettings instance' - repo = self.stwidget.repo - repoid = str(repo[0]) + repoid = str(self.repo[0])   # message history is stored in unicode   self.split.restoreState(s.value('commit/split').toByteArray())   self.msghistory = list(s.value('commit/history-'+repoid).toStringList()) @@ -395,7 +407,7 @@
  self.userhist = s.value('commit/userhist').toStringList()   self.userhist = [u for u in self.userhist if u]   try: - curmsg = repo.opener('cur-message.txt').read() + curmsg = self.repo.opener('cur-message.txt').read()   self.msgte.setPlainText(hglib.tounicode(curmsg))   self.msgte.document().setModified(False)   self.msgte.moveCursor(QTextCursor.End) @@ -404,14 +416,13 @@
    def storeConfigs(self, s):   'Save history, etc, in QSettings instance' - repo = self.stwidget.repo - repoid = str(repo[0]) + repoid = str(self.repo[0])   s.setValue('commit/history-'+repoid, self.msghistory)   s.setValue('commit/split', self.split.saveState())   s.setValue('commit/userhist', self.userhist)   try:   # current message is stored in local encoding - repo.opener('cur-message.txt', 'w').write(self.getMessage()) + self.repo.opener('cur-message.txt', 'w').write(self.getMessage())   except EnvironmentError:   pass   @@ -438,7 +449,7 @@
    # 2. Read from repository   try: - return self.stwidget.repo.ui.username() + return self.repo.ui.username()   except error.Abort:   pass   @@ -449,18 +460,17 @@
  from tortoisehg.hgqt.settings import SettingsDialog   dlg = SettingsDialog(False, focus='ui.username')   dlg.exec_() - self.stwidget.repo.ui.invalidateui() + self.repo.ui.invalidateui()   try: - return self.stwidget.repo.ui.username() + return self.repo.ui.username()   except error.Abort:   return None     def commit(self): - repo = self.stwidget.repo   cwd = os.getcwd()   try: - os.chdir(repo.root) - return self._commit(repo) + os.chdir(self.repo.root) + return self._commit(self.repo)   finally:   os.chdir(cwd)   @@ -472,7 +482,6 @@
  parent=self)   self.msgte.setFocus()   return - repo = self.stwidget.repo   if self.branchop is None:   brcmd = []   elif self.branchop == False: @@ -579,6 +588,8 @@
  self.msgte.clear()   self.msgte.document().setModified(False)   self.commitComplete.emit() + self.repo.decrementBusyCount() + self.stwidget.refreshWctx()     def keyPressEvent(self, event):   if event.key() in (Qt.Key_Return, Qt.Key_Enter): @@ -613,7 +624,7 @@
  'Utility dialog for configuring uncommon settings'   def __init__(self, opts, userhistory, parent):   QDialog.__init__(self, parent) - self.repo = parent.stwidget.repo + self.repo = parent.repo     layout = QVBoxLayout()   self.setLayout(layout) @@ -742,8 +753,8 @@
    BB = QDialogButtonBox   bb = QDialogButtonBox(BB.Ok|BB.Cancel) - self.connect(bb, SIGNAL("accepted()"), self, SLOT("accept()")) - self.connect(bb, SIGNAL("rejected()"), self, SLOT("reject()")) + bb.accepted.connect(self.accept) + bb.rejected.connect(self.reject)   self.bb = bb   layout.addWidget(bb)   @@ -884,8 +895,8 @@
    BB = QDialogButtonBox   bb = QDialogButtonBox(BB.Ok|BB.Cancel|BB.Discard) - self.connect(bb, SIGNAL("accepted()"), self, SLOT("accept()")) - self.connect(bb, SIGNAL("rejected()"), self, SLOT("reject()")) + bb.accepted.connect(self.accept) + bb.rejected.connect(self.reject)   bb.button(BB.Discard).setText('Undo')   bb.button(BB.Discard).clicked.connect(commit.rollback)   bb.button(BB.Cancel).setDefault(False) @@ -903,7 +914,7 @@
  commit.commitComplete.connect(self.postcommit)   commit.commitButtonName.connect(self.setButtonName)   - name = hglib.get_reponame(commit.stwidget.repo) + name = hglib.get_reponame(commit.repo)   self.setWindowTitle('%s - commit' % name)   self.commit = commit   self.commit.reload() @@ -937,7 +948,6 @@
  if repo.ui.configbool('tortoisehg', 'closeci'):   self.reject()   return - self.commit.reload()     def accept(self):   self.commit.commit()
 
135
136
137
138
139
140
141
142
 
162
163
164
165
166
167
168
 
135
136
137
 
 
138
139
140
 
160
161
162
 
163
164
165
@@ -135,8 +135,6 @@
  cw.buttonHBox.addWidget(b)   cw.commitButtonName.connect(lambda n: b.setText(n))   cw.loadConfigs(QSettings()) - self.repo.configChanged.connect(cw.configChanged) - self.repo.repositoryChanged.connect(cw.repositoryChanged)   cw.reload()   b.clicked.connect(cw.commit)   self.repo._commitwidget = cw @@ -162,7 +160,6 @@
  self.repo._syncwidget = sw   sw.outgoingNodes.connect(self.setOutgoingNodes)   sw.showMessage.connect(self.showMessage) - self.repo.configChanged.connect(sw.configChanged)   return SharedWidget(sw)     def setOutgoingNodes(self, nodes):
 
48
49
50
51
 
 
52
53
54
 
48
49
50
 
51
52
53
54
55
@@ -48,7 +48,8 @@
  if not log:   self.setWindowTitle(_('TortoiseHg Sync'))   self.resize(850, 550) - self.repo.configChanged.connect(self.configChanged) + + self.repo.configChanged.connect(self.configChanged)     hbox = QHBoxLayout()   hbox.setContentsMargins(0, 0, 0, 0)