There's some nested ternary operator logic in _link_sanitize() that should be simplified.

CommentFileSizeAuthor
#3 link-n3208147-3.patch1.05 KBdamienmckenna
#2 link-n3208147-2.patch1.04 KBdamienmckenna

Comments

DamienMcKenna created an issue. See original summary.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Needs review
StatusFileSize
new link-n3208147-2.patch1.04 KB

This should cover it.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
StatusFileSize
new link-n3208147-3.patch1.05 KB

Duh.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Fixed

Committed.

  • DamienMcKenna committed 093c7b8 on 7.x-1.x 
    Issue #3208147 by DamienMcKenna: Simplify nested ternary logic in...
renatog’s picture

renatog commented

THANK YOU, clear now

Easy to understand

Status: Fixed » Closed (fixed)

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