Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

No branch id in filepath #49

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

No branch id in filepath #49

wants to merge 2 commits into from

Conversation

maelick
Copy link
Contributor

@maelick maelick commented Dec 9, 2013

The branch id was added to the path of files in DBContentHandler, thus a new file entry is created the first time a file is modified in a new branch and was added in an another one. Theses two commits fixes this and also removes unused local variables. It partially fixes issue #42.

@andygrunwald
Copy link
Contributor

THX!
I will try to get another look at this patch and then we will merge it.

@sduenas
Copy link
Member

sduenas commented Dec 9, 2013

Please don't pull this patch before reading my comment (that I'm writting right now) in #42. Thanks

@andygrunwald
Copy link
Contributor

@sduenas with "another look" i mean another person like you who got more knowledge about this part :)
I didn`t want to merge this patch before another person had a look.

@sduenas
Copy link
Member

sduenas commented Dec 9, 2013

@andygrunwald Hehehe, I didn't get that :-)

My comments here #42 (comment)

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

Successfully merging this pull request may close these issues.

3 participants