# |
Dec 19th 2017, 16:23 |
neon1024 |
Strange as when you dump it directly in the App::className method it’s getting the right class |
# |
Dec 19th 2017, 16:15 |
neon1024 |
File uploaded https://cakesf.slack.com/files/U1BT622HW/F8GUG77B6/-.txt / https://slack-files.com/T053DPNCM-F8GUG77B6-a2cb1ca240 |
# |
Dec 19th 2017, 16:15 |
neon1024 |
As the App:className actually just returns the same string as input |
# |
Dec 19th 2017, 16:15 |
neon1024 |
I think you’re right, the issue seems to actually be much further up the code chain here, https://github.com/dereuromark/cakephp-ide-helper/blob/master/src/Annotator/ModelAnnotator.php#L118 |
# |
Dec 19th 2017, 16:12 |
neon1024 |
Yep |
# |
Dec 19th 2017, 16:11 |
dereuromark |
you are working on a mac, right? |
# |
Dec 19th 2017, 16:10 |
dereuromark |
using App:: class |
# |
Dec 19th 2017, 16:10 |
dereuromark |
the annotation is fqcn, which is resolved from the dot syntax |
# |
Dec 19th 2017, 16:10 |
neon1024 |
So I must have the code flow backwards in my brain |
# |
Dec 19th 2017, 16:10 |
neon1024 |
But the annotation doesn’t use a dot |
# |
Dec 19th 2017, 16:09 |
neon1024 |
But then you concat with a dot for some reason, https://github.com/dereuromark/cakephp-ide-helper/blob/master/src/Annotator/ModelAnnotator.php#L249 |
# |
Dec 19th 2017, 16:09 |
dereuromark |
jep |
# |
Dec 19th 2017, 16:09 |
neon1024 |
Like this? https://regex101.com/r/algUh1/1 |
# |
Dec 19th 2017, 16:09 |
neon1024 |
You meant here right when you mentioned regex? https://github.com/dereuromark/cakephp-ide-helper/blob/master/src/Annotator/ModelAnnotator.php#L271 |
# |
Dec 19th 2017, 16:08 |
dereuromark |
what? |
# |
Dec 19th 2017, 16:08 |
neon1024 |
Man this hurts my brain |
# |
Dec 19th 2017, 15:56 |
neon1024 |
I’ve managed to get a failing test finally, so just working on a fix |
# |
Dec 19th 2017, 15:56 |
neon1024 |
Yeps, it’s in its own repo as per the path above :slightly_smiling_face: |
# |
Dec 19th 2017, 15:55 |
dereuromark |
Are you developing it inside your app? best to do that standalone, and add classes there with custom autoloader picking them up prbably |
# |
Dec 19th 2017, 15:51 |
hmic |
welcome |
# |
Dec 19th 2017, 15:51 |
hmic |
;-) |
# |
Dec 19th 2017, 15:50 |
neon1024 |
Good job hmic! Thanks dude, spot on as always |
# |
Dec 19th 2017, 15:49 |
neon1024 |
Yes, good point. Let me try that! |
# |
Dec 19th 2017, 15:48 |
hmic |
as it's not a vendor plugin, you need the autolaoder setup in composer.json to recognise it |
# |
Dec 19th 2017, 15:48 |
hmic |
how's that setup in your composer.json |
# |
Dec 19th 2017, 15:48 |
neon1024 |
There is no composer so I’m a bit confused, https://github.com/dereuromark/cakephp-ide-helper/tree/master/tests |
# |
Dec 19th 2017, 15:48 |
hmic |
i'd say so too neon1024 |
# |
Dec 19th 2017, 15:47 |
neon1024 |
Which is why I can only think it’s an autoloader issue, but the test bootstrap loads the plugin okay it seems |
# |
Dec 19th 2017, 15:47 |
neon1024 |
string(72) “/Users/david/Sites/CakePHP-IDEHelper/tests/test_app/plugins/Example/Slug” |
# |
Dec 19th 2017, 15:47 |
hmic |
but you can add whatever you like in there. you could add the whole partner array, if you like to |
# |
Dec 19th 2017, 15:47 |
neon1024 |
@hmic Well the class exists at that namespace, the Plugin class see’s the plugin as loaded, and has the correct path too |
# |
Dec 19th 2017, 15:46 |
hmic |
yes, "partner" is an associated record, there is data in there, just not the neccessary participant_type_id which i add |
# |
Dec 19th 2017, 15:46 |
joop |
but will $data['partner'] will exist before submit?? |
# |
Dec 19th 2017, 15:45 |
obinoob |
neon1024 thanks |
# |
Dec 19th 2017, 15:45 |
hmic |
and this namespace is setup correctly? |
# |
Dec 19th 2017, 15:45 |
joop |
i think you answered my question now |
# |
Dec 19th 2017, 15:45 |
neon1024 |
@hmic string(45) “Example\\Slug\\Model\\Behavior\\SluggableBehavior” |
# |
Dec 19th 2017, 15:44 |
hmic |
neon1024: whats in your classname variable? |
# |
Dec 19th 2017, 15:44 |
joop |
i see now your concept,load the $data[....][....] with calculated data(or wahtever you call now) |
# |
Dec 19th 2017, 15:44 |
neon1024 |
You have to build it yourself if you want the complete url |
# |
Dec 19th 2017, 15:44 |
neon1024 |
Nope |