Updating MTV office - #1
Closed
pdehaan wants to merge 1 commit into
Closed
Conversation
|
Thanks for the contribution, but this is a read-only mirror of git.mozilla.org/webtools/bmo/bugzilla. Please open a bug and attach a review as per standard process. https://wiki.mozilla.org/Bugzilla:Submitting_Patches Sorry for the inconvenience! |
globau
added a commit
that referenced
this pull request
Feb 3, 2015
…ment #1 appears if using description + newest to oldest comment sorting
djmitche
pushed a commit
to djmitche/webtools-bmo-bugzilla
that referenced
this pull request
Oct 29, 2015
Initial framework
djmitche
pushed a commit
to djmitche/webtools-bmo-bugzilla
that referenced
this pull request
Oct 29, 2015
Bug 1194767 - use slugid.nice() instead of slugid.v4()
moz-v2v-gh
pushed a commit
that referenced
this pull request
Nov 20, 2015
This reverts commit 9e1efb5.
michaelvanstraten
pushed a commit
to michaelvanstraten/bmo
that referenced
this pull request
Nov 28, 2023
* use base to make merging with bmo easier * Revert "Bug 1497042 - Enclose table names in CREATE queries" This reverts commit 62d5637. * refactor Bugzilla::DB to not subclass DBI * use DBIx::Connector to manage database connections * Ensure we always call DBIx::Connector->dbh before any DBI method The code didn't allow a way of doing this without a lot of work. So I had to take the following approach: The 'dbh' attribute is now a method that delegates to DBIx::Connector's dbh method. Per the docs, ->dbh() "Returns the connection's database handle. It will use a an existing handle if there is one, if the process has not been forked or a new thread spawned, and if the database is pingable. Otherwise, it will instantiate, cache, and return a new handle." Then there is the matter of the 'handles' on dbh. I've used Package::Stash to insert proxy methods into the class when it is loaded. * Bug 1328659 - Add support for utf8=utf8mb4 (switches to dynamic/compressed row format, and changes charset to utf8mb4) * add deps * add prepare_cached to the list of delegated methods. This was added in bug 340160 * improve the migration to compressed/dynamic rows, skip views * add missing semicolon * remove pre-utf8mb4 emoji support * fix small issue * add debugging fatal error
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.
No description provided.