Title: Some constructive feedback
Last modified: August 21, 2016

---

# Some constructive feedback

 *  [Christopher Anderton](https://wordpress.org/support/users/anderton/)
 * (@anderton)
 * [12 years, 6 months ago](https://wordpress.org/support/topic/some-constructive-feedback/)
 * Just tested out some dev plugins for my personal WordPress dev deployment instance.
   While WP Error Fix was not for me (i use the ”safe” bets in the Developer Plugin),
   i did notice a couple fo days later a folder named ”ahmlog” in the wp-content
   directory. There where also no references in that folder about how it got there.
 * The only info is the folder names inside the ahmlog directory named ”_backup”
   and ”_cache” (nothing inside).
 * Searching for ”ahmlog” on Google only gave me no clue. The only thing i did manage
   to find out, was the fact that there was others searching for the same thing (
   you can pick that up on some sites (using Blackhat SEO) that grabs different 
   search criterias and adds them to the content). However they searched for ”ahmlog
   virus” (that was my fear at first. A folder suddenly shows up in wp-content and
   there was no plugin named ahm installed or anything even close to that name”.
 * So, i was a bit confused. The folder was on my localhost with no open ports for
   the outside world. At last i did manage to find a reference to WP Error Fix searching
   my system logs (which i should have did in the first case).
 * So, my suggestions (note that i’m not a plugin developer, and don’t have any 
   insight about code quality as this plugin is a bit different from many others):
 * – The folder in wp-content containing the plugin cache and log files should be
   named something that relates to the plugin name. Ex. “wperrorfix” or “wpefix_data”.
 * – I did notice that the naming of the plugin functions/hooks/filters (just in
   general) is based on the developers name. I would suggest (just a suggestion!)
   changing the naming conventions. Example from the plugin:
 * `ahm_report_cron`
    `AHM_HealthManager`
 * could be
 * `wpefix_report_cron`
    `wpefix_health_manager`
 * With regards to possible required naming conventions for the plugin application.
 * – The plugin file names should use a better file naming convention. Now the main
   plugin file is called ”manager.php”. Also there are 8 files (in different directories)
   named ”bootstrap.php”. My suggesting is to try to follow some naming patterns
   for plugin file names.
 * “manager.php” should be “wp-error-fix.php” and the other files in the sub directories
   should follow a pattern. Example: “bootstrap.php” in /wp-error-fix/application/
   ajax/ to “buffer-output-cleanup.php” and so on.
 * Also. The readme (and the Installation tab on WordPress.org) should be updated
   as the plugin name states _“wp-bug-tracker”_ and not_ “wp-error-fix” _(maybe 
   due to a name change?).
 * I know that this is not something you could do in a snap, but it will benefit
   both you and the users in the long run. It’s not naging, but just some constructive
   feedback 🙂
 * Cheers!
 * [http://wordpress.org/plugins/wp-error-fix/](http://wordpress.org/plugins/wp-error-fix/)

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

 *  Plugin Author [vasyl_m](https://wordpress.org/support/users/vasyl_m/)
 * (@vasyl_m)
 * [12 years, 6 months ago](https://wordpress.org/support/topic/some-constructive-feedback/#post-4324161)
 * Hi **Anderton**,
    Appreciate your time and feedback.
 *  The reason it has ahm_ is because this plugin evolved from Advanced Health Manager
   to WP Bug Tracker and finally ended with WP Error Fix.
    I took time to play with
   names 🙂 And yep, definitely there are some leftovers here and there. I’m changing
   this one-by-one and every new release will have this changes.
 *  The reason I’m keeping ahmlog folder is that if I switch to new folder structure
   all logs will gone as well as cache and reports. So I’ll add a readme note to
   make sure that users know where it comes from.
 *  The Cron name and Main class name I’m planing to change shortly. I have development
   plan and it’s a long story.
 *  In WP Error Fix development I put the idea of Router pattern (pattern is the
   way the code and files are organized). You can find more here: [https://github.com/whimba/Router](https://github.com/whimba/Router).
 *  I’m truly appreciate you for your feedback.
    Vasyl
 *  Thread Starter [Christopher Anderton](https://wordpress.org/support/users/anderton/)
 * (@anderton)
 * [12 years, 5 months ago](https://wordpress.org/support/topic/some-constructive-feedback/#post-4324200)
 * Nice to hear Vasyl! It’s always nice to see plugin devs answer that quick! I 
   suspected that it was something like that. And regarding patterns, you probably
   have a lot more knowledge when it comes to Application code (as WP-Error-Fix 
   is an app in form of a WordPress plugin) than i (as the app code is way over 
   my head).
 * Cheers and good luck!

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

The topic ‘Some constructive feedback’ is closed to new replies.

 * ![](https://s.w.org/plugins/geopattern-icon/wp-error-fix_c44675.svg)
 * [CodePinch - WP Error Fix](https://wordpress.org/plugins/wp-error-fix/)
 * [Frequently Asked Questions](https://wordpress.org/plugins/wp-error-fix/#faq)
 * [Support Threads](https://wordpress.org/support/plugin/wp-error-fix/)
 * [Active Topics](https://wordpress.org/support/plugin/wp-error-fix/active/)
 * [Unresolved Topics](https://wordpress.org/support/plugin/wp-error-fix/unresolved/)
 * [Reviews](https://wordpress.org/support/plugin/wp-error-fix/reviews/)

## Tags

 * [error fix](https://wordpress.org/support/topic-tag/error-fix/)

 * 2 replies
 * 2 participants
 * Last reply from: [Christopher Anderton](https://wordpress.org/support/users/anderton/)
 * Last activity: [12 years, 5 months ago](https://wordpress.org/support/topic/some-constructive-feedback/#post-4324200)
 * Status: not a support question