Basic programming flaws in this plugins

  • Resolved tedinoz

    (@tedinoz)


    13 years, 3 months ago

    I’m running WP 3.5.1 and the plugin version is 1.0.1.

    1 = The plugin creates a folder called “wp-file-permission-check”. However the ‘wp_register’ code on lines 33 and 34 refer to “file-permission-check”.

    2 = The file permissions displayed for files in the wp-admin folder (I haven’t tested wp-content and wp-includes”) are incorrect. This is because the $fileinfo variable which declared for the “root” folder is inadvertently re-used in the wp-admin folder BUT is not reset beforehand.

    The objectives for this plugin seemed worthwhile but it simply does NOT work as advertised. I suggest that you don’t use this plugin until it has had a major revision.

    Ted

    http://ww.wp.xz.cn/extend/plugins/wp-file-permission-check/

Viewing 4 replies - 1 through 4 (of 4 total)
  • Plugin Author wycks

    (@wycks)

    13 years, 3 months ago

    The plugin was abandoned because jquery.ui and jquery.tools have hardcoded “tabs” in core, which caused conflicts for output, I was going to rewrite it and fix the above as well, but never got around to it, maybe one of these days.

    see: http://ww.wp.xz.cn/support/topic/plugin-file-permissions-size-check-javascript-error?replies=4#post-3956083

    Thread Starter tedinoz

    (@tedinoz)

    13 years, 3 months ago

    Hi wycks
    If you’re interested, I’ve edited the code so that it works without jquery.

    I’ve also made a small change to the css so that file permissions of 0777 are more prominent.

    I’m only a PHP learner and the Iterators are new to me; so the format of file names on the wp-admin, wp-content and wp-includes tabs might be better done. I’d also like to see things sorted by file name, but I figured that it was more important to get the basics working than spend time on that. i.e. someone else can worry about that <<grin>>.

    The code is here: http://pastebin.com/skWzLR2y
    The css is here: http://pastebin.com/2ednma4d
    Again, pastebin is new to me so hopefully this works ok.

    Ted

    Plugin Author wycks

    (@wycks)

    13 years, 3 months ago

    I completely re-wrote the whole plugin from scratch because it was poorly done.

    1. It now uses the build in Jquery UI.
    2. It now shows modified time and does better iteration
    3. It ignore more files and some directories (cached)

    I tested this on several installs of 3.5.1 so if you run into any problems with new version please let me know via github because I don’t check here as often: https://github.com/wycks/WP-File-Permission-Check

    Thread Starter tedinoz

    (@tedinoz)

    13 years, 3 months ago

    The new version runs very nicely.
    All problems fixed.
    Recommended.

Viewing 4 replies - 1 through 4 (of 4 total)

The topic ‘Basic programming flaws in this plugins’ is closed to new replies.