-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Support multiple plugin paths #14051
Merged
Merged
Changes from 1 commit
Commits
Show all changes
36 commits
Select commit
Hold shift + click to select a range
cfc147e
do not hard code plugins directory
tsteur 5333bf7
remove method that is not needed for now
tsteur fe67c2b
use plugins directory in more places
tsteur b54c966
some work on supporting multiple plugin directories
tsteur 9c9cc94
use more unique name
tsteur 1966259
couple fixes
tsteur 0e2c2d0
and another fix
tsteur 1820b73
sort plugins
tsteur 420b04d
adjust languagesmanager
tsteur cee5130
Merge branch '3.x-dev' into multipluginpaths
tsteur 8e64fa3
adjust more usages
tsteur 0ff06e5
Update Manager.php
tsteur 2f9cb52
adding a plugin to test
tsteur f0cfaa6
more tests
tsteur 8fc08d3
make sure plugin resources can be located in custom directory
tsteur 6abd286
adding more tests
tsteur a724048
rewrite image paths
tsteur f30c6d1
handle more cases
tsteur def1ca4
add tests
tsteur 4ac8246
make sure to load plugin
tsteur 4c3fd77
trying to fix test
tsteur 0280aaa
trying it this way
tsteur a3c832c
load plugin
tsteur b334967
fix ui test?
tsteur aa42da0
testing if tests succeed this way
tsteur c27b86d
another test
tsteur 48e1df3
load custom dir plugin
tsteur a6f4170
load plugin in ui fixture
tsteur 2d64fb4
change the update statement
tsteur 8447bed
remove update script
tsteur 9baa3f9
delete column
tsteur 5a5f04e
fix ui test
tsteur e45e782
make it work for tests
tsteur f5e0b70
fix some tests
tsteur bb347dd
Merge branch '3.x-dev' into multipluginpaths
diosmosis 0e4361c
Fix merge.
diosmosis File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Think you'll have to add it to top of performSetUp()
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.
thank you @diosmosis will try
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.
didn't help... https://travis-ci.org/matomo-org/matomo/jobs/489864454#L1203 will need to see tmrw and otherwise just remove all possible updates from that plugin