ian-cox / Kirby-Color-Picker

A simple color picker field for the Kirby CMS
MIT License
86 stars 13 forks source link

Install not working #16

Closed Mattrushka closed 7 years ago

Mattrushka commented 7 years ago

I have been using your plugin before which worked 100% fine.
But substituting my running version with the latest update throws an error: include(/PATHTOMYSITE/site/fields/color/color.php): failed to open stream: No such file or directory

Any ideas why?

This is the old version I am using:
color.zip

luismqueral commented 7 years ago

Yep, I just ran into this as well. The Whoops debugger said that the "color" type wasn't installed. @Mattrushka 's version worked. @ian-cox I might test an install of the latest and see if it works on your end! captura de pantalla 2017-07-25 a la s 2 20 39 pm

Targoran commented 7 years ago

+1, current version isn't working with the current Kirby Versions (2.5.4 in my case)

thisislawatts commented 7 years ago

Not sure if anyone else got to the bottom of it but it appears to be tied to folder structure. I've flattened it out a little and this has fixed the issue on Kirby 2.5.4 my end https://github.com/thisislawatts/Kirby-Color-Picker/tree/fix-for-2.5.4

Not sure if this will affect previous versions of Kirby, will test it out on 2.4.x, 2.3.x and 2.2.x.

Could be tied to: Added support for custom field assets #1060

sonn-gamm commented 7 years ago

@thisislawatts trying your fork on kirby 2.5.5 and I still get the 'missing field' error

/panel/app/src/panel/form.php

if(!class_exists($class)) {
  throw new Exception('The ' . $type . ' field is missing. Please add it to your installed fields or remove it from your blueprint');      
}

any idea anyone?

luismqueral commented 7 years ago

@afincato have you tried the version linked in the OP? That's been working for me!

sonn-gamm commented 7 years ago

@luismqueral it works indeed! I missed the link at first, ty.

Targoran commented 7 years ago

@thisislawatts Cool, maybe write a pull request so @ian-cox can update it eventually?

puckey commented 7 years ago

Thanks for the fix, works for me too - perhaps you should make a pull request to this repo?

thisislawatts commented 7 years ago

Glad to hear this is working out for everyone. I will look to formalise a PR over the weekend. 🙏 Happy Friday

luxlogica commented 7 years ago

Pull request #18 should fix this issue!

Mattrushka commented 7 years ago

Awesome, the fix solves the issue — many thanks for fixing!

StefanEndress commented 6 years ago

Hi guys, anyone knows what to do on 2.5.8 > the same error comes back