totten / civix

CiviCRM Extension Builder
http://civicrm.org/
Other
56 stars 56 forks source link

Smarty local mix-in missing? #292

Closed bjendres closed 1 year ago

bjendres commented 1 year ago

Maybe there's something wrong with my setup, but I get the following error on v23.01.1:

Incremental upgrades
====================

Current civix format is v22.12.1.

Upgrade v22.12.1 => v23.01.0
----------------------------

Executing upgrade script /home/bjoern/workspace/civix/upgrades/23.01.0.up.php

 ! [NOTE] Civix v23.01 simplifies the boilerplate used for Smarty registration.                                         
 !                                                                                                                      
 !        The following may be affected:                                                                                
 !                                                                                                                      
 !        1. *_civix_civicrm_config          Remove Smarty boilerplate                                                  
 !        2. info.xml                        Enable smarty-v2@1.0.0                                                     

 [WARNING] If Smarty templates are used by any lifecycle hooks (install,enable,disable,uninstall,upgrade,managed), then 
           please re-test them.                                                                                         

 Continue with upgrade? (yes/no) [yes]:
 > 

Enable mixin smarty-v2@1.0.0

In Mixlib.php line 184:

  Failed to locate smarty-v2@1/mixin.php (mixlibDir=/home/bjoern/workspace/civix/extern/mixin, mixlibUrl=)  
bjendres commented 1 year ago

Or to put the question in another way: where do the contents of civix/extern/mixin come from? Because smarty-v2@1 is definitely missing.

totten commented 1 year ago

@bjendres You probably need to run composer install.

For a system that's running off civix.git, the download process goes a bit like this:

But this should only affect git-based deployments. If one runs from civix.phar, then those should be bundled.

totten commented 1 year ago

Feel free to re-open if it's still an issue.

bjendres commented 1 year ago

Thanks