Problem/Motivation

The view plugins provided in this module have views schema. By replacing the plugin id, the views have the additional schema(s) but timestamps do not. (views_date_format_sql_views_data_alter)

Steps to reproduce

Install config inspector and views_date_format_sql. Update a view with a timestamp. Views the status report.

Proposed resolution

Instead of replacing the plugin id, create new fields/arguments.

Remaining tasks

Make a release with new fields and the override fields; include a status message to convert the affected views

Make a release only with new fields.

User interface changes

Users will have to add the views_date_format_sql field/argument to a view.

API changes

Data model changes

Users will need to update their views to use the new field/argument.

Issue fork views_date_format_sql-3489373

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

bluegeek9 created an issue. See original summary.

bluegeek9’s picture

bluegeek9 commented
Issue summary: View changes
bluegeek9’s picture

bluegeek9 commented
Assigned: Unassigned » bluegeek9

bluegeek9 opened merge request !15

eugene bocharov’s picture

eugene bocharov commented

Though 'title' is usually translatable markup, but it still can be just a string in contrib or custom module. Probably more safe would be

      case 'title':
        $new[$key] = t('@value SQL date', ['@value' => (string) $value]);
        break;

instead of

      case 'title':
        $new[$key] = t('@value SQL date', ['@value' => $value->__toString()]);
        break;