Found small problems and made an enhancement
-
Thanks for making such a useful plugin! It really works how I want it to work ??
Few problems I noticed:
- There is a risk of SQL injects in some functions. For example, parameter
page
sometimes enters directly into queries but can be manipulated by the user. Should be fixed. - Changing the CSS class of the widget to
widget_pages
fromwidget_pages_plus
helps with plugin integration, because (e.g. 2015) have styles for the native pages widget - It would be useful to have CSS classes for whether a page is currently active (so it can be highlighted) and whether it is parent or child (for theming)
My changes to the code that solve these three issues should be available for the next month on this pastebin: https://pastebin.com/PAav0pSJ
If you could find them useful and consider incorporating into your plugin, I would be happy. And once again, thanks for the really nice plugin!
- There is a risk of SQL injects in some functions. For example, parameter
Viewing 2 replies - 1 through 2 (of 2 total)
Viewing 2 replies - 1 through 2 (of 2 total)
- The topic ‘Found small problems and made an enhancement’ is closed to new replies.