Butterflyknife Updates#41
Merged
alexweissman merged 9 commits intouserfrosting:butterflyknifefrom May 29, 2014
lilfade:butterflyknife
Merged
Butterflyknife Updates#41alexweissman merged 9 commits intouserfrosting:butterflyknifefrom lilfade:butterflyknife
alexweissman merged 9 commits intouserfrosting:butterflyknifefrom
lilfade:butterflyknife
Conversation
Butterflyknife
… tag <? in db_functions.php on line 1
… removed permission/groups from loading there moved to groups.php (not working currently).
…ate settings and load them as well.
…ate settings and load them as well.
…lity to enable/disable user from the admin panel. Switched more vars to use the validator class.
Updates
Contributor
Author
There was a problem hiding this comment.
random extra div tag
… as allow site_settings functions to work.
Update Install file
Contributor
Author
There was a problem hiding this comment.
changed this to make it work not sure how this should work though any input?
Member
There was a problem hiding this comment.
For now, we can just set the permissions for that action to always() for user 1. We also need to implement action permits for groups, so we don't have to add a new set of permits for every new user. I was thinking a separate table, and then the checkActionPermissions function could check both a user and a user's groups' permissions. What do you think?
Member
|
Cool, this looks good. I'll try to take a crack at it today if I have time :-D |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Updates to users.php, site_settings.php, other api files as well, fixed a few errors missing variables/globals and a php short tag found in db_functions.php. Transferred some variables to the new validator class as well in the api pages. Permissions on user update not working as of yet.