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

Update with bibfilter: add a new bibtex template #89

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

Conversation

mngu012
Copy link

@mngu012 mngu012 commented Aug 5, 2014

Allow search for publication year, keyword,
Improve the search filters

example:
[bibfilter
file=bib.bib
group=year
group_order=desc
allow=inproceedings,article,book,inbook,incollection,techreport,mastersthesis,phdthesis,unpublished
author=Nguyen|Gimel'farb|Delmas
sortauthors=1
keyword="3D content,Web browsers,World Wide Web,Internet,Online front-ends,Stereo image,3D image,3D reconstruction,Image analysis,Vision,Binocular,Stereopsis,Calibration"
]

Allow search for publication year, keyword,
Improve the search filters

example:
[bibfilter
file=bib.bib
group=year
group_order=desc
allow=inproceedings,article,book,inbook,incollection,techreport,mastersthesis,phdthesis,unpublished
author=Nguyen|Gimel'farb|Delmas
sortauthors=1
keyword="3D content,Web browsers,World Wide Web,Internet,Online front-ends,Stereo image,3D image,3D reconstruction,Image analysis,Vision,Binocular,Stereopsis,Calibration"
]
@@ -125,7 +161,7 @@ class Papercite {
function getCached($url, $timeout = 3600) {
// check if cached file exists
$name = strtolower(preg_replace("@[/:]@","_",$url));
$dir = plugins_dir_path(__FILE__) . "/papercite/cache";
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bpiwowar
Copy link
Owner

bpiwowar commented Aug 5, 2014

There are several problems with the pull request, but I guess most of them is because you modified old code (in the repository, there are changes that are not yet in the papercite releases). You should do the following :

  1. Download the original plugin you modified from the wordpress plugin page
  2. Create a diff with your changes
  3. Apply the patch on the git repository and then submit the pull request again

thanks - if you need help, please email.

@bpiwowar
Copy link
Owner

Hi, I will look at this soon, thanks.

@bpiwowar bpiwowar added this to the 0.5.x milestone May 5, 2015
@bpiwowar bpiwowar self-assigned this May 5, 2015
@bpiwowar bpiwowar modified the milestone: 0.5.x Aug 25, 2015
@bpiwowar
Copy link
Owner

See pull request #101 comment - but I keep the idea of more filtering, and will merge some part of your code.

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.

2 participants