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

Fix error on mysql-server-5.7.7-rc #28

Merged
merged 7 commits into from
May 29, 2015
Merged

Fix error on mysql-server-5.7.7-rc #28

merged 7 commits into from
May 29, 2015

Conversation

vaindil
Copy link
Collaborator

@vaindil vaindil commented May 12, 2015

The RC of mysql-server-5.7.7 doesn't like the format of the input time when creating new secrets. Using DATE_ADD() and a little syntax editing corrects the problem. I've verified this on my live server (mysql-server-5.7.7-rc, Ubuntu Server 14.04) and on a fresh test server (mysql-server-5.5, Ubuntu Server 14.04); the changes work properly on each.

The specific error is below, where 300 is replaced with the correct number of minutes based on the input.

SQLSTATE[22007]: Invalid datetime format: 1292 Truncated incorrect INTEGER value: '+300 minutes'

vaindil and others added 4 commits May 11, 2015 23:01
Updated jQuery to v1.11.3, Bootstrap to v3.3.4. interface.php has been
updated to accommodate the changes in Bootstrap on the home page.
Further work is necessary to update the remaining portions of the file.
Readme has been updated with a demo site. Remaining files are edited as
a result of a minor reformatting in Phpstorm.
@vaindil
Copy link
Collaborator Author

vaindil commented May 18, 2015

Just pushed a big commit that starts the process of updating jQuery to v1.11.3, Bootstrap to v3.3.4, and editing (primarily) interface.php to address the new versions. I've also added Bootstrap tooltips to the images next to each field to address #19.

@bemosior
Copy link
Owner

👍 Thanks for taking the time. As soon as I have time, I'll test, verify, and merge.

vaindil added 3 commits May 18, 2015 12:07
updated license link in all files to HTTPS instead of non-SSL link
-completed formatting interface.php and pw.php for updated Bootstrap
-formatted about.php for updated Bootstrap
-updated ZeroClipboard to v2.2.0
-updated readme with new test server (same server, just has a domain
name now)
-converted line endings to LF (some were CRLF previously)
-removed Placeholder library as it is not being used
-verified all pages with W3C HTML Validator for HTML5
bemosior added a commit that referenced this pull request May 29, 2015
@bemosior bemosior merged commit f001429 into bemosior:master May 29, 2015
@bemosior
Copy link
Owner

Thanks again for all the changes! I do need to revisit this project to clean it up, as I'm generally not happy with the design.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants