Puphpet: Mendukung properti `charset` di set konfigurasi `mysql.databases`

Dibuat pada 12 Feb 2018  ·  3Komentar  ·  Sumber: puphpet/puphpet

Saat ini, parameter charset ada dalam konfigurasi database MySQL di config.yaml akan diteruskan ke modul MySQL yang sudah didukung .

Saya pribadi merasa fitur tersembunyi ini berguna karena saya sering menggunakan utf8mb4 .

Saya juga telah mengkonfirmasi bahwa menyetel kunci collate ke utf8mb4_general_ci atau utf8mb4_unicode_ci tidak mengubah default charset dari utf8 menjadi utf8mb4 . Kecuali charset secara khusus disetel ke utf8mb4 , menggunakan salah satu dari utf8mb4_* collations akan menyebabkan provisi gagal.

Dari apa yang saya tahu, mendukung sepenuhnya fitur ini hanya memerlukan dua perubahan:

Di src/PuphpetBundle/Resources/config/mysql/defaults.yml , ubah:

databases:
    database1:
        name: dbname
        collate: utf8_general_ci
        sql: ~

Ke:

databases:
    database1:
        name: dbname
        charset: utf8
        collate: utf8_general_ci
        sql: ~

Dan di src/PuphpetBundle/Resources/views/mysql/database.html.twig , tambahkan:

    <div class="form-group col-xs-12 col-sm-6">
        <label for="{{ idBase }}-charset">
            DB Charset
        </label>
        <input type="text" id="{{ idBase }}-charset"
               name="{{ nameBase }}[charset]"
               placeholder="utf8" class="form-control"
               value="{{ database.charset }}" />
    </div>

Apakah ada alasan mengapa ini tidak didukung secara resmi sehingga saya tidak melihatnya? Apakah ada lebih banyak untuk menerapkan dukungan sepenuhnya daripada yang telah saya uraikan? Apakah Anda akan menerima permintaan tarik dengan perubahan ini?

Komentar yang paling membantu

@jtreminio terima kasih atas respon cepatnya! Saya harus punya waktu ~ siang ini ~ dalam dua minggu untuk membuat PR.

Selain menambahkan bidang DB Charset dijelaskan di atas, saya juga akan mengganti nama bidang DB Encoding menjadi DB Collation untuk kejelasan.

Semua 3 komentar

Apakah ada alasan mengapa ini tidak didukung secara resmi sehingga saya tidak melihatnya?

Setiap modul pada puppet memiliki lebih banyak fitur daripada yang terlihat di GUI. Menambahkan semua opsi itu, bagaimanapun, akan dengan cepat membuat GUI luar biasa.

Saya akan menerima PR, terutama untuk hal seperti ini.

@jtreminio terima kasih atas respon cepatnya! Saya harus punya waktu ~ siang ini ~ dalam dua minggu untuk membuat PR.

Selain menambahkan bidang DB Charset dijelaskan di atas, saya juga akan mengganti nama bidang DB Encoding menjadi DB Collation untuk kejelasan.

@jtreminio hanya memastikan Anda telah melihat permintaan tarik saya (#2761). Tolong beri tahu saya jika ada sesuatu yang ingin Anda ubah untuk menerimanya.

Apakah halaman ini membantu?
0 / 5 - 0 peringkat