The attached field formatter patch requires running DB updates.

CommentFileSizeAuthor
#7 mason-field-formatter.patch17.87 KBvadimk
#2 mason-field-formatter.patch20.68 KBvadimk
mason-field-formatter.patch20.75 KBvadimk
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

madalien created an issue. See original summary.

vadimk’s picture

vadimk CreditAttribution: vadimk commented
FileSize
mason-field-formatter.patch20.68 KB
vadimk’s picture

vadimk CreditAttribution: vadimk as a volunteer commented
gausarts’s picture

gausarts CreditAttribution: gausarts commented
Status: Active » Needs review
gausarts’s picture

gausarts CreditAttribution: gausarts commented

Thank you for contribution. Please allow some delay to get back on this.

In the mean time, just a quick review, you can remove any @file docblock from PHP files as per #2665992: @file is not required for classes, interfaces and traits, #2304909: Relax requirement for @file when using OO Class or Interface per file. And may also need to cleanup any Slick related stuffs which are not used for Mason, but that is okay for now.

gausarts’s picture

gausarts CreditAttribution: gausarts commented
Status: Needs review » Needs work

Let's make it a need work.

vadimk’s picture

vadimk CreditAttribution: vadimk as a volunteer commented
FileSize
mason-field-formatter.patch17.87 KB

Removed @file docblock.

Bwolf’s picture

Bwolf CreditAttribution: Bwolf commented

Is this patch going to be added to the mason module in the near future?

Gausarts, thank you for building this module it works great!

gausarts’s picture

gausarts CreditAttribution: gausarts commented
Status: Needs work » Closed (outdated)

Truly sorry for terrible delay. This module simply got no love from me. I was more in love with GridStack.

However due to changes since 2016, we cannot merge the patch. Thank you, anyway!
The relevant field formatters which are more related to Blazy 2.x were just added.