Skip to content

Copy Paste Source files#10

Merged
eduardoconto merged 6 commits into
masterfrom
copypaste_srcfiles
Feb 24, 2026
Merged

Copy Paste Source files#10
eduardoconto merged 6 commits into
masterfrom
copypaste_srcfiles

Conversation

@shweta1096

Copy link
Copy Markdown
Collaborator
  • Only nodes and connections were copied earlier. Added copy paste of data and fn src files also.
  • Fixes bug.

Shweta Menon added 3 commits February 19, 2026 18:20
- Only nodes and connections
  were copied earlier. Added
  copy paste of data and fn src
  files also.
- Fixes bug.
Comment thread src/UndoCommands.cpp Outdated
targetDir = dagScene->getDataDir();
}

QJsonArray nodesJsonArray = _newSceneJson["nodes"].toArray();

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

make this in a function

Comment thread src/UndoCommands.cpp Outdated
setObsolete(true);
return;
}
// Wrap scene and sourceDir

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For copy-paste data: include the data path (sourceDir) in the scene

Shweta Menon added 2 commits February 24, 2026 12:26
- make function private
Comment thread src/UndoCommands.cpp

if (QFile::exists(srcPath) && !QFile::exists(targetPath)) {
QFile::copy(srcPath, targetPath);
}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

else {
warning("Cannot copy file {srcPath} to {targetPath}")
}

@eduardoconto eduardoconto merged commit 7c44af0 into master Feb 24, 2026
3 checks passed
@eduardoconto eduardoconto deleted the copypaste_srcfiles branch February 24, 2026 05:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants