Skip to content
This repository has been archived by the owner on Oct 16, 2020. It is now read-only.

Fixed bug #674 Unhandled Exception when Moving Folder of Classes from Project A to Project B #718

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

daniil-tr
Copy link
Contributor

Hello. Should I write every times about license? Or is once enough?

I certify that I own, and have sufficient rights to contribute, all source code and related material intended to be compiled or integrated with the source code for the #develop open source product (the "Contribution"). My Contribution is licensed under the MIT License.

@mrward
Copy link
Member

mrward commented Nov 10, 2015

You can sign the JCA or include the comment about the MIT license with each pull request.

This pull request has a merge conflict. I cherry picked your original commit and re-formatted the code.

Another problem is that the behaviour seems to be incorrect since the copy does not happen if (includeInProject && ProjectService.OpenSolution != null) is false when previously it did copy. I guess it could be added in an else part of that if block.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants