diff --git a/docs/next/guides/fetching-data.mdx b/docs/next/guides/fetching-data.mdx index 8602e51d4..e1f547b50 100644 --- a/docs/next/guides/fetching-data.mdx +++ b/docs/next/guides/fetching-data.mdx @@ -12,12 +12,12 @@ Faust.js uses [GQty](https://gqty.dev/docs/react/fetching-data) as the primary w GQty works primarily based on TypeScript typings generated using introspection on your GraphQL API. Using WPGraphQL as your Headless WordPress API, you can enable introspection and then generate typings for GQty. You will need to do the following: -1. Run `npm install -D @gqty/cli dotenv` +1. Run `npm install -D @gqty/cli dotenv-flow` 1. Create a `generate` script in your `package.json` that runs `gqty generate`. 1. Create a `gqty.config.js` file at the root of your project. Use the following example config: ```js - require('dotenv').config(); + require('dotenv-flow').config(); /** * @type {import("@gqty/cli").GQtyConfig} diff --git a/docs/release-notes.mdx b/docs/release-notes.mdx index e3f840f3f..555161100 100644 --- a/docs/release-notes.mdx +++ b/docs/release-notes.mdx @@ -147,7 +147,7 @@ module.exports = withFaust({ In the root of your project, alongside `package.json`, replace the `gqty.config.js` file with the following: ```js -require('dotenv').config(); +require('dotenv-flow').config(); /** * @type {import("@gqty/cli").GQtyConfig} diff --git a/docs/tutorial/setup-faustjs.mdx b/docs/tutorial/setup-faustjs.mdx index 8b0c91c71..41737640e 100644 --- a/docs/tutorial/setup-faustjs.mdx +++ b/docs/tutorial/setup-faustjs.mdx @@ -56,7 +56,7 @@ npm install --save-prod @faustjs/core @faustjs/next #### Dev Dependencies ```bash -npm install --save-dev @gqty/cli dotenv +npm install --save-dev @gqty/cli dotenv-flow ``` ### Create `faust.config.js` @@ -105,7 +105,7 @@ Replace the `NEXT_PUBLIC_WORDPRESS_URL` value with the URL of your WordPress sit We use [GQty](https://gqty.dev) as our GraphQL client. To configure GQty, we need to create a `gqty.config.js` file in the root of our project. ```js title=gqty.config.js -require('dotenv').config(); +require('dotenv-flow').config(); /** * @type {import("@gqty/cli").GQtyConfig} @@ -186,7 +186,7 @@ Next, add the following `generate` script to your `package.json`: "devDependencies": { "@gqty/cli": "^1.0.4", "@types/react": "^17.0.15", - "dotenv": "^10.0.0", + "dotenv-flow": "^3.2.0", "typescript": "^4.3.5" } } diff --git a/plugins/faustwp/includes/replacement/graphql-callbacks.php b/plugins/faustwp/includes/replacement/graphql-callbacks.php index dc7ea0e25..dbeb250e3 100644 --- a/plugins/faustwp/includes/replacement/graphql-callbacks.php +++ b/plugins/faustwp/includes/replacement/graphql-callbacks.php @@ -17,7 +17,9 @@ /** * Callback for WP GraphQL 'graphql_request_results' filter. * - * Replaces the WordPress Site URL with the replacement domain in 'url' nodes. + * Replaces the WordPress Site URL with the replacement domain in 'url' and + * 'href' fields. Response data for RootQuery.generalSettings is intentionally + * left unaltered. * * @param object $response The default GraphQL query response. * @@ -31,20 +33,25 @@ function url_replacement( $response ) { if ( is_object( $response ) && property_exists( $response, 'data' ) && - is_array( $response->data ) && - domain_replacement_enabled() && - ! array_key_exists( 'generalSettings', $response->data ) + is_array( $response->data ) ) { - array_walk_recursive( - $response->data, - function( &$value, $key ) { - if ( 'url' === $key || 'href' === $key ) { - $replacement = faustwp_get_setting( 'frontend_uri', '/' ); - $value = str_replace( site_url(), $replacement, $value ); - } - } - ); + url_replace_recursive( $response->data ); } return $response; } + +function url_replace_recursive( &$data ) { + foreach ( $data as $key => &$value ) { + if ( $key === 'generalSettings' ) { + continue; + } + + if ( ( 'url' === $key || 'href' === $key ) && is_string( $value ) ) { + $replacement = faustwp_get_setting( 'frontend_uri', '/' ); + $value = str_replace( site_url(), $replacement, $value ); + } else if ( is_array( $value ) ) { + url_replace_recursive( $value ); + } + } +} diff --git a/plugins/faustwp/tests/integration/replacement/test-graphql-callbacks.php b/plugins/faustwp/tests/integration/replacement/test-graphql-callbacks.php new file mode 100644 index 000000000..1390a6293 --- /dev/null +++ b/plugins/faustwp/tests/integration/replacement/test-graphql-callbacks.php @@ -0,0 +1,136 @@ + [ + 'url' => 'http://example.org' + ], + 'menuItems' => [ + 'nodes' => [ + [ + 'label' => 'Hello World', + 'url' => 'http://example.org/hello-world/' + ], + [ + 'label' => 'External Site', + 'url' => 'http://external.site/no-replacement' + ] + ] + ], + 'menus' => [ + 'nodes' => [ + [ + 'menuItems' => [ + 'nodes' => [ + [ + 'label' => 'Hello World', + 'url' => 'http://example.org/hello-world/' + ] + ] + ] + ] + ] + ], + 'aThingWithHref' => [ + 'nodes' => [ + [ + 'label' => "This uses href instead of url fields", + 'href' => 'http://example.org/href' + ] + ] + ] + ]; + + private $expectedData = [ + 'generalSettings' => [ + 'url' => 'http://example.org' + ], + 'menuItems' => [ + 'nodes' => [ + [ + 'label' => 'Hello World', + 'url' => 'http://frontend/hello-world/' + ], + [ + 'label' => 'External Site', + 'url' => 'http://external.site/no-replacement' + ] + ] + ], + 'menus' => [ + 'nodes' => [ + [ + 'menuItems' => [ + 'nodes' => [ + [ + 'label' => 'Hello World', + 'url' => 'http://frontend/hello-world/' + ] + ] + ] + ] + ] + ], + 'aThingWithHref' => [ + 'nodes' => [ + [ + 'label' => "This uses href instead of url fields", + 'href' => 'http://frontend/href' + ] + ] + ] + ]; + + public function setUp() { + parent::setUp(); + + faustwp_update_setting( 'frontend_uri', 'http://frontend' ); + + $this->graphqlResponse = new \stdClass(); + $this->graphqlResponse->data = $this->responseData; + } + + public function test_graphql_request_results_filter() { + $this->assertSame( 10, has_action( 'graphql_request_results', 'WPE\FaustWP\Replacement\url_replacement' ) ); + } + + /** + * Tests url_replacement() returns original data when rewrites are not enabled. + */ + public function test_url_replacement_does_not_alter_response_when_rewrites_are_not_enabled() { + faustwp_update_setting( 'enable_rewrites', '0' ); + $filteredRespone = url_replacement( $this->graphqlResponse ); + $this->assertSame( $this->responseData, $filteredRespone->data ); + } + + /** + * Tests url_replacement() does not modify generalSettings even when rewrites are enabled. + */ + public function test_url_replacement_does_not_alter_generalSettings_when_rewrites_are_enabled() { + faustwp_update_setting( 'enable_rewrites', '1' ); + $filteredRespone = url_replacement( $this->graphqlResponse ); + $this->assertSame( $this->responseData['generalSettings'], $filteredRespone->data['generalSettings'] ); + } + + /** + * Tests url_replacement() modifies url and href fields when rewrites are enabled. + */ + public function test_url_replacement_replaces_url_fields_when_rewrites_are_enabled() { + faustwp_update_setting( 'enable_rewrites', '1' ); + $filteredRespone = url_replacement( $this->graphqlResponse ); + $this->assertSame( $this->expectedData, $filteredRespone->data ); + } +}