I have raised merge request. Lets wait its to be merge and new release.
JaydipJD → made their first commit to this issue’s fork.
I have re-rolled the patch for 2.0.0-alpha3
JaydipJD → created an issue.
Patch is merged and released as part of '8.x-1.8'.
JaydipJD → created an issue.
We are waiting this changes to be release since long. We must have to upgrade the latest release of this module on our site. Hence created patch.
I had a look at the conversation above. It is better to provide widget setting to control the behavior.
I have rerolled the patch #15
I have removed ajax response which return on closing the model.
We are getting success message in a new popup once we save the entity and here contributor require extra effort to close the popup having success message. We are already getting error messages in a same popup hence as per my opinion to show extra popup for success message seems bad UX.
Any different suggestions or solutions are most welcome.
Modified patch to display error messages only on the entity browser popup.
Didn't get any error when i disabled aggregation for js and css.
I am facing the same issue today. I just took fresh D10.1.2 and did local setup. I am getting the same error when i hit drush cr. permission for files directory got changed automatically.
Patch #74 is working fine. Hiding mine.
[error] SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'source_uuid' cannot be null: INSERT INTO "coh_usage" ("source_uuid", "source_type", "requires_uuid", "requires_type") VALUES (:db_insert_placeholder_0, :db_insert_placeholder_1, :db_insert_placeholder_2, :db_insert_placeholder_3); Array
> (
> [:db_insert_placeholder_0] =>
> [:db_insert_placeholder_1] => entity_view_display
> [:db_insert_placeholder_2] => b23bb549-4b29-488f-9bf8-16a183cdac64
> [:db_insert_placeholder_3] => node_type
> )
>
> [error] Update failed: system_post_update_timestamp_formatter
> [warning] Invalid placeholder (severity_level) with string: "%type: @message in %function (line %line of %file)." FormattableMarkup.php:245
> [warning] Invalid placeholder (exception) with string: "%type: @message in %function (line %line of %file)." FormattableMarkup.php:245
[error] Update aborted by: system_post_update_timestamp_formatter
Facing the same issue.
Created patch to use till changes got merge and new version release.
Created patch from Merge request 7(https://git.drupalcode.org/project/footnotes/-/merge_requests/7) to use till latest changes got merge and new version release.
Removed Patch created. No change. Uploaded same patch by mistake.
Re-rolled patch against 3.1.0
I have re-roll the patch for D10.1.1 from MR 3681
Modified patch to add Drupal 10 support.
core_version_requirement: ^8.8 || ^9 modified to core_version_requirement: ^9.5 || ^10
fixed the mistake I made in previous patch.
I have re-roll the patch for 9.5.x from MR 3681 (https://git.drupalcode.org/project/drupal/-/merge_requests/3681)
Full Drupal 10 patch has been created so that others can use this patch till latest release.
I have added Dependency injection and other phpcs warning fixes. Please review it.
JaydipJD → made their first commit to this issue’s fork.
I will review and merge it later.
I have applied PHPCS fixes as well.
Tested OK.
JaydipJD → made their first commit to this issue’s fork.
Thanks @cottser for the contribution. I will review and release this version once work done.
Added Missing parenthesis.
Created patch against 1.x branch
use this patch with 1.x branch like mentioned here: composer require 'drupal/field_zip_file:1.x-dev'
PHPCS warning fixes has been added.
I have added fixes code at https://git.drupalcode.org/project/field_zip_file/-/merge_requests/5 with Drupal 10 Readiness branch. I am marking this issue as a fixed and will changes the status once the code will merge with the 1.x branch.
JaydipJD → created an issue.
Patch against 1.x branch.
On php 8.1, Still causing the same issue. Fixes have been added.
Created patch against 1.0.0-beta2 version.