We use cookies to improve your experience. No personal information is gathered and we don't serve ads. Cookies Policy.

ExpressionEngine Logo ExpressionEngine
Features Pricing Support Find A Developer
Partners Upgrades
Blog Add-Ons Learn
Docs Forums University
Log In or Sign Up
Log In Sign Up
ExpressionEngine Logo
Features Pro new Support Find A Developer
Partners Upgrades
Blog Add-Ons Learn
Docs Forums University Blog
  • Home
  • Forums

Grid Backspace Fix?

Development and Programming

Jonathan Kelly's avatar
Jonathan Kelly
2 posts
12 years ago
Jonathan Kelly's avatar Jonathan Kelly

I was experiencing an issue with the backspace=”n” param not being respected when iterating through the tagdata of a grid fieldtype.

I’ve resolved the issue by doing a substr() at the end of the Grid_parser’s parse() method.

// hack: backspace param fix
if(isset($params['backspace']) && $params['backspace'])
 $grid_tagdata = substr($grid_tagdata, 0, (strlen($grid_tagdata)-$params['backspace']));

More than anything, this post is meant to raise awareness, if there is in fact a bug here.

       
bceglowski's avatar
bceglowski
1 posts
12 years ago
bceglowski's avatar bceglowski

Salient. Added a bugfix request. Thanks.

       
hammerquist's avatar
hammerquist
1 posts
12 years ago
hammerquist's avatar hammerquist

Hey Kelly,

Thanks for the post. Can you help me point to where I can add that quick fix?

       
Dotwebs's avatar
Dotwebs
13 posts
12 years ago
Dotwebs's avatar Dotwebs

Where does this fix go?

Is it added to the template page? If so, where?

Backspace is an important feature, without it comma separated lists look ridiculous.

       
Dotwebs's avatar
Dotwebs
13 posts
12 years ago
Dotwebs's avatar Dotwebs

Found it here: https://support.ellislab.com/bugs/detail/20030

Thanks

       

Reply

Sign In To Reply

ExpressionEngine Home Features Pro Contact Version Support
Learn Docs University Forums
Resources Support Add-Ons Partners Blog
Privacy Terms Trademark Use License

Packet Tide owns and develops ExpressionEngine. © Packet Tide, All Rights Reserved.