Item13585: ExtendedWebListPlugin minor fixes

pencil
Priority: Normal
Current State: Closed
Released In: n/a
Target Release: n/a
Applies To: Extension
Component: ExtendedWebListPlugin
Branches: master
Reported By: KennethLavrsen
Waiting For:
Last Change By: KennethLavrsen
ExtendedWebListPlugin minor fixes had a few problems

A regex did not always get the right list of webs

A debug message was not removed

I re-arranged the original Human Tidy. You know. Where a human arrange the code so it is readable and easy to understand. Perltidy is nice for code shared by many because it ensures that changes checked in are only actual code changes - and it is in some cases more tidy that messy programmers.

For plugins mainly maintained by 1 person it is bad. When you improve your plugin and come back 6 months later and the git version is molested, it is very risky that you do not get all the code changes applied. I had one \b that I missed when I tried. That is why I revert to my well proven and easy to read Human Tidy (TM) format

I think I keep a good coding style. It is mainly the 80 char limit that molest the code. And in my view makes the code much harder to read.

My plugin go to 110 chars as the longest and it is not even half of my monitor in width. That 80 char limit comes from a time that is long over.

Please respect that I do not want my plugins machine tidied. I will respect the tidy requirement for the core and default plugins. And I will ask follow the authors wishes if I touch another persons Free To Modify plugin.

-- KennethLavrsen - 30 Jul 2015

 
Topic revision: r1 - 30 Jul 2015, KennethLavrsen - This page was cached on 18 Nov 2017 - 16:34.

The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. See Copyright Statement. Creative Commons License