sql - support for scss,sass source formats

[alikon]

Pull Request for Issue #10186 .

Summary of Changes

added sql for #10186 (comment)

[zero-24]

We need to revert the update part as it would overwrite the current settings by the site owners.

[mbabker]

This overwrites the entire component parameters. Not sure that's a good idea...

[zero-24]

@mbabker we just need to revert the update part the install part is correct ;) or do i miss something?

[mbabker]

If you want to add the values during the update, it can't be a full replacement of the parameters column like it is now. If you choose not to add the values then just removing the update SQLs is fine.

[C-Lodder]

Won't the updating part need to be done in administrator\components\com_admin\script.php?

@zero-24 - It will also need to be done in an update, as well as new installation.

[alikon]

lets'go only with install for now , the full update of the param field as noticed was a really bad idea

[andrepereiradasilva]

I have tested this item ✅ successfully on 4edd94d

on code review

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12712.}

[C-Lodder]

I have tested this item ✅ successfully on 4edd94d

I do hope the update side of things isn't forgotten about though

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12712.}

[alikon]

i'm thinking something about this update query:

UPDATE params = 
        replace(params,'"source_formats":"','"source_formats":"scss,sass,')
  FROM #__extensions
where extension_id =20;`

what do you think?

[zero-24]

Thanks!

[alikon]

@C-Lodder not forgotten please test/comment #12727