PontusHorn / Pico-Search

Simple search feature for pages in your Pico CMS
MIT License
22 stars 4 forks source link

Fixed errors in php 7.2 #5

Closed azjezz closed 6 years ago

azjezz commented 6 years ago

Pico search trigger some notices ( in php 7.2.3 ) , which are : 1 - Notice: Undefined index: q in .... 2 - Notice: Undefined variable $folder .... this PR should fix both of these errors.

PontusHorn commented 6 years ago

Sorry, what I meant by giving your commits more descriptive names is that you can clean up the git history by rebasing the branch. Here's a good guide to that if you're not familiar with the process: https://github.com/mockito/mockito/wiki/Using-git-to-prepare-your-PR-to-have-a-clean-history

Let me know if that's unclear or if there's another way I can help :)