← Back to team overview

mahara-contributors team mailing list archive

[Bug 1557825] Re: Remove unused view.numcolumns database field

 

** Changed in: mahara
       Status: In Progress => Fix Committed

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1557825

Title:
  Remove unused view.numcolumns database field

Status in Mahara:
  Fix Committed

Bug description:
  Spun off from Bug 1537861.

  Prior to Mahara 1.8, the view.numcolumns field indicated how many
  columns a view had, which was important because that's pretty much all
  the page layout we had in Mahara 1.7 and earlier.

  In Mahara 1.8 we moved to the more complicated "flexible layout"
  system, where a page can have multiple rows, and each row can have a
  different number of columns and column widths. With that in place, we
  stopped using the view.numcolumns field.

  However, the field stayed in the database for some reason, possibly
  just overlooked or possibly for backwards compatibility. While working
  on Bug 1537861, Robert suggested that we should remove this column,
  because at the moment it's vestigial and just makes things confusing
  for people looking at the database and/or the code.

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1557825/+subscriptions


References