-
Notifications
You must be signed in to change notification settings - Fork 8
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
[91] - switching the patterns in the MediaInternetGovCmsCkanHandler->… #93
[91] - switching the patterns in the MediaInternetGovCmsCkanHandler->… #93
Conversation
Merge latest develop branch for 1.1 release.
7.2 Release for govCMS CKAN. - Fixes govCMS#74 Adds support for non-standard CKAN ports - Fixes govCMS#76 for formatting improvements - Adds defensive coding practises to prevent errors - Updates max lenght on URL field - Adds record limit configuration by variables - Adds cache control per graph - Addresses IE11 download issue
…parse function to use baseUrl instead of baseHost
Requested reviews from some of the primary contributors. |
'@' . $this->baseHost . '/dataset/([^"\&\? ]+)/resource/([^"\&\? ]+)/download/(?:[^"\&\? ]+)@i', | ||
'@' . $this->baseHost . '/dataset/([^"\&\? ]+)/resource/([^"\&\? ]+)@i', | ||
'@' . $this->baseUrl . '/dataset/([^"\&\? ]+)/resource/([^"\&\? ]+)/download/(?:[^"\&\? ]+)@i', | ||
'@' . $this->baseUrl . '/dataset/([^"\&\? ]+)/resource/([^"\&\? ]+)@i', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That one was changed to support ckan endpoints using different ports, could we instead change where baseUrl is set to also include the path if it exists?
…dded to the baseURL will still work as requested.
@seanhamlin , @jeremy-doghouse , I've made a small update to include the baseHost and a basePath. |
Approved pending changing the base branch to develop. @SRowlands or @steveworley can you please have a look at this too. |
@tobybellwood @steveworley @SRowlands , any idea on a timeline for getting this into govCMS? |
#91