Merge pull request #2310 from zephir-lang/development

0.15.1
zephir-lang · Oct 8, 2021 · 643e639 · 643e639
2 parents d56e764 + d5bcff3
commit 643e639
Show file tree

Hide file tree

Showing 13 changed files with 136 additions and 13 deletions.
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -55,11 +55,21 @@ jobs:
       - name: Build Zephir PHAR
         run: .ci/build-phar.sh
 
+      - name: Upload Zephir Phar
+        uses: actions/upload-artifact@v2
+        with:
+          name: zephir.phar
+          path: zephir.phar
+
       - name: Get the release version
         id: get-version
         run: |
           echo ::set-output name=version::${GITHUB_REF#refs/tags/}
 
+      - name: Prepare Release Notes
+        run: |
+          ./.ci/release-notes.sh ./CHANGELOG.md > ./release-notes.md
+
       - name: Create Release
         uses: ncipollo/release-action@v1
         with:

diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,6 +6,10 @@ and this project adheres to [Semantic Versioning](http://semver.org).
 
 ## [Unreleased]
 
+## [0.15.1] - 2021-10-08
+### Fixed
+- Fix support of `string` type in struct globals [#2308](https://github.com/zephir-lang/zephir/issues/2308)
+
 ## [0.15.0] - 2021-10-05
 ### Added
 - Added support for `string` type in php.ini [#2280](https://github.com/zephir-lang/zephir/issues/2280)
@@ -553,7 +557,8 @@ and this project adheres to [Semantic Versioning](http://semver.org).
   [#1524](https://github.com/zephir-lang/zephir/issues/1524)
 
 
-[Unreleased]: https://github.com/zephir-lang/zephir/compare/0.15.0...HEAD
+[Unreleased]: https://github.com/zephir-lang/zephir/compare/0.15.1...HEAD
+[0.15.1]: https://github.com/zephir-lang/zephir/compare/0.15.0...0.15.1
 [0.15.0]: https://github.com/zephir-lang/zephir/compare/0.14.0...0.15.0
 [0.14.0]: https://github.com/zephir-lang/zephir/compare/0.14.0-beta.3...0.14.0
 [0.14.0-beta.3]: https://github.com/zephir-lang/zephir/compare/0.14.0-beta.2...0.14.0-beta.3

diff --git a/Library/Code/Builder/Struct.php b/Library/Code/Builder/Struct.php
@@ -122,6 +122,9 @@ public function getCDefault(string $name, array $global, string $namespace): str
             case 'bool':
                 return '';
 
+            case 'string':
+                return "\t".$namespace.'_globals->'.$this->simpleName.'.'.$name.' = ZSTR_VAL(zend_string_init(ZEND_STRL("'.$global['default'].'"), 0));';
+
             case 'int':
             case 'uint':
             case 'long':
@@ -192,6 +195,9 @@ protected function convertToCType(string $type): string
             case 'hash':
                 return 'HashTable* ';
 
+            case 'string':
+                return 'zend_string* ';
+
             case 'int':
             case 'uint':
             case 'long':

diff --git a/Library/Compiler/CompilerFileFactory.php b/Library/Compiler/CompilerFileFactory.php
@@ -9,6 +9,8 @@
  * the LICENSE file that was distributed with this source code.
  */
 
+declare(strict_types=1);
+
 namespace Zephir\Compiler;
 
 use Psr\Log\LoggerInterface;
@@ -19,15 +21,12 @@
 
 final class CompilerFileFactory
 {
-    private $config;
-    private $filesystem;
-    private $logger;
-
-    public function __construct(
-        Config $config,
-        FileSystemInterface $filesystem,
-        LoggerInterface $logger
-    ) {
+    private Config $config;
+    private FileSystemInterface $filesystem;
+    private LoggerInterface $logger;
+
+    public function __construct(Config $config, FileSystemInterface $filesystem, LoggerInterface $logger)
+    {
         $this->config = $config;
         $this->filesystem = $filesystem;
         $this->logger = $logger;
@@ -43,7 +42,7 @@ public function __construct(
      *
      * @return FileInterface
      */
-    public function create($className, $filePath)
+    public function create(string $className, string $filePath): FileInterface
     {
         $compiler = new CompilerFile($this->config, new AliasManager(), $this->filesystem);
 

diff --git a/Library/Zephir.php b/Library/Zephir.php
@@ -16,7 +16,7 @@
  */
 final class Zephir
 {
-    public const VERSION = '0.15.0-$Id$';
+    public const VERSION = '0.15.1-$Id$';
 
     public const LOGO = <<<'ASCII'
  _____              __    _

diff --git a/config.json b/config.json
@@ -119,6 +119,10 @@
       "type": "bool",
       "default": true
     },
+    "orm.cache_prefix": {
+      "type": "string",
+      "default": "prefix-string-"
+    },
     "extension.test_ini_variable": {
       "type": "bool",
       "default": true,

diff --git a/ext/php_stub.h b/ext/php_stub.h
@@ -14,7 +14,7 @@
 #define PHP_STUB_VERSION     "1.0.0"
 #define PHP_STUB_EXTNAME     "stub"
 #define PHP_STUB_AUTHOR      "Phalcon Team and contributors"
-#define PHP_STUB_ZEPVERSION  "0.15.0-$Id$"
+#define PHP_STUB_ZEPVERSION  "0.15.1-$Id$"
 #define PHP_STUB_DESCRIPTION "Description <b>test</b> for<br/>Test Extension."
 
 typedef struct _zephir_struct_db { 
@@ -26,6 +26,7 @@ typedef struct _zephir_struct_db {
 typedef struct _zephir_struct_orm { 
 	int cache_level;
 	zend_bool cache_enable;
+	zend_string*  cache_prefix;
 } zephir_struct_orm;
 
 typedef struct _zephir_struct_extension { 

diff --git a/ext/stub.c b/ext/stub.c
@@ -254,6 +254,7 @@ PHP_INI_BEGIN()
 
 
 	STD_PHP_INI_BOOLEAN("stub.orm.cache_enable", "1", PHP_INI_ALL, OnUpdateBool, orm.cache_enable, zend_stub_globals, stub_globals)
+
 	STD_PHP_INI_BOOLEAN("extension.test_ini_variable", "1", PHP_INI_ALL, OnUpdateBool, extension.test_ini_variable, zend_stub_globals, stub_globals)
 	STD_PHP_INI_BOOLEAN("ini-entry.my_setting_1", "1", PHP_INI_ALL, OnUpdateBool, my_setting_1, zend_stub_globals, stub_globals)
 	STD_PHP_INI_BOOLEAN("stub.test_setting_1", "1", PHP_INI_ALL, OnUpdateBool, test_setting_1, zend_stub_globals, stub_globals)
@@ -520,6 +521,7 @@ static void php_zephir_init_globals(zend_stub_globals *stub_globals)
 	stub_globals->db.my_setting_3 = 7.5;
 	stub_globals->orm.cache_level = 3;
 
+	stub_globals->orm.cache_prefix = ZSTR_VAL(zend_string_init(ZEND_STRL("prefix-string-"), 0));
 
 	stub_globals->my_setting_1 = 1;
 	stub_globals->test_setting_1 = 1;

diff --git a/ext/stub/globals.zep.c b/ext/stub/globals.zep.c
diff --git a/ext/stub/globals.zep.h b/ext/stub/globals.zep.h
diff --git a/release-notes.md b/release-notes.md
@@ -0,0 +1,13 @@
+### Fixed
+- Fixed multiple return types in stubs [#2283](https://github.com/zephir-lang/zephir/issues/2283)
+- Fixed `bool` return type in stubs [#2272](https://github.com/zephir-lang/zephir/issues/2272)
+
+### Changed
+- Removed `.zep` from stubs filenames [#2273](https://github.com/zephir-lang/zephir/issues/2273)
+
+## [0.14.0] - 2021-09-18
+### Added
+- Added support for `require_once` [#2253](https://github.com/zephir-lang/zephir/issues/2253)
+
+### Changed
+- Bumped minimal version of Zephir Parser to `1.4.1`. [#2284](https://github.com/zephir-lang/zephir/issues/2284)
diff --git a/stub/globals.zep b/stub/globals.zep
@@ -35,6 +35,11 @@ class Globals
 		globals_set("orm.cache_level", value);
 	}
 
+	public function setDefaultGlobalsOrmCachePrefix(string value) -> void
+	{
+	    globals_set("orm.cache_prefix", value);
+	}
+
 	/* Get Default Properties */
 
 	/**
@@ -108,4 +113,12 @@ class Globals
 	{
 		return globals_get("orm.cache_level");
 	}
+
+	/**
+     * @return mixed
+     */
+    public function getDefaultGlobalsOrmCachePrefix()
+    {
+        return globals_get("orm.cache_prefix");
+    }
 }
diff --git a/tests/Extension/GlobalsTest.php b/tests/Extension/GlobalsTest.php
@@ -83,6 +83,23 @@ public function testSetStringValue(): void
 
         $this->test->setStringValue('Long Text without any sense...');
         $this->assertSame('Long Text without any sense...', $this->test->getDefaultGlobals8());
+
+        /**
+         * Get and set string value from globals struct.
+         */
+        $this->assertSame('prefix-string-', $this->test->getDefaultGlobalsOrmCachePrefix());
+
+        $this->test->setDefaultGlobalsOrmCachePrefix('1');
+        $this->assertSame('1', $this->test->getDefaultGlobalsOrmCachePrefix());
+
+        $this->test->setDefaultGlobalsOrmCachePrefix('c');
+        $this->assertSame('c', $this->test->getDefaultGlobalsOrmCachePrefix());
+
+        $this->test->setDefaultGlobalsOrmCachePrefix('char');
+        $this->assertSame('char', $this->test->getDefaultGlobalsOrmCachePrefix());
+
+        $this->test->setDefaultGlobalsOrmCachePrefix('Long Text without any sense...');
+        $this->assertSame('Long Text without any sense...', $this->test->getDefaultGlobalsOrmCachePrefix());
     }
 
     public function testSetBoolValueUsingInt(): void