Skip to content

Error In CategoryEdit #1766

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Conversation

shaheerali49
Copy link

@shaheerali49 shaheerali49 commented Aug 31, 2015

When I click on any category in backend, it return this string in white page:

{"data":[{"text":"Apple (1)","id":"4","store":1,"path":"1\/2\/4","cls":"folder active-category","allowDrop":true,"allowDrag":true,"expanded":true}],"parameters":{"text":"Default Category (0)","draggable":false,"allowDrop":true,"id":2,"expanded":0,"store_id":1,"category_id":4,"root_visible":1}}

The issue is coming from the file Magento\Catalog\Controller\Adminhtml\Category\Tree.php. It return the json result. The solution is that copy the code from line no 60 in execute function $resultRedirect = $this->resultRedirectFactory->create(); and paste it at the start of the function and end of the execute function remove return from the line
return $resultJson->setData(...);
and add a line below this code
return $resultRedirect->setPath('catalog/category/edit', ['_current' => true, 'id' => $categoryId]);

@vpelipenko vpelipenko added the MX label Sep 1, 2015
@magento-cicd2
Copy link
Contributor

We have automated a Magento Contributor License Agreement verifier for contributions sent to our GitHub projects.
Please see the CLA agreement in the Pull Request comments below.

@benmarks
Copy link
Contributor

benmarks commented Nov 9, 2015

@shausol Can you check the CLA is signed?

@shaheerali49
Copy link
Author

Yes i signed the CLA right now.

@shaheerali49
Copy link
Author

May i know you that why my pull request is not yet merged. I know that now you people change the way of category edit. But my solution works on previous version of m2

@KrystynaKabannyk
Copy link

Hello @shaheerali49, could you please clarify what particular version do you work with?

@sshrewz
Copy link

sshrewz commented Aug 11, 2016

@shaheerali49, can you advise on what particular version you were woking with?

@shaheerali49
Copy link
Author

Please check the issue
#1676

@shaheerali49
Copy link
Author

@Snohe please check the reference link for issue and the fix is for M2 beta version

@shaheeraliki5
Copy link

@benmarks @Snohe @KrystynaKabannyk @vpelipenko @kandy any update about this thread ?

@vrann
Copy link
Contributor

vrann commented Mar 22, 2017

@shaheerali49 Hi, is this issue still reproduced on the latest develop branch?

I noticed that in the code of the pull request, this variable is completely ignored $resultJson. Does it need to be instantiated at the first place?

@vrann vrann self-assigned this Mar 22, 2017
@vrann vrann added this to the March 2017 milestone Mar 22, 2017
@okorshenko okorshenko modified the milestones: March 2017, April 2017 Apr 2, 2017
@vrann
Copy link
Contributor

vrann commented Apr 12, 2017

@shaheerali49 does not seem like applicable anymore. Please feel free to re-create if it can be reproduced.

@vrann vrann closed this Apr 12, 2017
okorshenko pushed a commit that referenced this pull request Nov 27, 2017
Public Pull Requests

#12135 10210: Transport variable can not be altered in email_invoice_set_template_vars_before Event (backport MAGETWO-69482 to 2.1). by @RomaKis
#12043 [Backport 2.1] Add swatch option: Prevent loosing data and default value if data is not populated via adminhtml by @gomencal

Fixed Public Issues

#10210 Transport variable can not be altered in email_invoice_set_template_vars_before Event
#9410 Impossible to add swatch options via Service Contracts if there is no existing swatch option for attribute
#10707 Create attribute option via API for swatch attribute fails
#10737 Can't import attribute option over API if option is 'visual swatch'
#11032 Unable to add new options to swatch attribute
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.