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

Assign dataset to the same oranization as the harvest source #5

Open
FuhuXia opened this issue Aug 15, 2013 · 1 comment
Open

Assign dataset to the same oranization as the harvest source #5

FuhuXia opened this issue Aug 15, 2013 · 1 comment

Comments

@FuhuXia
Copy link
Contributor

FuhuXia commented Aug 15, 2013

If a harvest source is assigned to an organization, the harvested datasets should be by default assigned to the same organization. I have created a fork and enabled this option in the fork.

Since I am having problem get the current master branch working with our CKAN 2.1 core, as reported in a previous issue, I created a dev branch and made the code change on a last known working commit. Will merge the code and submit a pull request once the previous issue is resolved.

Would highly appreciate it if you can exam the change and offer feedback.

@JoshData
Copy link
Contributor

JoshData commented Sep 8, 2013

Hey, sorry for the delay.

I fixed the whitespace issue, and I also merged your commit assigning datasets to the organization that owns the harvest source (24da0e2).

For the hard-coded "harvest" user, I'm not sure if get_site_user is appropriate. It looks like an undocumented internal function. Maybe we can just make the user name configurable in the ini file?

zr2d2 pushed a commit that referenced this issue Jan 15, 2015
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

No branches or pull requests

2 participants