To participate you must create an account on apostrophenow.org. If you have already done so, click Login.

Ticket #464 (closed defect: fixed)

Opened 20 months ago

Last modified 19 months ago

new_slots_top: false doesn't work

Reported by: jn78 Owned by: jkeller
Priority: major Milestone: 1.4.1
Component: apostrophePlugin Version: 1.4
Keywords: Cc: geoffd,
Symfony version: 1.4

Description

After setting new_slots_top: false new slot appears at the bottom,
but in area it is saved at the top, so when i refresh it apears at
the top.

Change History

Changed 20 months ago by tboutell

  • cc geoffd, added
  • owner changed from boutell to tboutell
  • version set to 1.4
  • status changed from new to accepted
  • milestone set to 1.4.1

Changed 20 months ago by jn78

Suspect:

BaseaSlotActions?.class.php
line: 92

missing top parameter, so it makes adding new slot independent on config setting:

array('permid' => $this->permid, 'slot' => $this->slot, 'top' => sfConfig::get('app_a_new_slots_top', true) ));

Changed 19 months ago by agilbert

  • owner changed from tboutell to jkeller
  • status changed from accepted to assigned

Assigning to Jon for him to take a stab at this.

Jon, attach your changes as a patch.

Changed 19 months ago by jkeller

  • status changed from assigned to closed
  • resolution set to fixed

Just tested it and it works. Tom, it looks like you fixed this in [1586].

Note: See TracTickets for help on using tickets.