Search by title is capped to 128 characters limit

patch for 10.0.x

Patch #14 applied successfully on drupal 9.5 and drupal 10.0.x work fine and it looks good to me.
Thank you

I have tested the patch #14 on drupal version 10.1.x. After applying the patch#14 maxlength attribute value changed to 255.
But patch #2 maxlength attribute value is null. Why this changes happened in new patch. Which value is acceptable ?

Questions and comments in #10 have not been addressed
Issue summary update is still needed

Patch #2 still applied to D10.1 so rerolls in #13 and #14 were not needed so removing credit.

I think this core issue should be solved, we're running into this again and again in different situations and modules: 📌 Increase or remove default textfield #maxlength=128 Needs review

Needs issue summary update
See comment #10

Tagging this for DrupalCon Pittsburgh 2023.

Please save for a mentee at the mentored workshop at DrupalCon Pittsburgh 2023.

Migrating Pittsburgh 2023 to Pittsburgh2023 tag for cleanup

Looking at this issue, it seems like the following specific tasks will help get it to RTBC:

1. rewrite of the issue summary using the standard template. Also, generalizing the summary since this is about a search input but the character limit affects views generally.

2. agreeing on the patch that sets maxlength to 255. This seems to be a responsible setting (as opposed to null) and matches the aforementioned title character limit.

3. Check to see what tests are available for this module and if tests are adequate or need to be updated.

These notes are mostly for #novice #contribution #DrupalConPittsburgh2023

The parent issue has proposed a fix that would address this issue "further up the chain" from Views. However, that fix is against Drupal 11.x, so we do need a fix applied for Drupal 9.5 (as this will be around for a while yet). The parent issue also suggests (option b) to set the maxlength to to 256 instead of 128 and we should probably go with that, providing a fix against Drupal 9.5.x

As suggested in #7 and #10, I have created an updated version of the patch that has comments.

Existing tests for the StringFilter in the Views module all have to do with whether or not use of the exposed filter produces the expected result set. I don't know what sort of assertion one might use to to test this change other than adding a very simple test that checks to see if the input string longer than 128 characters is truncated. This is easy enough to test manually, so I don't know that it's necessary to add an automated test.

If no one has a differing viewpoint, this could be RTBC.

Setting to needs review to get some other eyes on this.

Thank you for working on this!

Changing the issue to 11.x as that's the current development branch. Fixes and changes will be merged there first and backported.

#28 had a CC failure You can check for build errors by running ./core/scripts/dev/commit-code-check.sh locally.

If this issue is resolved in another issue we should close this one out and move over credit.

I think CC failure is because of cspell plugin in core. Somehow the term 'backported' is not present in the cspell dictionary.txt. Regenerating the cspell dictionary.txt will resolve the issue. Also before starting, ensure that JavaScript dev dependencies are up-to-date.

cd core
yarn install

To regenerate cspell dictionary.txt you can use the following command in terminal:

cd core
yarn spellcheck:make-drupal-dict

Not sure how to fix this in drupal issue queue via patch therefore providing patch with the regenerated dictionary.txt.

Not yet ready for review.

There are still open tags that need to be addressed.

And unless you’re a maintainer shouldn’t assign tickets to yourself. Leaving a comment is just fine

Alright Providing patch for 11.x. Thanks.

Nice to see this move!

+++ b/core/modules/views/src/Plugin/views/filter/StringFilter.php
@@ -277,6 +277,9 @@ protected function valueForm(&$form, FormStateInterface $form_state) {
+        // This could be removed if #3331028 is resolved and backported.

// @todo remove when https://www.drupal.org/project/drupal/issues/3331028 📌 Increase or remove default textfield #maxlength=128 Needs review lands.

That is what we tend to do as wording, not just mention the issue number. This also means we wouldn't need to expand the dictionary and that change can be removed.

Updated patch as per #36.

+++ b/core/modules/views/src/Plugin/views/filter/StringFilter.php
@@ -277,6 +277,9 @@ protected function valueForm(&$form, FormStateInterface $form_state) {
+        // @todo remove when https://www.drupal.org/project/drupal/issues/3331028 lands.

this is over 80 characters but doesn't bother the linting? ¯\_(ツ)_/¯

Test failure seems unrelated

Seems issue summary update and test coverage still missing