WPIntell

Source evidence

Limiter Display character count bug

Advanced Custom Fields: Limiter Field · support · 2013-11-18T14:56:00+00:00

mixedsentiment
mediumseverity
0.84relevance
2replies
Evidence linked to opportunitycommercial context

Proof Health

Open evidence

Commercial opportunities need traceable source links before they are treated as build-worthy.

5 / 31 rows with source links

16.1% of this page's analysis has direct source links.

0 build-decision rows missing links

0 rows here require auditable proof before promotion.

26 rows with no attached evidence

0 rows have source counts but still need direct links.

Conversation

support
Pete resolved
Hi Dave, I’m using this version of your plugin which supports WYSIWYG Limiting. https://github.com/AtomicSmash/ACF-limiter-field/tree/wysiwyg This bug is when a Limited field is set, that’s the standard limiter on a textarea. When you choose Yes for the option Display character count. The count is not displayed in the progress bar. Only when I choose No it is displayed. I think the error is in limiter-4.php line 141 if(isset($field['displayCount'])){ if($field['displayCount'] == 0){ echo('<div class="counterWrapper"><span class="limiterCount"></span> / <span class="limiterTotal">'.$field['character_number'].'</span></div>'); }; } Shouldn’t the second if statement be if($field['displayCount'] == 1){ True / 1? http://wordpress.org/plugins/advanced-custom-fields-limiter-field/ Hi Pete, thanks for this! you are 100% correct! I’ll make adjustments to the WordPress and Github version ASAP. Thanks again! You’re welcome, if I knew how to use Git trac/svn better i’d have made the change and requested the merge.

Comments

2 shown
Atomicsmash 2013-11-18T15:18:00+00:00

Hi Pete, thanks for this! you are 100% correct! I’ll make adjustments to the WordPress and Github version ASAP. Thanks again!

Pete 2013-11-18T15:19:00+00:00

You’re welcome, if I knew how to use Git trac/svn better i’d have made the change and requested the merge.