So I decided to get rid of the category module that would appear on the right, and decided to add my categories to appear in the top menu bar.

The bug I am seeing is that I added all my categories fine, but added a category twice by accident. So when I delete the duplicate which is in the middle of all the other categories, once deleted, all the other categories entered below the deleted one vanish also. I assumed something drastic happened. So I added all the categories below that again. When refreshing the store page, I saw many duplicates so I had to go back and check the menu area of the admin again.

How I set it up was I created a "Category" category, and added all my categories below it as sub-categories.

The issue I see is that if you delete a sub-category, all the sub-categories below it vanish. But if you click on the "Menus" button again in the admin area, which refreshes the page, they all reappear.

Not sure why this happens, but might need further looking at.
In General
Friday, January 06 2017, 12:45 AM
Share this post:

Accepted Answer

Friday, January 06 2017, 11:53 AM - #Permalink
Confirmed and added to the tracker Breck: https://github.com/arastta/arastta/issues/642

Thanks for reporting.

Regards
The reply is currently minimized Show
Responses (8)
  • Accepted Answer

    Sunday, January 22 2017, 03:38 AM - #Permalink
    Then instead of posting them here, submit the bugs and edits on github so they can be reviewed and added to the base code.
    Like
    1
    The reply is currently minimized Show
  • Accepted Answer

    Sunday, January 22 2017, 05:56 PM - #Permalink
    Thanks David for your contributions. We can review and merge to the core if you submit a Pull Request on GitHub. Also, you can use the Modification system instead of changing the core files:

    https://arastta.org/docs/developers/modification-system

    Regards
    Like
    1
    The reply is currently minimized Show
  • Accepted Answer

    Monday, January 23 2017, 02:13 AM - #Permalink
    I wan't giving you an attitude. You were complaining about having to apply the edits again after updates.
    I was suggesting you submit them on github so they can be added to the core files for everyone and you wouldn't have to edit anymore after updating.
    Like
    1
    The reply is currently minimized Show
  • Accepted Answer

    Monday, January 23 2017, 11:26 AM - #Permalink
    Hi @David,

    Thank you for the report and for your fix. As friends told, it would be better to send a Pull Request over Github for your valuable fix. Thus, all the community benefit from it.
    I think there is a misunderstood between you and Breck. Take it easy and keep in mind the community ;)

    Regards
    Like
    1
    The reply is currently minimized Show
  • Accepted Answer

    Sunday, January 15 2017, 02:41 AM - #Permalink
    Hopefully helpful in testing your fix.
    You dont need to do anything fancy. just delete one of the "test ??" menu items under the default "tablet" menu. All items lower in the list disappear.
    The reply is currently minimized Show
  • Accepted Answer

    Friday, January 20 2017, 04:28 PM - #Permalink
    I've fixed the issue, will be released in Arastta 1.5.2
    The reply is currently minimized Show
  • Accepted Answer

    Sunday, January 22 2017, 02:25 AM - #Permalink
    Hi cneyt
    I have a couple of other bugs I reported as part of a 'help' post. I dont think anyone has picked them up yet.

    Both bugs are in code which doesnt handle multi sites properly.

    1.

    bug = system\library\app\catalog.php is loading the wrong shop id on all pages except home -- I changed to the code below - two lines commented

    // Store
    if (isset($_SERVER['HTTPS']) && (($_SERVER['HTTPS'] == 'on') || ($_SERVER['HTTPS'] == '1'))) {
    // $store_query = $this->db->query("SELECT * FROM " . DB_PREFIX . "store WHERE REPLACE(`ssl`, 'www.', '') = '" . $this->db->escape('https://' . str_replace('www.', '', $_SERVER['HTTP_HOST']) . rtrim(dirname($_SERVER['PHP_SELF']), '/.\\') . '/') . "'");
    $sql = "SELECT * FROM " . DB_PREFIX . "store WHERE REPLACE(`ssl`, 'www.', '') = '" . $this->db->escape('https://' . str_replace('www.', '', $_SERVER['HTTP_HOST']) . '/') . "'";
    } else {
    // $store_query = $this->db->query("SELECT * FROM " . DB_PREFIX . "store WHERE REPLACE(`url`, 'www.', '') = '" . $this->db->escape('http://' . str_replace('www.', '', $_SERVER['HTTP_HOST']) . rtrim(dirname($_SERVER['PHP_SELF']), '/.\\') . '/') . "'");
    $sql = "SELECT * FROM " . DB_PREFIX . "store WHERE REPLACE(`url`, 'www.', '') = '" . $this->db->escape('http://' . str_replace('www.', '', $_SERVER['HTTP_HOST']) . '/') . "'";
    }
    $store_query = $this->db->query($sql);



    2.

    Admin menus bug = Add menu bug is caused by code not including the 'stores' array index when the menu item is added

    Fix is admin\model\appearance\menu.php at line 109
    $menu = array(
    'name' =>$data['menu_desc'][0]['name'],
    'menu_type' =>$data['type'],
    'menu_id' => $menu_id
    );
    needs store array adding to match previous line which auto added store 1 to the db and later code which expects a store array
    $menu = array(
    'name' =>$data['menu_desc'][0]['name'],
    'menu_type' =>$data['type'],
    'menu_id' => $menu_id,
    'store' => [0]
    );


    It would be lovely if I didnt have to keep re-editing these into each Arastta update I do.

    Cheers
    The reply is currently minimized Show
  • Accepted Answer

    Monday, January 23 2017, 01:52 AM - #Permalink
    Thanks for your reply Denis. I did look for a way of reporting bugs but didnt find one.
    Since then I have found another 5 bugs, that and the attitude of Breck Demers make me think I should just head off in my own direction so I probably wont be bothering you again.
    Cheers
    The reply is currently minimized Show
Your Reply