Hi,

No need to tell, great module :)

Sometimes we generate a lot of content while developing a website. The title fields are generated as loremipsum text. So when we look at all the content we generated, they all seem the same. No way to tell from what content type its comming.

Maby an option to add a fix text to the title like "event - loremipsumtext" or "product - loremispum" so we can tell what is what when oversee all the content.

I set the priority to minor. But sometimes it can be a lifesaver. Mostly on entity reference views development with referencing something to something.

What do you think?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

RgnYLDZ created an issue. See original summary.

bapi_22’s picture

Content type added before the content title.

RgnYLDZ’s picture

Works like a charm :)

Thank you very much

bapi_22’s picture

Assigned: Unassigned » bapi_22
salvis’s picture

Version: 7.x-1.5 » 8.x-1.x-dev
Assigned: bapi_22 » Unassigned
Priority: Minor » Normal
Status: Needs review » Needs work

Makes sense to me, but
-- patches must go against the -dev version, and
-- D8 needs to go first.

Feel free to reassign yourself again if you intend to post a D8 patch.

bapi_22’s picture

I have uploaded the patch for drupal 8.x-1.x-dev.

Please review.

Status: Needs review » Needs work

The last submitted patch, 6: fix-text-on-title-issue-2714659-d8.patch, failed testing.

The last submitted patch, 6: fix-text-on-title-issue-2714659-d8.patch, failed testing.

bapi_22’s picture

Hi Salvis,

I have uploaded the patch for Drupal 8.x-1.x-dev for the same issue .

Please review.

bapi_22’s picture

Status: Needs work » Needs review
moshe weitzman’s picture

Status: Needs review » Needs work
Issue tags: +Needs reroll

Looks fine, but needs reroll (not surprising - sorry for the long delay)

Neslee Canil Pinto’s picture

Updated Patch

rpayanm’s picture

Issue tags: -Needs reroll

  • salvis committed fff3d8c on 8.x-3.x
    Issue #2714659 by bapi_22, Neslee Canil Pinto, salvis: Fix text on title...

  • salvis committed f41ad67 on 8.x-2.x
    Issue #2714659 by bapi_22, Neslee Canil Pinto, salvis: Fix text on title...
salvis’s picture

Version: 8.x-1.x-dev » 8.x-3.x-dev
Assigned: bapi_22 » Unassigned
Status: Needs review » Fixed

Just a tiny nit: This has gone back and forth in various versions of this patch — I'm changing #12's period

  /**
   * Return the same array passed as parameter.
   * but with an array of uids for the key 'users'.
   */

to a comma:

  /**
   * Return the same array passed as parameter,
   * but with an array of uids for the key 'users'.
   */

Also, changing double quotes

      'title' => $node_type . "_" . $this->getRandom()->sentences(mt_rand(1, $results['title_length']), TRUE),

to single quotes:

      'title' => $node_type . '_' . $this->getRandom()->sentences(mt_rand(1, $results['title_length']), TRUE),

Pushed based on #11, thank you bapi_22 and Neslee Canil Pinto!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.