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

Custom search #16

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

dpaulat
Copy link
Contributor

@dpaulat dpaulat commented Dec 31, 2017

I'd like to have a few additional hooks added to the cpg1.6.x base to allow for custom search capabilities outlined in the following forum post:
http://forum.coppermine-gallery.net/index.php/topic,79260.0.html

The following capabilities are introduced:

  • Modify $allowed array (search fields)
  • Join additional tables to search query
  • Modify WHERE clause of search query

This supports the plugin I'm developing:
https://github.com/dpaulat/cpg-plugin-extensible-metadata

@dpaulat dpaulat changed the title Dev/custom search Custom search Dec 31, 2017
@ron4mac
Copy link
Collaborator

ron4mac commented Jan 1, 2018

Nice work, Dan. I want to take a bit of time to look at this (as soon as I can)

@dpaulat
Copy link
Contributor Author

dpaulat commented Jan 1, 2018 via email

@ron4mac
Copy link
Collaborator

ron4mac commented Jan 1, 2018

@dpaulat
In looking at this, I wanted to see how it works with your plugin. I had some problems with it because you have a couple of hard coded database prefixes (cpg16x). Fixed that but your javascript will not run with the current CPG jQuery default (1.4.2). I'll begin another effort with the dev team to move that forward (it's way past time).

@dpaulat
Copy link
Contributor Author

dpaulat commented Jan 1, 2018

@ron4mac
I've made these updates, as well as some general cleanup and readme updates to the develop branch. The readme should be a little clearer on installation dependencies and usage. Let me know if you have any further troubles.

@dpaulat dpaulat force-pushed the dev/custom_search branch from 7757dcd to 27f847f Compare January 1, 2018 18:36
@dpaulat dpaulat force-pushed the dev/custom_search branch from 27f847f to bfbbfb9 Compare May 25, 2022 03:14
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

Successfully merging this pull request may close these issues.

2 participants