Extension update - Technical submission failsHow to fix warnings / errors raised by the Magento Marketplace technical review report?Marketplace Extension RecommendationsMulti-Vendor extension for MagentoExtension Submission Expired in magento marketplaceMagento marketplace extension submission issueMagento 2 Extension submission to MarketplaceMagento 2 marketplace submission issueMagento Marketplace: After update the magento marketplace, extension stuck in “Awaiting Marketing Review”Magento 2 : Expected HIT on the second page load after product price update while Marketplace submissionCan not update the database during magento 2 module submission in marketplace
Like a Baby - Riddle
What is this plant I saw for sale at a Romanian farmer's market?
Is there a term for someone whose preferred policies are a mix of Left and Right?
Leaving job close to major deadlines
How would Japanese people react to someone refusing to say “itadakimasu” for religious reasons?
Manager wants to hire me; HR does not. How to proceed?
How useful is the GRE Exam?
Can you cover a cube with copies of this shape?
How to search for Android apps without ads?
1960s sci-fi anthology with a Viking fighting a U.S. army MP on the cover
Is my research statement supposed to lead to papers in top journals?
Can a non-invertible function be inverted by returning a set of all possible solutions?
How can I ping multiple IP addresses at the same time?
What could be the physiological mechanism for a biological Geiger counter?
Redirecting output only on a successful command call
How can Caller ID be faked?
How do I become a better writer when I hate reading?
How "fast" does astronomical events happen?
What is the context for Napoleon's quote "[the Austrians] did not know the value of five minutes"?
How does the Linux command "mount -a" work?
Have Steve Rogers (Captain America) and a young Erik Lehnsherr (Magneto) interacted during WWII?
When is the phrase "j'ai bon" used?
Converting 3x7 to a 1x7. Is it possible with only existing parts?
Interview was just a one hour panel. Got an offer the next day; do I accept or is this a red flag?
Extension update - Technical submission fails
How to fix warnings / errors raised by the Magento Marketplace technical review report?Marketplace Extension RecommendationsMulti-Vendor extension for MagentoExtension Submission Expired in magento marketplaceMagento marketplace extension submission issueMagento 2 Extension submission to MarketplaceMagento 2 marketplace submission issueMagento Marketplace: After update the magento marketplace, extension stuck in “Awaiting Marketing Review”Magento 2 : Expected HIT on the second page load after product price update while Marketplace submissionCan not update the database during magento 2 module submission in marketplace
.everyoneloves__top-leaderboard:empty,.everyoneloves__mid-leaderboard:empty,.everyoneloves__bot-mid-leaderboard:empty margin-bottom:0;
Magento 2.3 got CsrfAwareActionInterface
, which needs to be implemented in order to be able to handle webhooks properly.
I got the following
$part = interface_exists('MagentoFrameworkAppCsrfAwareActionInterface') ? 'new' : 'old';
include_once sprintf('a/b/c/d+%s.php', $part);
In practice, this works, CsrfAwareActionInterface
won't exist in Magento <2.3, but Magento's code sniffer won't agree.
It tests the code with PHP 7.1 and is successful, then it tries PHP 5.6 which has no return types, resulting in PHP syntax error: syntax error, unexpected ':', expecting ';' or '{'
, telling me to fix the errors and resubmit.
Implementing the interface without compatible return types won't work.
Releasing 2 different versions (2.3 & <2.3) and supporting them wouldn't be wise, given that CsrfAwareActionInterface
would be the sole difference between them.
Is the result of that test (code sniffer) breaking? Can an extension be approved despite that test failing?
extensions magento2.3 marketplace
New contributor
add a comment |
Magento 2.3 got CsrfAwareActionInterface
, which needs to be implemented in order to be able to handle webhooks properly.
I got the following
$part = interface_exists('MagentoFrameworkAppCsrfAwareActionInterface') ? 'new' : 'old';
include_once sprintf('a/b/c/d+%s.php', $part);
In practice, this works, CsrfAwareActionInterface
won't exist in Magento <2.3, but Magento's code sniffer won't agree.
It tests the code with PHP 7.1 and is successful, then it tries PHP 5.6 which has no return types, resulting in PHP syntax error: syntax error, unexpected ':', expecting ';' or '{'
, telling me to fix the errors and resubmit.
Implementing the interface without compatible return types won't work.
Releasing 2 different versions (2.3 & <2.3) and supporting them wouldn't be wise, given that CsrfAwareActionInterface
would be the sole difference between them.
Is the result of that test (code sniffer) breaking? Can an extension be approved despite that test failing?
extensions magento2.3 marketplace
New contributor
Which codesniff ruleset?
– Dominic Xigen
Jun 9 at 12:26
MEQP2.PHP.Syntax.PHPSyntax. It's not about the Code Sniffer itself, but how it's configured @ Magento marketplace level, it seems to be an acknowledged bug - github.com/magento/marketplace-eqp/issues/104
– Ravenous
Jun 9 at 12:35
add a comment |
Magento 2.3 got CsrfAwareActionInterface
, which needs to be implemented in order to be able to handle webhooks properly.
I got the following
$part = interface_exists('MagentoFrameworkAppCsrfAwareActionInterface') ? 'new' : 'old';
include_once sprintf('a/b/c/d+%s.php', $part);
In practice, this works, CsrfAwareActionInterface
won't exist in Magento <2.3, but Magento's code sniffer won't agree.
It tests the code with PHP 7.1 and is successful, then it tries PHP 5.6 which has no return types, resulting in PHP syntax error: syntax error, unexpected ':', expecting ';' or '{'
, telling me to fix the errors and resubmit.
Implementing the interface without compatible return types won't work.
Releasing 2 different versions (2.3 & <2.3) and supporting them wouldn't be wise, given that CsrfAwareActionInterface
would be the sole difference between them.
Is the result of that test (code sniffer) breaking? Can an extension be approved despite that test failing?
extensions magento2.3 marketplace
New contributor
Magento 2.3 got CsrfAwareActionInterface
, which needs to be implemented in order to be able to handle webhooks properly.
I got the following
$part = interface_exists('MagentoFrameworkAppCsrfAwareActionInterface') ? 'new' : 'old';
include_once sprintf('a/b/c/d+%s.php', $part);
In practice, this works, CsrfAwareActionInterface
won't exist in Magento <2.3, but Magento's code sniffer won't agree.
It tests the code with PHP 7.1 and is successful, then it tries PHP 5.6 which has no return types, resulting in PHP syntax error: syntax error, unexpected ':', expecting ';' or '{'
, telling me to fix the errors and resubmit.
Implementing the interface without compatible return types won't work.
Releasing 2 different versions (2.3 & <2.3) and supporting them wouldn't be wise, given that CsrfAwareActionInterface
would be the sole difference between them.
Is the result of that test (code sniffer) breaking? Can an extension be approved despite that test failing?
extensions magento2.3 marketplace
extensions magento2.3 marketplace
New contributor
New contributor
edited Jun 9 at 12:10
Ravenous
New contributor
asked Jun 9 at 11:54
RavenousRavenous
1011
1011
New contributor
New contributor
Which codesniff ruleset?
– Dominic Xigen
Jun 9 at 12:26
MEQP2.PHP.Syntax.PHPSyntax. It's not about the Code Sniffer itself, but how it's configured @ Magento marketplace level, it seems to be an acknowledged bug - github.com/magento/marketplace-eqp/issues/104
– Ravenous
Jun 9 at 12:35
add a comment |
Which codesniff ruleset?
– Dominic Xigen
Jun 9 at 12:26
MEQP2.PHP.Syntax.PHPSyntax. It's not about the Code Sniffer itself, but how it's configured @ Magento marketplace level, it seems to be an acknowledged bug - github.com/magento/marketplace-eqp/issues/104
– Ravenous
Jun 9 at 12:35
Which codesniff ruleset?
– Dominic Xigen
Jun 9 at 12:26
Which codesniff ruleset?
– Dominic Xigen
Jun 9 at 12:26
MEQP2.PHP.Syntax.PHPSyntax. It's not about the Code Sniffer itself, but how it's configured @ Magento marketplace level, it seems to be an acknowledged bug - github.com/magento/marketplace-eqp/issues/104
– Ravenous
Jun 9 at 12:35
MEQP2.PHP.Syntax.PHPSyntax. It's not about the Code Sniffer itself, but how it's configured @ Magento marketplace level, it seems to be an acknowledged bug - github.com/magento/marketplace-eqp/issues/104
– Ravenous
Jun 9 at 12:35
add a comment |
0
active
oldest
votes
Your Answer
StackExchange.ready(function()
var channelOptions =
tags: "".split(" "),
id: "479"
;
initTagRenderer("".split(" "), "".split(" "), channelOptions);
StackExchange.using("externalEditor", function()
// Have to fire editor after snippets, if snippets enabled
if (StackExchange.settings.snippets.snippetsEnabled)
StackExchange.using("snippets", function()
createEditor();
);
else
createEditor();
);
function createEditor()
StackExchange.prepareEditor(
heartbeatType: 'answer',
autoActivateHeartbeat: false,
convertImagesToLinks: false,
noModals: true,
showLowRepImageUploadWarning: true,
reputationToPostImages: null,
bindNavPrevention: true,
postfix: "",
imageUploader:
brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
allowUrls: true
,
onDemand: true,
discardSelector: ".discard-answer"
,immediatelyShowMarkdownHelp:true
);
);
Ravenous is a new contributor. Be nice, and check out our Code of Conduct.
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function ()
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fmagento.stackexchange.com%2fquestions%2f277725%2fextension-update-technical-submission-fails%23new-answer', 'question_page');
);
Post as a guest
Required, but never shown
0
active
oldest
votes
0
active
oldest
votes
active
oldest
votes
active
oldest
votes
Ravenous is a new contributor. Be nice, and check out our Code of Conduct.
Ravenous is a new contributor. Be nice, and check out our Code of Conduct.
Ravenous is a new contributor. Be nice, and check out our Code of Conduct.
Ravenous is a new contributor. Be nice, and check out our Code of Conduct.
Thanks for contributing an answer to Magento Stack Exchange!
- Please be sure to answer the question. Provide details and share your research!
But avoid …
- Asking for help, clarification, or responding to other answers.
- Making statements based on opinion; back them up with references or personal experience.
To learn more, see our tips on writing great answers.
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function ()
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fmagento.stackexchange.com%2fquestions%2f277725%2fextension-update-technical-submission-fails%23new-answer', 'question_page');
);
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Which codesniff ruleset?
– Dominic Xigen
Jun 9 at 12:26
MEQP2.PHP.Syntax.PHPSyntax. It's not about the Code Sniffer itself, but how it's configured @ Magento marketplace level, it seems to be an acknowledged bug - github.com/magento/marketplace-eqp/issues/104
– Ravenous
Jun 9 at 12:35