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

Merge 2.2 into 2.x #10184

Closed
wants to merge 39 commits into from
Closed

Merge 2.2 into 2.x #10184

wants to merge 39 commits into from

Conversation

mrclay
Copy link
Member

@mrclay mrclay commented Sep 10, 2016

(I've requested StackOverflow help on this. Please leave this open for them and do not restart the Travis builds)

mrclay and others added 30 commits August 13, 2016 15:05
When system cache is off, the boot sequence was still trying to load it.
This caused no problem as the cache was always empty, but it's a waste
of cycles and could be a source of bugs in the future.
In order to fix `canEdit()` in 1.9, `update()` needed to load a fresh copy of
entity from the DB to check the persisted attributes. In that fix,
Elgg@66eb9e6,
a corner case was checked out of my paranoia that a dev would load an
invisible entity with access control on, but try to delete it with access
off.

I now believe this check was unnecessary (we never did similar checks for
`delete()`). When the original `canEdit()` attributes issue was resolved in
fix(access): updates no longer mistakenly blocked in some scenarios
These uses of the `ElggCrypto` constructor had not been properly updated
when the `SiteSecret` dependency was added.
fix(web_services): create_api_user() and create_user_token() work again
Either sites already added this via a plugin in 2.1 or they didn't want it.

Fixes Elgg#9731
If no plugin private settings were going to be preloaded, we no longer
run an invalid query.

Fixes Elgg#10119
If the supporting view is not defined in the current viewtype, we now
temporarily set the `default` viewtype while using the view. This makes
sure any nested views also use `default` viewtype.

Fixes Elgg#10114
fix(boot): boot cache now respects system cache setting
docs(license): clarifies dual licensing in LICENSE.txt
security(htmlawed): upgrades htmLawed to latest, currently 1.1.22
fix(js): output deprecation messages to admins in browser console only
fix(core): boot no longer throws DB exception in some edge cases
fix(reportedcontent): Reported Content admin widget works again
If `$vars['options']` is given as an array and an element `$option` is
also an array, we no longer emit a notice and properly set the `selected`
attribute based on `$option['text']`.

Fixes Elgg#10154
fix(members): Don't rely on newest members tab set as default tab in pagehandler for members page
fix(views): input/select view can select options more reliably
fix(likes): don't emit notice if a listing's `$vars['list_class']` isn't set
Sets `0` limit on queries for child queries, converts to batches, some
small refactoring for clarity.
chore(bookmarks): Removal of unused 'shares' code and unused language strings from bookmarks plugin
fix(pages): operations keep track of more than 10 child pages
hypeJunction and others added 9 commits September 8, 2016 12:33
fix(views): some functions that use views fallback to default viewtype
fix(discussions): removes site "Discussions" menu item added in 2.2.0
Adds tests to verify that entities with ACCESS_LOGGED_IN and ACCESS_PUBLIC
are treated as accessible with ignored access
chore(tests): test entity accessibility in the notifications flow
fix(plugins): Make activate/deactivate all plugins to work also on Firefox
@mrclay
Copy link
Member Author

mrclay commented Sep 10, 2016

@mrclay
Copy link
Member Author

mrclay commented Sep 10, 2016

@mrclay
Copy link
Member Author

mrclay commented Sep 10, 2016

I think we just ignore that commit check failures.

@hypeJunction
Copy link
Contributor

One thing we could try is remove git clone depth from travis config. Maybe it has something to do with that

@mrclay mrclay closed this Sep 23, 2016
@mrclay mrclay deleted the merge22 branch September 23, 2016 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants