-
Notifications
You must be signed in to change notification settings - Fork 20
Update esearch and efetch URLs to https #7
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
base: master
Are you sure you want to change the base?
Conversation
Due to upcoming move to https-only by NCBI: http://www.ncbi.nlm.nih.gov/home/develop/https-guidance.shtml
|
Aaaaah maybe this will fix this issue: #9 |
|
Mark, I made the changes to lines 42 and 43 but I am now throwing a new error at line 66: $this->count = (int)$xml->Count. Is there another change needed to ensure that HTTPS query is being sent? Thanks |
|
I have no other issues with the script. What is the exact error? |
|
Notice: Trying to get property of non-object in I'm assuming that means that it is not receiving properly formed XML. Obviously it's strange since we are both running identical code. The formed search string works normally in a browser. Which database are you accessing? Could that be the difference? I am hitting 'pubmed' specifically and only doing author queries.
|
|
It could be due to the script wrapping your query in quotes by default, thereby returning 0 results. Have you set exact_match to false? For example:
|
|
I have public $exact_match = false
|
|
@budflanksteak By default author search by someone[au]. Then how to search by title and year? |
Due to upcoming move to https-only by the NCBI, the esearch and efetch URLs require updating to https.