Differential D10745 Diff 27954 app/fileopscontextmanageritem.cpp

Changeset View

Standalone View

Context not available.
350	QModelIndex index = mThumbnailView->currentIndex();	350		QModelIndex index = mThumbnailView->currentIndex();
351	mThumbnailView->edit(index);	351		mThumbnailView->edit(index);
352	} else {	352		} else {
353	FileOperations::rename(urlList().first(), mGroup);	353		FileOperations::rename(urlList().first(), mGroup, contextManager());
354	contextManager()->slotSelectionChanged();	354		contextManager()->slotSelectionChanged();
			muhlenpfordtUnsubmitted Not Done Should we move this inside `rename()`? muhlenpfordt: Should we move this inside `rename()`?
			rkflxUnsubmitted Not Done My plan was to do the opposite: Move everything `contextManager` related to `fileopscontextmanageritem.cpp`, because I think D6379 muddled the separation a bit. I'd leave it as is for this patch, and once this one is in I'll send a follow-up fixing it everywhere (if you all agree). rkflx: My plan was to do the opposite: Move everything `contextManager` related to…
			muhlenpfordtUnsubmitted Not Done I don't care about the place, my intention was just to keep it in one place - agreed. ;) muhlenpfordt: I don't care about the place, my intention was just to keep it in one place - agreed. ;)
355	}	355		}
356	}	356		}
Context not available.