Uh oh!
There was an error while loading. Please reload this page.
refactor(generators): Introduce PhpGenerator class, Order enum - #7162
Merged
cpcallen merged 3 commits intoJun 14, 2023
Conversation
cpcallen
enabled auto-merge (squash)
June 14, 2023 03:38
BeksOmega
approved these changes
Jun 14, 2023
cpcallen added a commit
to cpcallen/blockly
that referenced
this pull request
Jun 15, 2023
These were inadvertently omitted from RaspberryPiFoundation#7161 and RaspberryPiFoundation#7162, respectively.
cpcallen added a commit
that referenced
this pull request
Jun 20, 2023
…ffects (#7173) * refactor(generators): Move lang.js -> lang/lang_gernator.js Move the LangGenerator definitions into their respective subdirectories and add a _generator suffix to their filenames, i.e. generators/javascript.js becomes generators/javascript/javascript_generator.js. This is to keep related code together and allow the `lang/all.js` entrypoints to be moved to the top level generators/ directory. No goog module IDs were changed, so playground and test code that accesses this modules by filename does not need to be modified. * refactor(generators) Move lang/all.js -> lang.js - Move the entrypoints in generators/*/all.js to correspondingly-named files in generators/ instead—i.e., generators/javascript/all.js becomes generators/javascript.js. - Update build_tasks.js accordingly. * fix(generators): Add missing exports for LuaGenerator, PhpGenerator These were inadvertently omitted from #7161 and #7162, respectively. * refactor(generators): Make block generator modules side-effect free - Move declaration of <lang>Generator instance from generators/<lang>/<lang>_generator.js to generators/<lang>.js. - Move .addReservedWords() calls from generators/<lang>/*.js to generators/<lang>.js - Modify generators/<lang>/*.js to export block generator functions individually, rather than installing on <lang>Generator instance. - Modify generators/<lang>.js to import and install block generator functions on <lang>Generator instance. * fix(tests): Fix tests broken by restructuring of generators Where these tests needed block generator functions preinstalled they should have been importing the Blockly.<Lang>.all module. Where they do not need the provided block generator functions they can now create their own empty <Lang>Generator instances. * chore: Update renamings file - Fix a malformation in previous entries that was not detected by the renaming file validator test. - Add entries describing the work done in this and related recent PRs. * fix: Correct minor errors in PR #7173 - Fix a search-and-replace error in renamings.json5 - Fix an incorrect-but-usable import in generator_test.js
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The basics
npm run formatandnpm run lintThe details
Resolves
Part of #7085.
Proposed Changes
Introduce a
PhpGeneratorclass to be the class of whichphpGeneratoris an instance (without any additional properties making it a singleton).Introduce an
Orderenum (for now Closure-style) for the operator precedence table.Don't rename
phpGeneratortoPHPin generators/php/*.js.Behaviour Before/After Change
There should be no changes in behaviour.
Reason for Changes
See #7085.
Test Coverage
Passes
npm test; no changes to manual testing anticipated.Documentation
Documentation for the
PhpGeneratorclass andOrderenum should be automatically generated by our JSDoc configs.Additional Information
It is now possible to create multiple
phpGeneratorobjects, which can share (or not share) block generator implementations. Note that new instances do not by default come with any generator functions pre-installed.We now provide an enum of the operator precedence values (order of operations) constants as a separate export:
DEPRECATION: The
.ORDER_*constants onphpGeneratorare deprecated; they still exist but are no longer part of the documented API and will be removed in a future version of Blockly. We recommend that you update your custom generator code to use theOrderenum at your earliest convenience.