Page MenuHomePhabricator
Differential D11030

setup warning for mysql with default boolean syntax
ClosedPublic
Actions

Authored by fabe on Dec 22 2014, 2:55 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Sep 16, 4:26 PM
Unknown Object (File)
Sat, Sep 7, 1:15 AM
Unknown Object (File)
Mon, Aug 26, 9:18 PM
Unknown Object (File)
Sat, Aug 24, 7:34 PM
Unknown Object (File)
Sat, Aug 24, 12:20 AM
Unknown Object (File)
Fri, Aug 23, 12:57 AM
Unknown Object (File)
Aug 19 2024, 3:44 AM
Unknown Object (File)
Aug 11 2024, 1:07 PM
View All Files
Subscribers
epriestley

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T6747: Add setup warning for ft_boolean_syntax for installs using MySQL search
Commits
Restricted Diffusion Commit
rP853ce5e92704: setup warning for mysql with default boolean syntax
Summary

Ref: T6747 add a setup warning if mysql is configured with the default boolean syntax

Test Plan

Run setup checks with default mysql config and then add the desired config option, restart and check again.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

fabe updated this revision to Diff 26482.Dec 22 2014, 2:55 PM
fabe retitled this revision from to setup warning for mysql with default boolean syntax.
fabe updated this object.
fabe edited the test plan for this revision. (Show Details)
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptDec 22 2014, 2:55 PM
Herald added a subscriber: epriestley. · View Herald Transcript
fabe added a task: T6747: Add setup warning for ft_boolean_syntax for installs using MySQL search .Dec 22 2014, 2:55 PM
epriestley accepted this revision.Dec 22 2014, 8:56 PM
epriestley added a reviewer: epriestley.

One minor inline, I'll clean that up in the pull. Thanks!

src/applications/config/check/PhabricatorSetupCheckMySQL.php
241

(Leading and trailing whitespace.)

This revision is now accepted and ready to land.Dec 22 2014, 8:56 PM
Closed by commit rP853ce5e92704: setup warning for mysql with default boolean syntax (authored by fabe, committed by epriestley). · Explain WhyDec 22 2014, 8:57 PM
This revision was automatically updated to reflect the committed changes.
epriestley mentioned this in T6747: Add setup warning for ft_boolean_syntax for installs using MySQL search .Dec 23 2014, 12:00 AM
epriestley mentioned this in T10589: Create useable search experience for Maniphest.Mar 15 2016, 1:21 PM

Revision Contents
Changeset List

PathSize
src/
applications/
config/
check/
35 lines

Diff 26488

View Options

src/applications/config/check/PhabricatorSetupCheckMySQL.php

Loading...
Phabricator · Built by Phacility