search

TYPO3-Caretaker / caretaker_instance

TYPO3 extension caretaker_instance
GNU General Public License v2.0
14 stars 23 forks source link

Update caretaker_instance to work with Typo3 7.xx #29

Closed TrueType closed 9 years ago

TrueType commented 9 years ago

As of now it does not work. I get:

Fatal error: Call to undefined method
TYPO3\CMS\Frontend\Utility\EidUtility::connectDB()
in typo3conf/ext/caretaker_instance/eid/eid.tx_caretakerinstance.php on line 51
Typo3 7.0.2

Is there any planing to have - at least the client - run with later Typo3 versions?

etobi commented 9 years ago

Did you try the 7compatible branch?

TrueType commented 9 years ago

Nope. I did actually check for branches. But somehow I must have missed recognizing "7compatible" as what I was looking for. I will try it right away. Thx.

etobi commented 9 years ago

Would be great if you could test it and give me feedback, if it's working for you.

TrueType commented 9 years ago

Hi Tobi, I installed the lastest 7compatible sources. I run "insecure extensions" and "extensions updates" from a 6.x caretaker server. I got error like

Fatal error:  Class 'tx_caretakerinstance_ServiceFactory' not found
in typo3conf/ext/caretaker_instance/eid/eid.tx_caretakerinstance.php on line 43

As I narrowed it down it comes by an upplerclass "Classes" in autoload.php. I changed the folder name in file system and all occurrences to "Classes" and it worked.

ext_autoload.php:$extensionClassesPath = $extensionPath . 'Classes/';

I am not sure about standards regarding extensions folder names, but the extension builder has all folders in upper case. And that probably is along an extbase standard.

etobi commented 9 years ago

Thanks for testing. Could you create a PR for your change?

TrueType commented 9 years ago

I'd love to. But I can't. For by me changing the "classes" folder name, I imply having the complete extension updated to a folder structure more developers would be familiar with. I cannot offer this as I probably lack the imperience. I could only offer an update with lower case "classes" folder. Will send you a PR for that one.