Added multitag support

[sarjona]

By default the code of this module supports more than one tag separated by comma (bpd_tag=tag1, tag2), but it's not possible to select more than one from the filtering page. This patch lets select more than one tags when filtering:

• Reviewed styles to show selected tags.
• Fixed bug with bp-docs to let AND filter. By default filtering is done with OR clausure.
• Added option to select more than one tag from filtering page.
• Added link to let users choose if the filter has to include all the tags (AND) or only some of them (OR). For this part, at the moment, it's necessary to add the following in the .po file:

msgid "<strong><a href=\"%s\" title=\"View Docs with some of the selected tags\">View Docs with some of the selected tags</a></strong>"
msgstr ""

msgid "<strong><a href=\"%s\" title=\"View Docs with all selected tags\">View Docs with all selected tags</a></strong>"
msgstr ""

[sarjona]

[boonebgorges]

Thank you so much for the pull request! I love the functionality.

A couple questions. First, there are some merge conflicts that are arising, primarily because you are working off of master rather than the latest release branch. (Sorry for not having better documentation of this.) If it's not too much trouble, could you regenerate the PR using the 2.0.x branch? I think the following should do it:

$ git checkout WIP-multitag
$ git checkout -b WIP-multitag-backup
$ git branch -D WIP-multitag
$ git checkout -b WIP-multitag upstream/2.0.x # replace "upstream" with whatever you've called my repo
$ git cherry-pick 5db0b56
$ git push --force origin WIP-multitag

Github should recognize that you're using the same branch name, and update this PR. (If you can't do all of this, we can continue to work off of the current PR.)

On the substance of the PR:

• I think the text "You are viewing docs with the following tags" should be modified according to the current boolean. So the default view would be "You are viewing docs with one or more of the following tags", and when the bool=and, it'd be something like "You are viewing docs with each of the following tags". So it'd work kinda like your contextual link "View docs with all selected tags", which I really like.
• If you don't mind, could you modify your spacing to match WP formatting standards? https://make.wordpress.org/core/handbook/best-practices/coding-standards/php/#space-usage

Thanks!

[sarjona]

Hi! It's great that you like this functionality! We have recreated the WIP-multitag branch from the last content of 2.0.x branch, we have adapted the strings with your proposal and we have modified the spacing mistakes. So, the WIP-multitag now has to be correct... Please, tell us if you can merge from this PR or if you need we make a new one. Kind regards!