D23419: Fix problems when importing new Data

[Martin Marmsoler]

Murmele added inline comments.

INLINE COMMENTS

> asemke wrote in Project.cpp:268
> where is the pointer column defined?

at the beginning of the function:

void Project::descriptionChanged(const AbstractAspect* aspect) {
	if (isLoading())

  		return;

if (this != aspect) {

  		const AbstractColumn* column = dynamic_cast<const AbstractColumn*>(aspect);

> asemke wrote in Spreadsheet.cpp:908
> where is this signal consumed?

In macrosXYCurve.h
it is connected to ColumnAboutToBeRemoved

> asemke wrote in XYCurve.h:136
> do I see it correctly that with this patch the connection of a column with a curve is lost if the column is renamed? At the moment it's possible to rename the columns (for example from "my great y-data" to "my great and fancy y-data") without affecting the curves - is this not possible anymore since we work now with your patch with names and not with the pointers anymore?

This makes the call of reset() from the column. This signal is connected to ColumnAboutToBeRemoved in XYCurve
see macrosXYCurve.h

template instead of a macro would be better, but I don't know how to create one in this case

REPOSITORY
  R262 LabPlot

REVISION DETAIL
  https://phabricator.kde.org/D23419

To: Murmele, asemke
Cc: yurchor, kde-edu, narvaez, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20190825/df37c0b3/attachment.html>