Tickets :: Comment on [#8079] Avoid notice in _removeBr from Text

* Your Email Address
* Spam protection	Enter the letters below: .__ .___..___ __..__ [ __ \| [__ (__ [__) [_./ \| \| .__)\| \
Comment	>> How about my patch, which removes for() loops entirely? > > > > Looks also fine to me. > > > > Instead of: > > foreach (array_keys($lines) as $i) { > > You could use: > > foreach ($lines as $i => $line) { > > if (!preg_match("/^\s<br\s\/>\s*$/i", $line)) { > > ... > > > > To eliminate the extra array_keys() call. > > But at the end its the same. > > Your new approach is _technically seen_ maybe a little more > expensive, but it does indeed remove the "unnice" for() loop. > > > > I am fine either. > > > > - Jens > >
Attachment
Watch this ticket