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

Updating MTV office #1

Closed
wants to merge 1 commit into from
Closed

Conversation

pdehaan
Copy link

@pdehaan pdehaan commented Apr 10, 2014

No description provided.

@markrcote
Copy link
Contributor

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!

@markrcote markrcote closed this Apr 14, 2014
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
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
michaelvanstraten pushed a commit to michaelvanstraten/bmo that referenced this pull request Nov 28, 2023
…-bteam#79)

* 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
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.

2 participants