-
Notifications
You must be signed in to change notification settings - Fork 40
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 #23
Open
k0a1a
wants to merge
72
commits into
master
Choose a base branch
from
dev
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
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
This helps when all you want is to select an object.
Should be a bit more reliable now.
This forces it out of editing mode, making the state of the newly cloned object more consistent.
On Chrome, changing the textarea's content does not seem to automatically update its innerHTML. This does not seem to be an issue for repeatedly editing a single object (as val() stays the same), but for cloning we need to have the innerHTML set.
added 'ctrl-z' shortcut to suggest using revisions inplace for undo
minified modified .js files, reverted changes in config.inc.php NOTE: 'stylesheet' module is still present regardless the planned transition to 'code' module.
…age is not resized with GD.
users don't need to care about removing those when updating.
removed (cheap webhosting compatibility) empty directory was not pulled in
updated WORM url
Content-Disposition now encloses filaname in double quotes. This is coherent with RFC 2616 19.5.1. http://www.w3.org/Protocols/rfc2616/rfc2616-sec19.html#sec19.5.1
When downloading a file with a space, the Content-Disposition header is malformed
Currently, the code isn't compliant with PHP strict standards, and will raise on PHP 5.5 strict issues. Meanwhile it's fixed, we can temporarily discard these errors.
Temporary disabled E_STRICT
…4 section to htaccess-dist
Fixed base URL construction port typo
In modern servers environment, we frequently put a front-end server to offer a cache (e.g. Varnish), load balancing or allow to isolate application in Docker containers. In such configurations, the responsibility to serve in TLS mode belongs to the front-end. We need to play along and serve https:// URL, not http:// ones in such case. The most used header (Apache, Varnish, nginx) to indicate this state is X-Forwarded-Proto, normalised as the Forwarded HTTP Extension. This is coherent with RFC 7239 5.4. https://tools.ietf.org/html/rfc7239#section-5.4
Allow Hotglue to work behind a load balancer or a reverse proxy
The array_shift PHP function expects an array to manipulate, stored in a variable. As such, it's a poor candidate to get the first element of an array, as it will raise a strict exception if we pass it a reference to an array instead. We supersede array_shift by an universal function to get the first element of any array or object which implements the Traversable interface, kicking the iterator to pick th first element. Fixes #9.
Get rid of array_shift
adding docker files, thanks @albjeremias
Improve code documentation spelling
get_magic_quotes_gpc() is not supported in php8, so stripslashes are always executed
Update json.php for php8 support
Support for php8
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
merging dev to master